summaryrefslogtreecommitdiff
path: root/drivers/hid/hid-picolcd_backlight.c
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2014-11-19 18:30:22 +0100
committerJiri Kosina <jkosina@suse.com>2015-06-29 14:51:12 +0200
commit4b8a8262086ece4b7eb34bd2e40cce3b3c9c7079 (patch)
treef7c6bd8249ca0daf1c9632e9a103e89f1787276f /drivers/hid/hid-picolcd_backlight.c
parent67db8a8086e9b865533348954f5547f1e433101e (diff)
HID: picoLCD: Deletion of unnecessary checks before three function calls
The functions backlight_device_unregister(), lcd_device_unregister() and rc_unregister_device() test whether their argument is NULL and then return immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Reviewed-by: Bruno Prémont <bonbons@linux-vserver.org> Signed-off-by: Jiri Kosina <jkosina@suse.com>
Diffstat (limited to 'drivers/hid/hid-picolcd_backlight.c')
-rw-r--r--drivers/hid/hid-picolcd_backlight.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/hid/hid-picolcd_backlight.c b/drivers/hid/hid-picolcd_backlight.c
index a32c5f86b0b3..808807ad388f 100644
--- a/drivers/hid/hid-picolcd_backlight.c
+++ b/drivers/hid/hid-picolcd_backlight.c
@@ -94,8 +94,7 @@ void picolcd_exit_backlight(struct picolcd_data *data)
struct backlight_device *bdev = data->backlight;
data->backlight = NULL;
- if (bdev)
- backlight_device_unregister(bdev);
+ backlight_device_unregister(bdev);
}
int picolcd_resume_backlight(struct picolcd_data *data)