lkml.org 
[lkml]   [2007]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] Map volume and brightness events on thinkpads
Date
On Tuesday, October 16, 2007 1:36 am Henrique de Moraes Holschuh wrote:
> > No, on Lenovo (and in general actually) the firmware should *not*
> > touch the backlight. Otherwise if another driver touches it the
> > driver and
>
> This is not an option on IBM ThinkPads, unless you patch the DSDT on
> non-ancient ACPI-based models, and unless you patch the BIOS (and
> maybe even the EC control program) itself on the really ancient
> models. It is that simple.
>

Yeah, unfortunately, "should not" doesn't mean "does not" in this case.
In cases where we can't control the firmware, we have to use it to
control the backlight exclusively, or we'll definitely get into
trouble.

> You want ACPI video to just pass the messages to userspace when X.org
> is driving the backlight? Fine with me. That *still* doesn't make
> it right to get these messages as hot key presses over the input
> layer through the thinkpad-acpi driver. So the NAK stands. Any
> changes should be done to the ACPI video driver in this case.

So is this really the direction that input is going? Last time I talked
with Dmitry, he seemed ok with adding input events for ACPI and other
firmware hotkeys...

> Good luck, and please interface it properly to the backlight class
> while at it. There's no reason to make the waters mudier :-)

Yeah, the backlight class needs some changes: the backlight dirs should
somehow identify the display they control and only *one* backlight
driver should be registered for a given display. Unfortunately we
don't have enough info in the kernel to identify displays, so those
changes may have to wait until the DRM grows such knowledge.

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

\
 
 \ /
  Last update: 2007-10-16 10:53    [W:0.077 / U:0.396 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site