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

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Henrique de Moraes Holschuh
Date: Tuesday, October 16, 2007 - 7:11 am

On Tue, 16 Oct 2007, Matthew Garrett wrote:

Yes, it does.  But none of it cannot be fixed, HAL is the really big
offender in userspace, and laptop drivers are the really big offenders in
kernel space.


It is not clear to me if they are notifications or not.  Does the firmware
act on the keys by itself? If it does, then they are notifications.  If it
does not, then they are regular hot keys and there is no controversy whether
they belong on the input layer or not (they do).


Well, I better make it clear once again. Please excuse me the very direct
and acid tone in the rest of this email, I want to make certain points
crystal clear to everyone (and not just you, I do believe you are already
very aware of my position in this):

1. I am not against sending notification events through input (but see point
two below).  However, AFAIK, Dmitry is.  He said as much in the last thread
about it.  I have added him to the CC list, he can make his position clear
in this thread, if I got it wrong.

Until I get a "you can do it" from Dmitry, forget about thinkpad-acpi
sending such events over the input layer.  I believe in a coherent kernel
where the schizophrenia on the use of the various interfaces and subsystems
is kept to a bare minimum.  If a subsystem maintainers tell me not to do
something, I won't do it.

2. I am against sending notification events through input **that look
exactly the same as regular events**.  That is not a wise design choice IMO,
it is a very dirty hack.

I did propose ways to fix that properly, though (and to write the patches to
do so).  Two simple and clean ways come to mind:

	1. Add a new EV_* class for such events.
	2. Alternatively, add a flags field to the higher bits of EV_*
	   class that could be used to mark events in a proper way.

And I am sure there are other ways too, so it *can* be done properly if it
is to be done at all.  However, it was not accepted because Dmitry does not
want notifications going over the input subsystem as far as I recall from
that thread.  Get Dmitry to accept a way to send notifications though the
input layer, and I will follow it.

3. We have a backlight class, a LED class, a rf-kill class and ALSA mixers.
Is there a real reason to pester Dmitry about the issue, if we can use these
alternate paths (that are indeed more generic and more suited for the job)?

-- 
  "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, 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)