diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-23 11:05:18 -0500 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2014-12-04 09:16:02 -0500 |
commit | 34685627288baa370a6102ce58d47266d24050fd (patch) | |
tree | 50d76e392c05059be905a305bba22d4f7d363939 /drivers/video | |
parent | 772849512798833daca4d88869c15604af3fbb9f (diff) |
video: uvesafb: Deletion of an unnecessary check before the function call "platform_device_put"
The platform_device_put() 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: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/fbdev/uvesafb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/video/fbdev/uvesafb.c b/drivers/video/fbdev/uvesafb.c index f718627b368f..d32d1c4d1b99 100644 --- a/drivers/video/fbdev/uvesafb.c +++ b/drivers/video/fbdev/uvesafb.c | |||
@@ -1922,8 +1922,7 @@ static int uvesafb_init(void) | |||
1922 | err = -ENOMEM; | 1922 | err = -ENOMEM; |
1923 | 1923 | ||
1924 | if (err) { | 1924 | if (err) { |
1925 | if (uvesafb_device) | 1925 | platform_device_put(uvesafb_device); |
1926 | platform_device_put(uvesafb_device); | ||
1927 | platform_driver_unregister(&uvesafb_driver); | 1926 | platform_driver_unregister(&uvesafb_driver); |
1928 | cn_del_callback(&uvesafb_cn_id); | 1927 | cn_del_callback(&uvesafb_cn_id); |
1929 | return err; | 1928 | return err; |