Commit de110fdd authored Apr 11, 2019 by YueHaibing Committed by Bartlomiej Zolnierkiewicz Apr 11, 2019

video: fbdev: mxsfb: remove set but not used variable 'line_count'



Fixes gcc '-Wunused-but-set-variable' warning:

drivers/video/fbdev/mxsfb.c: In function 'mxsfb_restore_mode':
drivers/video/fbdev/mxsfb.c:633:11: warning:
 variable 'line_count' set but not used [-Wunused-but-set-variable]

It's never used since introduction in commit f0a523b5 ("video: Add i.MX23/28
framebuffer driver") and can be removed.

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Melchior Franz <melchior.franz@ginzinger.com>
[b.zolnierkie: minor patch summary fixup]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

parent 730dd516

drivers/video/fbdev/mxsfb.c

+0 −2

Original line number	Diff line number	Diff line
		@@ -630,7 +630,6 @@ static int mxsfb_restore_mode(struct fb_info *fb_info,
		struct fb_videomode *vmode)
		{
		struct mxsfb_info *host = fb_info->par;
		unsigned line_count;
		unsigned period;
		unsigned long pa, fbsize;
		int bits_per_pixel, ofs, ret = 0;
		@@ -717,7 +716,6 @@ static int mxsfb_restore_mode(struct fb_info *fb_info,
		writel(fb_info->fix.smem_start, host->base + host->devdata->next_buf);
		}

		line_count = fb_info->fix.smem_len / fb_info->fix.line_length;
		fb_info->fix.ypanstep = 1;

		clk_prepare_enable(host->clk);

Admin message