From mboxrd@z Thu Jan 1 00:00:00 1970 From: pali.rohar@gmail.com (Pali =?utf-8?q?Roh=C3=A1r?=) Date: Sun, 3 May 2015 00:44:49 +0200 Subject: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer In-Reply-To: References: <20150217191527.GA9594@type.youpi.perso.aquilenet.fr> <20150423165544.GG7646@pali> Message-ID: <201505030044.49464@pali> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thursday 23 April 2015 19:04:49 Dmitry Torokhov wrote: > On Thu, Apr 23, 2015 at 9:55 AM, Pali Roh?r > wrote: > > On Thursday 02 April 2015 16:44:10 Pavel Machek wrote: > >> On Wed 2015-04-01 23:11:40, Samuel Thibault wrote: > >> > Pali Roh?r, le Wed 01 Apr 2015 22:00:07 +0200, a ?crit : > >> > > On Tuesday 17 February 2015 20:15:27 Samuel Thibault > >> > > wrote: > >> > > > Here is an updated version to fix the initialization > >> > > > of the vt_led_work queues before registering LEDs, > >> > > > and refresh against 3.19. > >> > > > >> > > Hello! I would like to ask when will be this patch > >> > > series merged into mainline kernel? Are there still > >> > > some problems with it? > >> > > >> > There are no known problems ATM. > >> > >> I thought it made it to -next, but apparently not. > >> > >> Dmitry, can you comment what needs to be done, or just > >> merge it, please? > >> > >> Pavel > > > > Dmitry, can you merge this patch? > > Sorry, I keep intending to go back to it and keep getting > distracted with other items. Last time I tried it it did not > appear to work for some scenarios that I tried, but I did not > document it to provide reasonable feedback to Samuel. > > One thing that I know we'd have to fix is that input device > must be "opened" before we can engage it, right now LED > interface violates this requirement. It works right now > because keyboard handler attaches to most input devices with > LEDs early enough for it to be unnoticeable, but it does not > mean that it is correct. It might be as easy as calling > input_open() unconditionally if devices has LEDs. > > Another issue is that I do not think we should be introducing > virtual VT leds. I believe LEDs should belong to real > devices; multiplexing several into one usually ends up with > problems (like the whole mousedev and various users having to > "grab" touchpads to exclude their data form mousedev to avoid > duplicate movement/button presses). > > Hopefully I will have more coherent response RSN. > > Thanks and sorry. Samuel, can you look at those issues? -- Pali Roh?r pali.rohar at gmail.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part. URL: