Commit 673b4600 authored by Ben Dooks's avatar Ben Dooks Committed by Linus Torvalds

S3C2410: ensure that FB_BLANK_POWERDOWN shuts down the controller

When a blank level of FB_BLANK_POWERDOWN is used, we should shut down the
controller so that it no longer tries to produce any panel signals or
data, and shuts down the DMA which is not needed.
Signed-off-by: default avatarBen Dooks <ben-linux@fluff.org>
Cc: Arnaud Patard <arnaud.patard@rtp-net.org>
Acked-by: default avatarKrzysztof Helt <krzysztof.h1@wp.pl>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent cdc83ae2
...@@ -580,6 +580,27 @@ static int s3c2410fb_setcolreg(unsigned regno, ...@@ -580,6 +580,27 @@ static int s3c2410fb_setcolreg(unsigned regno,
return 0; return 0;
} }
/* s3c2410fb_lcd_enable
*
* shutdown the lcd controller
*/
static void s3c2410fb_lcd_enable(struct s3c2410fb_info *fbi, int enable)
{
unsigned long flags;
local_irq_save(flags);
if (enable)
fbi->regs.lcdcon1 |= S3C2410_LCDCON1_ENVID;
else
fbi->regs.lcdcon1 &= ~S3C2410_LCDCON1_ENVID;
writel(fbi->regs.lcdcon1, fbi->io + S3C2410_LCDCON1);
local_irq_restore(flags);
}
/* /*
* s3c2410fb_blank * s3c2410fb_blank
* @blank_mode: the blank mode we want. * @blank_mode: the blank mode we want.
...@@ -589,9 +610,6 @@ static int s3c2410fb_setcolreg(unsigned regno, ...@@ -589,9 +610,6 @@ static int s3c2410fb_setcolreg(unsigned regno,
* blanking succeeded, != 0 if un-/blanking failed due to e.g. a * blanking succeeded, != 0 if un-/blanking failed due to e.g. a
* video mode which doesn't support it. Implements VESA suspend * video mode which doesn't support it. Implements VESA suspend
* and powerdown modes on hardware that supports disabling hsync/vsync: * and powerdown modes on hardware that supports disabling hsync/vsync:
* blank_mode == 2: suspend vsync
* blank_mode == 3: suspend hsync
* blank_mode == 4: powerdown
* *
* Returns negative errno on error, or zero on success. * Returns negative errno on error, or zero on success.
* *
...@@ -605,6 +623,12 @@ static int s3c2410fb_blank(int blank_mode, struct fb_info *info) ...@@ -605,6 +623,12 @@ static int s3c2410fb_blank(int blank_mode, struct fb_info *info)
tpal_reg += is_s3c2412(fbi) ? S3C2412_TPAL : S3C2410_TPAL; tpal_reg += is_s3c2412(fbi) ? S3C2412_TPAL : S3C2410_TPAL;
if (blank_mode == FB_BLANK_POWERDOWN) {
s3c2410fb_lcd_enable(fbi, 0);
} else {
s3c2410fb_lcd_enable(fbi, 1);
}
if (blank_mode == FB_BLANK_UNBLANK) if (blank_mode == FB_BLANK_UNBLANK)
writel(0x0, tpal_reg); writel(0x0, tpal_reg);
else { else {
...@@ -983,21 +1007,6 @@ static int __init s3c2412fb_probe(struct platform_device *pdev) ...@@ -983,21 +1007,6 @@ static int __init s3c2412fb_probe(struct platform_device *pdev)
return s3c24xxfb_probe(pdev, DRV_S3C2412); return s3c24xxfb_probe(pdev, DRV_S3C2412);
} }
/* s3c2410fb_stop_lcd
*
* shutdown the lcd controller
*/
static void s3c2410fb_stop_lcd(struct s3c2410fb_info *fbi)
{
unsigned long flags;
local_irq_save(flags);
fbi->regs.lcdcon1 &= ~S3C2410_LCDCON1_ENVID;
writel(fbi->regs.lcdcon1, fbi->io + S3C2410_LCDCON1);
local_irq_restore(flags);
}
/* /*
* Cleanup * Cleanup
...@@ -1010,7 +1019,7 @@ static int s3c2410fb_remove(struct platform_device *pdev) ...@@ -1010,7 +1019,7 @@ static int s3c2410fb_remove(struct platform_device *pdev)
unregister_framebuffer(fbinfo); unregister_framebuffer(fbinfo);
s3c2410fb_stop_lcd(info); s3c2410fb_lcd_enable(info, 0);
msleep(1); msleep(1);
s3c2410fb_unmap_video_memory(fbinfo); s3c2410fb_unmap_video_memory(fbinfo);
...@@ -1043,7 +1052,7 @@ static int s3c2410fb_suspend(struct platform_device *dev, pm_message_t state) ...@@ -1043,7 +1052,7 @@ static int s3c2410fb_suspend(struct platform_device *dev, pm_message_t state)
struct fb_info *fbinfo = platform_get_drvdata(dev); struct fb_info *fbinfo = platform_get_drvdata(dev);
struct s3c2410fb_info *info = fbinfo->par; struct s3c2410fb_info *info = fbinfo->par;
s3c2410fb_stop_lcd(info); s3c2410fb_lcd_enable(info, 0);
/* sleep before disabling the clock, we need to ensure /* sleep before disabling the clock, we need to ensure
* the LCD DMA engine is not going to get back on the bus * the LCD DMA engine is not going to get back on the bus
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment