On Fri, 01 Jun 2007, Matthew Garrett wrote: > On Thu, May 31, 2007 at 07:28:14PM -0300, Henrique de Moraes Holschuh wrote: > > We have all the pieces needed to have sane, generic userland keyboard handling > > in place for a while now, but it was not sufficiently documented (or used!). > > > > If EV_KEY input drivers always generate scan codes that can be used to > > reprogram their keycode maps, and always generate EV_MSC MSC_SCAN events when > > they output an EV_KEY KEY_UNKNOWN event, userspace can trap those and feed it > > to a generic helper that can ask the user to assign a key code and function to > > that key. > > I still disagree that this is the best approach. Userspace already has > the functionality to map keys if they produce a keycode. Producing > KEY_UNKNOWN would require the implementation of a stack of extra code. Well, we already produce KEY_UNKNOWN anyway, and the stuff you quoted above just makes KEY_UNKNOWN useful for something instead of keeping it as an useless notice to the user that some key (which one? who knows!) was pressed. Drivers already have KEY_RESERVED to mark positions in the keycode map for keys that should generate no events, so we are not forcing anyone to always generate useless events, either. Perhaps what you dislike re. KEY_UNKNOWN is the part where KEY_UNKNOWN+scan code is declared to be the prefered way to report keys that do not have a specific function? Your reply seems to indicate this, but I am not sure I really understood what you meant. I am not exactly in love with the idea of using KEY_UNKNOWN in place of stuff like KEY_FN_F1 either (I'd prefer to just bump up KEY_MAX and have more posicional keycodes), but Dmitry is being quite clear that he does not want to increase KEY_MAX to add more positional keycodes. -- "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 - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
