Re: 2.6.21-rc2 radeon backlight

[Richard Purdie]

On Sat, 2007-03-03 at 16:06 +0000, James Simmons wrote:
> > > Richard, is this actually a bug, or is it a config error or something like that?
> > > 
> > > And should we track it as a post-2.6.20 regression?
> > 
> > Its a regression IMO. Arguably its a Kconfig error but a nasty one as
> > the defaults cause the problems. Different people seem to have different
> > interpretations but to me it appears that the patch from James causes
> > backlights to fail to turn on for a variety of devices which worked
> > before.
> 
>    It is NOT a Kconfig error. The problem is that for many fbdev drivers 
> the backlight code is broken. For some magic reason it only works on 
> pmacs. Most likely because the firmware properly sets up the backlight. 
> Plus we have the conflict with acpi backlight. 
>    Think about it. Enabling the backlight code breaks the backlight. 
> Without the backlight driver the default behaviour of the backlight works.
> Who sets up the default behavior? I don't have a LCD panel with a 
> backlight othewise I would track the problem down.

I not arguing several fb driver's backlight code isn't broken, it is. We
have a Kconfig problem though since we used to stop users selecting
things that didn't work and now we're allowing them. Worse still, the
defaults break for people. That is a regression.

Anyhow, the patch I proposed should let people enable/disable it at
runtime with defaults known to work which should address the problem
until someone figures out how to fix the backlight drivers themselves
properly.

Cheers,

Richard

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/