Home » Mailing list archives » linux-kernel » 2007 » October » 15

Re: [PATCH] Map volume and brightness events on thinkpads

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Henrique de Moraes Holschuh <hmh@...>
To: Jeremy Katz <katzj@...>
Cc: <linux-kernel@...>, <torvalds@...>, <davej@...>
Subject: Re: [PATCH] Map volume and brightness events on thinkpads
Date: Monday, October 15, 2007 - 5:07 pm

On Mon, 15 Oct 2007, Jeremy Katz wrote:

NAK.  It is the completely wrong thing to do for IBM thinkpads which process
volume and brightness completely in firmware.

And the input subsystem maintainer has made it extremely clear in various
threads that the input devices are *not* to be used as a notification
service for on-screen-display or other such stuff.  If you send volume and
brightness *key* events to userspace, it is supposed to act on them and
raise/lower brightness/volume, which is the wrong thing to do on thinkpads.
Never mind that HAL is ignoring the input maintainer's directions and
violating this.

We should fix the backlight class to be more useful and support poll() or
somesuch, for userspace to track the backlight level in a resource-friendly
way for OSD (the only sane thing to do on an IBM thinkpad with such events).
And an ALSA mixer to provide a proper path to the thinkpad-acpi volume
functionality is also in my schedule for 2.6.25.

As for Lenovo thinkpads, brightness control is to be processed by the ACPI
video module, so brightness hot keys are not to be reported by default there
either.  I am not so sure about the volume keys, but your patch touches the
IBM keymap *and* you provide no testing information for the various Lenovo
models, so I have to NAK it as well until more information is available.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH] Map volume and brightness events on thinkpads, Jeremy Katz, (Mon Oct 15, 4:45 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 2:53 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Renato S. Yamane, (Tue Oct 16, 3:23 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Mon Oct 15, 5:07 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 9:00 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 16, 10:11 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 10:27 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 10:54 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 11:54 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 11:59 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 10:21 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 16, 10:31 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 10:40 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 16, 12:56 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 2:46 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Linus Torvalds, (Tue Oct 16, 3:14 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dave Airlie, (Tue Oct 16, 4:32 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 4:12 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Linus Torvalds, (Tue Oct 16, 4:32 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Wed Oct 17, 11:57 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Wed Oct 17, 12:28 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Wed Oct 17, 1:35 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Wed Oct 17, 2:59 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Wed Oct 17, 4:42 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Thu Oct 18, 10:37 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Thu Oct 18, 6:39 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 23, 11:54 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 23, 7:34 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Kok, Auke, (Tue Oct 16, 4:48 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 4:55 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 5:18 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Jeremy Katz, (Tue Oct 16, 4:11 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 4:19 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 3:20 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 16, 4:02 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 4:15 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Jesse Barnes, (Mon Oct 15, 11:38 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 16, 4:36 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Jesse Barnes, (Tue Oct 16, 4:49 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 16, 5:18 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Jesse Barnes, (Tue Oct 16, 10:40 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Wed Oct 17, 2:25 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Jesse Barnes, (Wed Oct 17, 2:32 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Wed Oct 17, 2:39 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Arjan van de Ven, (Mon Oct 15, 5:43 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Jeremy Katz, (Mon Oct 15, 5:27 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes Holschuh..., (Tue Oct 16, 5:12 am)