video: fbdev: mxsfb: remove set but not used variable 'line_count'
authorYueHaibing <yuehaibing@huawei.com>
Thu, 11 Apr 2019 17:25:12 +0000 (19:25 +0200)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Thu, 11 Apr 2019 17:25:12 +0000 (19:25 +0200)
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 f0a523b5e5e2 ("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>
drivers/video/fbdev/mxsfb.c

index a017200..1fdd1eb 100644 (file)
@@ -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);