On Tue, 04 Aug 2009, Dmitry Torokhov wrote: > On Tue, Aug 04, 2009 at 11:01:58AM +0100, Matthew Garrett wrote: > > On Mon, Aug 03, 2009 at 10:04:51AM -0700, Dmitry Torokhov wrote: > > > I don't think this is the proper layer to do the filtering, the filter > > > should be done in i8042, not input core. The way you done it would kill > > > any possibility of user assigning KEY_WLAN to a spare key on his or her > > > keyboard and using it to control wireless cards that are not hardwired. > > > > Ok, that seems fine. > > > > > I would accept the patch that would add a filter in the form > > > > > > bool (*i8042_fillter)(unsigned char data, unsigned int flags, > > > struct serio *port); > > > > > > I don't think we'd need to chain them into a list or something, one hook > > > should be enough since I don't expect we'll have several platform > > > drivers loaded on one box. > > > > So filter on the raw i8042 data rather than after any processing? > > Wouldn't this require some level of state machine in the driver using > > this functionality? > > Yes, but it should not be too complex - with laptops you can be pretty > much sure keyboard is in translated set 2 mode. But don't we have a crapload of implementation weirdness work-arounds in the i8042 drivers, including some for problems with the raw protocol itself? We wouldn't want to duplicate THAT... will the filter be placed somewhere that doesn't have to deal with that kind of bogosity? With laptops, you can be sure that 99% of the time, the i8042 is just firmware running inside the EC, and subject to a LOT MORE bugs and incompetence than an emulated i8042 inside the chipset would be... -- "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
