linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] fix: radeon backlight shutoff flashes momentarily
@ 2009-03-11 11:53 Robert T. Drury
  0 siblings, 0 replies; only message in thread
From: Robert T. Drury @ 2009-03-11 11:53 UTC (permalink / raw)
  To: linux-fbdev-devel, benh, akpm

aluminum powerbook G4 with radeon 
The backlight momentarily flashes brightly when turning off.
If the backlight is originally dim, the flash is more noticeable.
This momentary flash is stressful on the power circuit.
The first OUTREG to LVDS_GEN_CNTL causes the flashing problem. 
To fix, remove the first manipulation of lvds_gen_cntl above it.
Second manipulation of lvds_gen_cntl does nothing (level always zero).
The first manipulation was needed so the display would not be 
garbled after turning the backlight back on, but at least on my powerbook,
the garbling seems to go away when the udelay is 1000. 
bugzilla[Bug 12845]

Signed-off-by: Robert Drury <rtdrury@earthlink.net>

--- linux-2.6.23.9/drivers/video/aty/radeon_backlight.c.orig	2009-03-11 01:57:10.000000000 -0700
+++ linux-2.6.23.9/drivers/video/aty/radeon_backlight.c	2009-03-11 02:26:28.000000000 -0700
@@ -101,12 +101,9 @@ static int radeon_bl_update_status(struc
 		tmpPixclksCntl = INPLL(PIXCLKS_CNTL);
 		if (rinfo->is_mobility || rinfo->is_IGP)
 			OUTPLLP(PIXCLKS_CNTL, 0, ~PIXCLK_LVDS_ALWAYS_ONb);
-		lvds_gen_cntl &= ~(LVDS_BL_MOD_LEVEL_MASK | LVDS_BL_MOD_EN);
-		lvds_gen_cntl |= (radeon_bl_get_level_brightness(pdata, 0) <<
-				  LVDS_BL_MOD_LEVEL_SHIFT);
 		lvds_gen_cntl |= LVDS_DISPLAY_DIS;
 		OUTREG(LVDS_GEN_CNTL, lvds_gen_cntl);
-		udelay(100);
+		udelay(1000);
 		lvds_gen_cntl &= ~(LVDS_ON | LVDS_EN);
 		OUTREG(LVDS_GEN_CNTL, lvds_gen_cntl);
 		lvds_gen_cntl &= ~(LVDS_DIGON);



------------------------------------------------------------------------------
Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are
powering Web 2.0 with engaging, cross-platform capabilities. Quickly and
easily build your RIAs with Flex Builder, the Eclipse(TM)based development
software that enables intelligent coding and step-through debugging.
Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2009-03-11 11:52 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-11 11:53 [PATCH] fix: radeon backlight shutoff flashes momentarily Robert T. Drury

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).