diff options
author | Fancy Fang <chen.fang@nxp.com> | 2019-12-03 11:26:16 +0800 |
---|---|---|
committer | Fancy Fang <chen.fang@nxp.com> | 2019-12-03 17:54:39 +0800 |
commit | ece58701447200ba093861b0acd5b3d0a96f592b (patch) | |
tree | 31dabc645bc1de8d8fdb1de6537e3f60fd9ad69b /drivers | |
parent | 0d583079c7308e8f29cfabacf7d7b7ea39b4b0d5 (diff) |
vide: fbdev: mxsfb: don't check return value for lock_fb_info()
Since the commit
commit cf4a3ae4ef3399179166a464af1d6b172225bef4
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date: Tue May 28 11:02:47 2019 +0200
fbdev: lock_fb_info cannot fail
has changed the lock_fb_info() to void type, so cannot check its
return value anymore.
Signed-off-by: Fancy Fang <chen.fang@nxp.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/video/fbdev/mxsfb.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/drivers/video/fbdev/mxsfb.c b/drivers/video/fbdev/mxsfb.c index b6d2d93b031b..81b4a1c46203 100644 --- a/drivers/video/fbdev/mxsfb.c +++ b/drivers/video/fbdev/mxsfb.c @@ -1620,8 +1620,7 @@ static void overlayfb_enable(struct mxsfb_layer *ofb) { struct mxsfb_info *fbi = ofb->fbi; - if (!lock_fb_info(fbi->fb_info)) - return; + lock_fb_info(fbi->fb_info); if (fbi->cur_blank == FB_BLANK_UNBLANK) { mxsfb_disable_controller(fbi->fb_info); @@ -1876,8 +1875,7 @@ static int overlayfb_set_par(struct fb_info *info) if (ofb->video_mem_size < size) return -EINVAL; - if (!lock_fb_info(fbi->fb_info)) - return -EINVAL; + lock_fb_info(fbi->fb_info); if (fbi->cur_blank != FB_BLANK_UNBLANK) { clk_enable_pix(fbi); @@ -1894,8 +1892,7 @@ static int overlayfb_set_par(struct fb_info *info) if (ofb->blank_state == FB_BLANK_UNBLANK) ofb->ops->enable(ofb); - if (!lock_fb_info(fbi->fb_info)) - return -EINVAL; + lock_fb_info(fbi->fb_info); if (fbi->cur_blank != FB_BLANK_UNBLANK) { clk_disable_disp_axi(fbi); @@ -1919,8 +1916,7 @@ static int overlayfb_blank(int blank, struct fb_info *info) if (ofb->blank_state == blank) return 0; - if (!lock_fb_info(fbi->fb_info)) - return -EINVAL; + lock_fb_info(fbi->fb_info); if (fbi->cur_blank != FB_BLANK_UNBLANK) { clk_enable_pix(fbi); @@ -1941,8 +1937,7 @@ static int overlayfb_blank(int blank, struct fb_info *info) break; } - if (!lock_fb_info(fbi->fb_info)) - return -EINVAL; + lock_fb_info(fbi->fb_info); if (fbi->cur_blank != FB_BLANK_UNBLANK) { clk_disable_disp_axi(fbi); @@ -1966,8 +1961,7 @@ static int overlayfb_pan_display(struct fb_var_screeninfo *var, init_completion(&fbi->flip_complete); - if (!lock_fb_info(fbi->fb_info)) - return -EINVAL; + lock_fb_info(fbi->fb_info); if (fbi->cur_blank != FB_BLANK_UNBLANK) { unlock_fb_info(fbi->fb_info); |