video: uvesafb: Deletion of an unnecessary check before the function call "uvesafb_free"
authorMarkus Elfring <elfring@users.sourceforge.net>
Sun, 23 Nov 2014 10:00:26 +0000 (11:00 +0100)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 4 Dec 2014 14:16:01 +0000 (16:16 +0200)
The uvesafb_free() 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>
drivers/video/fbdev/uvesafb.c

index 509d452..f718627 100644 (file)
@@ -1219,8 +1219,7 @@ static int uvesafb_release(struct fb_info *info, int user)
        uvesafb_vbe_state_restore(par, par->vbe_state_orig);
 out:
        atomic_dec(&par->ref_count);
-       if (task)
-               uvesafb_free(task);
+       uvesafb_free(task);
        return 0;
 }