diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-06-28 09:35:03 -0400 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2014-06-30 03:54:22 -0400 |
commit | ff2019456d745d4ebc1307eba128393a39790217 (patch) | |
tree | 4343761e351f02a17e154666867ff75327af8850 | |
parent | 0a9597b70c76234babf7650437578abaeb9259e1 (diff) |
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>
-rw-r--r-- | drivers/hid/hid-picolcd_debugfs.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/hid/hid-picolcd_debugfs.c b/drivers/hid/hid-picolcd_debugfs.c index 024cdf3c2297..3c13af684410 100644 --- a/drivers/hid/hid-picolcd_debugfs.c +++ b/drivers/hid/hid-picolcd_debugfs.c | |||
@@ -883,16 +883,13 @@ void picolcd_exit_devfs(struct picolcd_data *data) | |||
883 | 883 | ||
884 | dent = data->debug_reset; | 884 | dent = data->debug_reset; |
885 | data->debug_reset = NULL; | 885 | data->debug_reset = NULL; |
886 | if (dent) | 886 | debugfs_remove(dent); |
887 | debugfs_remove(dent); | ||
888 | dent = data->debug_eeprom; | 887 | dent = data->debug_eeprom; |
889 | data->debug_eeprom = NULL; | 888 | data->debug_eeprom = NULL; |
890 | if (dent) | 889 | debugfs_remove(dent); |
891 | debugfs_remove(dent); | ||
892 | dent = data->debug_flash; | 890 | dent = data->debug_flash; |
893 | data->debug_flash = NULL; | 891 | data->debug_flash = NULL; |
894 | if (dent) | 892 | debugfs_remove(dent); |
895 | debugfs_remove(dent); | ||
896 | mutex_destroy(&data->mutex_flash); | 893 | mutex_destroy(&data->mutex_flash); |
897 | } | 894 | } |
898 | 895 | ||