On Wednesday 03 December 2014 14:24:11 Pali Rohár wrote: > On Wednesday 03 December 2014 14:12:54 Henrique de Moraes > > Holschuh wrote: > > On Wed, 03 Dec 2014, Pali Rohár wrote: > > > Ok, and what about KEY_KBDILLUMTOGGLE when bios also handle > > > keyboard backlight level? Should be this key filtered too? > > > > IME, heck yes. > > > > If you ever make the mistake of sending to userspace something > > the BIOS/kernel already reacted to, they will find a way to > > loop it back to you and cause all sort of issues. > > Ok, I think same. In KDE4 I see some loop/noop operation when new > dell keyboard backlight driver is used. Key KEY_KBDILLUMTOGGLE > cause that BIOS change brightness and also KDE4 see it and change > it too... > > Similar problem there is with KEY_WLAN and NetworkManager. > > Gabriele, can you create patch which disable both KEY_WLAN and > KEY_KBDILLUMTOGGLE in dell-wmi.c driver? KEY_KBDILLUMTOGGLE is actually disabled already, but KEY_WLAN (which I guess it should changed to KEY_RFKILL now that it exists) isn't, so I will submit a patch. The comment above it in dell-wmi.c makes me think that for all the systems the BIOS does everything and not only mine. Gabriele -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
