video: mxsfb: fix crash when unblanking the display
authorLothar Waßmann <LW@KARO-electronics.de>
Thu, 22 Nov 2012 12:49:14 +0000 (13:49 +0100)
committerShawn Guo <shawn.guo@linaro.org>
Fri, 4 Jan 2013 11:52:53 +0000 (19:52 +0800)
The VDCTRL4 register does not provide the MXS SET/CLR/TOGGLE feature.
The write in mxsfb_disable_controller() sets the data_cnt for the LCD
DMA to 0 which obviously means the max. count for the LCD DMA and
leads to overwriting arbitrary memory when the display is unblanked.

Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
Acked-by: Juergen Beisert <jbe@pengutronix.de>
Tested-by: Lauri Hintsala <lauri.hintsala@bluegiga.net>
Cc: <stable@vger.kernel.org>
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
drivers/video/mxsfb.c

index 49619b4415000efc1ef87756d09667017a682244..f2a49ef772f3b59c47fd0cf12e81218e5ddb0325 100644 (file)
@@ -369,7 +369,8 @@ static void mxsfb_disable_controller(struct fb_info *fb_info)
                loop--;
        }
 
-       writel(VDCTRL4_SYNC_SIGNALS_ON, host->base + LCDC_VDCTRL4 + REG_CLR);
+       reg = readl(host->base + LCDC_VDCTRL4);
+       writel(reg & ~VDCTRL4_SYNC_SIGNALS_ON, host->base + LCDC_VDCTRL4);
 
        clk_disable_unprepare(host->clk);