summary refs log tree commit diff
path: root/drivers/pci/hotplug/cpqphp_sysfs.c
diff options
context:
space:
mode:
authorFabian Frederick <fabf@skynet.be>2014-06-28 11:44:43 +0200
committerBjorn Helgaas <bhelgaas@google.com>2014-07-07 14:53:44 -0600
commit5d37818b9ccbec660abb3a11d4ffb0aba0e3c809 (patch)
treeb1c70752a91361f5cbd530660ad2a611d4d94a8a /drivers/pci/hotplug/cpqphp_sysfs.c
parent0d25d35c987d7b0b63368d9c1ae35a917e1a7bab (diff)
downloadlinux-5d37818b9ccbec660abb3a11d4ffb0aba0e3c809.tar.gz
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.c3
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;
 }