summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorWen Yang <wen.yang99@zte.com.cn>2018-12-20 19:13:08 +0100
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>2018-12-20 19:13:08 +0100
commit5c29085a724fb8b84bb492a6e2645e28d9bc3dae (patch)
treefbd15f6e8d6782d5b9edc44d0887eb2d2d0e567b /drivers
parentfbdev: omap2: omapfb: convert to DEFINE_SHOW_ATTRIBUTE (diff)
downloadlinux-5c29085a724fb8b84bb492a6e2645e28d9bc3dae.tar.xz
linux-5c29085a724fb8b84bb492a6e2645e28d9bc3dae.zip
fbdev: fsl-diu: remove redundant null check on cmap
The null check on &info->cmap is redundant since cmap is a struct inside fb_info and can never be null, so the check is always true. We may remove it. Signed-off-by: Wen Yang <wen.yang99@zte.com.cn> Acked-by: Timur Tabi <timur@kernel.org> Cc: zhong.weidong@zte.com.cn Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/video/fbdev/fsl-diu-fb.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/video/fbdev/fsl-diu-fb.c b/drivers/video/fbdev/fsl-diu-fb.c
index 332a56b6811f..9a5451ba4d44 100644
--- a/drivers/video/fbdev/fsl-diu-fb.c
+++ b/drivers/video/fbdev/fsl-diu-fb.c
@@ -1575,8 +1575,7 @@ static void uninstall_fb(struct fb_info *info)
unregister_framebuffer(info);
unmap_video_memory(info);
- if (&info->cmap)
- fb_dealloc_cmap(&info->cmap);
+ fb_dealloc_cmap(&info->cmap);
mfbi->registered = 0;
}