diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-06-28 11:44:43 +0200 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-07-07 22:53:44 +0200 |
commit | 5d37818b9ccbec660abb3a11d4ffb0aba0e3c809 (patch) | |
tree | b1c70752a91361f5cbd530660ad2a611d4d94a8a /drivers/pci/hotplug | |
parent | PCI: pciehp: Clear Data Link Layer State Changed during init (diff) | |
download | linux-5d37818b9ccbec660abb3a11d4ffb0aba0e3c809.tar.xz linux-5d37818b9ccbec660abb3a11d4ffb0aba0e3c809.zip |
PCI: cpqphp: Remove unnecessary null test before debugfs_remove()
Fix checkpatch warning:
"WARNING: debugfs_remove(NULL) is safe this check is probably not required"
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
CC: Ryan Desfosses <ryan@desfo.org>
Diffstat (limited to 'drivers/pci/hotplug')
-rw-r--r-- | drivers/pci/hotplug/cpqphp_sysfs.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pci/hotplug/cpqphp_sysfs.c b/drivers/pci/hotplug/cpqphp_sysfs.c index 4a392c44e3d3..d81648f71425 100644 --- a/drivers/pci/hotplug/cpqphp_sysfs.c +++ b/drivers/pci/hotplug/cpqphp_sysfs.c @@ -216,8 +216,7 @@ void cpqhp_create_debugfs_files(struct controller *ctrl) void cpqhp_remove_debugfs_files(struct controller *ctrl) { - if (ctrl->dentry) - debugfs_remove(ctrl->dentry); + debugfs_remove(ctrl->dentry); ctrl->dentry = NULL; } |