[ltp] Re: Bug#448467: xserver-xorg-video-intel: Thinkpad brightness keys don't work

[Jesse Barnes]

On Monday, December 24, 2007 8:09 am Henrique de Moraes Holschuh wrote:
> > Since on Thinkpads, the kernel method is available (either via the
> > kernel thinkpad driver or ACPI) you should try to use that, but it
> > may be that the acpi_video1 method is your best choice.  You can
> > try that interface by changing the order of the string array in
> > i830_lvds.c around line 67.
>
> Patches to fix the kernel to not export a backlight entry for
> inactive ACPI nodes are queued, but really, it is *an extremely bad
> idea* to hardcode *anything* of the sort inside X.org.  It needs to
> be made a configurable *list* of backlight devices, please.

That's good, but there has to be a list somewhere since the kernel 
backlight interface was so poorly designed.  I don't care if it's in 
the driver with an override list in xorg.conf or what, but it has to be 
somewhere.

Jesse