diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-19 20:30:22 +0300 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.com> | 2015-06-29 15:51:12 +0300 |
| commit | 4b8a8262086ece4b7eb34bd2e40cce3b3c9c7079 (patch) | |
| tree | f7c6bd8249ca0daf1c9632e9a103e89f1787276f /drivers/hid/hid-picolcd_backlight.c | |
| parent | 67db8a8086e9b865533348954f5547f1e433101e (diff) | |
| download | linux-4b8a8262086ece4b7eb34bd2e40cce3b3c9c7079.tar.xz | |
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.c | 3 |
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) |