diff options
| author | Fabian Frederick <fabf@skynet.be> | 2014-06-28 11:44:43 +0200 |
|---|---|---|
| committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-07-07 14:53:44 -0600 |
| commit | 5d37818b9ccbec660abb3a11d4ffb0aba0e3c809 (patch) | |
| tree | b1c70752a91361f5cbd530660ad2a611d4d94a8a /drivers/pci/hotplug/cpqphp_sysfs.c | |
| parent | 0d25d35c987d7b0b63368d9c1ae35a917e1a7bab (diff) | |
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/cpqphp_sysfs.c')
| -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; } |