diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-23 05:00:26 -0500 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2014-12-04 09:16:01 -0500 |
commit | e87d9f295a7f3e9e7aae7a64f8fa9b45ceb1bf1c (patch) | |
tree | aab6f7524472ead6bea65aed7761660d58519842 | |
parent | f858275869af5f2be58782cfaffcf56944bbfe39 (diff) |
video: uvesafb: Deletion of an unnecessary check before the function call "uvesafb_free"
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>
-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 509d452e8f91..f718627b368f 100644 --- a/drivers/video/fbdev/uvesafb.c +++ b/drivers/video/fbdev/uvesafb.c | |||
@@ -1219,8 +1219,7 @@ static int uvesafb_release(struct fb_info *info, int user) | |||
1219 | uvesafb_vbe_state_restore(par, par->vbe_state_orig); | 1219 | uvesafb_vbe_state_restore(par, par->vbe_state_orig); |
1220 | out: | 1220 | out: |
1221 | atomic_dec(&par->ref_count); | 1221 | atomic_dec(&par->ref_count); |
1222 | if (task) | 1222 | uvesafb_free(task); |
1223 | uvesafb_free(task); | ||
1224 | return 0; | 1223 | return 0; |
1225 | } | 1224 | } |
1226 | 1225 | ||