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

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linus Torvalds
Date: Tuesday, October 16, 2007 - 12:14 pm

On Tue, 16 Oct 2007, Matthew Garrett wrote:

To get somewhat back on track: volume and brightness (and similar - lid 
close etc) events clearly are keypresses.

However, I would also argue that a keypress that is acted on by the 
firmware automatically is *different* from a keypress that hasn't been 
acted on: one is a "key was pressed *and* hardware did something 
automatically", and the other is just a "key was pressed" event.

IOW, I think the thinkpad issue (and others like it) should be fixed by 
splitting up the KEY_VOLUMEUP "key" into separate KEY_VOLUMEUP and 
KEY_VOLUMEUP_NOTIFY key events, so that downstream user mode (and the 
kernel itself, for that matter) can know whether it's a informational 
message or whether it should be acted upon.

Because clearly we seem to have both cases, and while I think we should 
try to move towards a "user mode does all actions" model where screen 
brightness is under the control of X, that isn't necessarily the case now, 
nor perhaps even reachable on all hw platforms.

Hmm?

		Linus
-
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, 1:45 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Mon Oct 15, 2:07 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Arjan van de Ven, (Mon Oct 15, 2:43 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 1:36 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 2:12 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 2:18 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 6:00 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 7:11 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 7:21 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 7:27 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 7:31 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 7:40 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 7:54 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 8:54 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 8:59 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 9:56 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 11:46 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 11:53 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Linus Torvalds, (Tue Oct 16, 12:14 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 12:20 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Renato S. Yamane, (Tue Oct 16, 12:23 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 1:02 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 1:12 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 1:15 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 1:19 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Linus Torvalds, (Tue Oct 16, 1:32 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Tue Oct 16, 1:55 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 16, 2:18 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 11:25 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Jesse Barnes, (Tue Oct 16, 11:32 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 16, 11:39 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Wed Oct 17, 8:57 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Matthew Garrett, (Wed Oct 17, 9:28 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Wed Oct 17, 10:35 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Wed Oct 17, 11:59 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Wed Oct 17, 1:42 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Thu Oct 18, 7:37 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Thu Oct 18, 3:39 pm)
Re: [PATCH] Map volume and brightness events on thinkpads, Dmitry Torokhov, (Tue Oct 23, 8:54 am)
Re: [PATCH] Map volume and brightness events on thinkpads, Henrique de Moraes H ..., (Tue Oct 23, 4:34 pm)