diff options
| author | Lothar Waßmann <LW@KARO-electronics.de> | 2012-11-22 13:49:14 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-01-17 08:43:57 -0800 |
| commit | fc65b0bb8b56913b0610588d83c78d0e5e968e9d (patch) | |
| tree | 4bf3ce9f8a95693ae369b2478a8c18cb5464196b /drivers/video | |
| parent | f125f803570b6e617ea417d08091703e81951d87 (diff) | |
| download | kernel_samsung_smdk4412-fc65b0bb8b56913b0610588d83c78d0e5e968e9d.zip kernel_samsung_smdk4412-fc65b0bb8b56913b0610588d83c78d0e5e968e9d.tar.gz kernel_samsung_smdk4412-fc65b0bb8b56913b0610588d83c78d0e5e968e9d.tar.bz2 | |
video: mxsfb: fix crash when unblanking the display
commit 6c1ecba8d84841277d68140ef485335d5be28485 upstream.
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>
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/mxsfb.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/video/mxsfb.c b/drivers/video/mxsfb.c index 0b2f2dd..2f90290 100644 --- a/drivers/video/mxsfb.c +++ b/drivers/video/mxsfb.c @@ -365,7 +365,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(host->clk); |