summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-02-05 16:07:43 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-03-18 16:56:40 +0100
commitc60b89aa3640ac4eb279213d93828a8100acb6b5 (patch)
tree8a6d58e6dbd560d4bf89f79c345b7b10d49e73dc
parent533726f736c2c32956d78ad6f49ffaada110cbfd (diff)
downloadlwn-c60b89aa3640ac4eb279213d93828a8100acb6b5.tar.gz
lwn-c60b89aa3640ac4eb279213d93828a8100acb6b5.zip
USB: appledisplay: Deletion of a check before backlight_device_unregister()
The backlight_device_unregister() function tests whether its argument is NULL and then returns 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> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/usb/misc/appledisplay.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/misc/appledisplay.c b/drivers/usb/misc/appledisplay.c
index b3d245ef46ef..a0a3827b4aff 100644
--- a/drivers/usb/misc/appledisplay.c
+++ b/drivers/usb/misc/appledisplay.c
@@ -329,7 +329,7 @@ error:
pdata->urbdata, pdata->urb->transfer_dma);
usb_free_urb(pdata->urb);
}
- if (pdata->bd && !IS_ERR(pdata->bd))
+ if (!IS_ERR(pdata->bd))
backlight_device_unregister(pdata->bd);
kfree(pdata->msgdata);
}