Browse Source

HID: picolcd: 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: Jiri Kosina <jkosina@suse.cz>
Fabian Frederick 11 years ago
parent
commit
ff2019456d
1 changed files with 3 additions and 6 deletions
  1. 3 6
      drivers/hid/hid-picolcd_debugfs.c

+ 3 - 6
drivers/hid/hid-picolcd_debugfs.c

@@ -883,16 +883,13 @@ void picolcd_exit_devfs(struct picolcd_data *data)
 
 
 	dent = data->debug_reset;
 	dent = data->debug_reset;
 	data->debug_reset = NULL;
 	data->debug_reset = NULL;
-	if (dent)
-		debugfs_remove(dent);
+	debugfs_remove(dent);
 	dent = data->debug_eeprom;
 	dent = data->debug_eeprom;
 	data->debug_eeprom = NULL;
 	data->debug_eeprom = NULL;
-	if (dent)
-		debugfs_remove(dent);
+	debugfs_remove(dent);
 	dent = data->debug_flash;
 	dent = data->debug_flash;
 	data->debug_flash = NULL;
 	data->debug_flash = NULL;
-	if (dent)
-		debugfs_remove(dent);
+	debugfs_remove(dent);
 	mutex_destroy(&data->mutex_flash);
 	mutex_destroy(&data->mutex_flash);
 }
 }