greybus: es1: no need to check for NULL on debugfs_remove()
authorGreg Kroah-Hartman <greg@kroah.com>
Tue, 24 Mar 2015 19:06:41 +0000 (20:06 +0100)
committerGreg Kroah-Hartman <greg@kroah.com>
Tue, 24 Mar 2015 19:06:41 +0000 (20:06 +0100)
The function can, and even expects NULL, so don't check.

Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
drivers/staging/greybus/es1.c

index cce3155..723d8b7 100644 (file)
@@ -558,10 +558,8 @@ static void usb_log_enable(struct es1_ap_dev *es1, int enable)
                                                        gb_debugfs_get(), NULL,
                                                        &apb1_log_fops);
        } else {
-               if (apb1_log_dentry) {
-                       debugfs_remove(apb1_log_dentry);
-                       apb1_log_dentry = NULL;
-               }
+               debugfs_remove(apb1_log_dentry);
+               apb1_log_dentry = NULL;
 
                if (apb1_log_task) {
                        kthread_stop(apb1_log_task);