Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030340AbbDWREx (ORCPT ); Thu, 23 Apr 2015 13:04:53 -0400 Received: from mail-lb0-f171.google.com ([209.85.217.171]:34577 "EHLO mail-lb0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750741AbbDWREu convert rfc822-to-8bit (ORCPT ); Thu, 23 Apr 2015 13:04:50 -0400 MIME-Version: 1.0 In-Reply-To: <20150423165544.GG7646@pali> References: <20150217191527.GA9594@type.youpi.perso.aquilenet.fr> <201504012200.07408@pali> <20150401211140.GI2901@type.youpi.perso.aquilenet.fr> <20150402144410.GB18125@amd> <20150423165544.GG7646@pali> Date: Thu, 23 Apr 2015 10:04:49 -0700 Message-ID: Subject: Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer From: Dmitry Torokhov To: =?UTF-8?Q?Pali_Roh=C3=A1r?= Cc: Pavel Machek , Samuel Thibault , Andrew Morton , David Herrmann , Jiri Slaby , Bryan Wu , Richard Purdie , lkml , Evan Broder , Arnaud Patard , Peter Korsgaard , Sascha Hauer , Rob Clark , Niels de Vos , "linux-arm-kernel@lists.infradead.org" , blogic@openwrt.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2224 Lines: 52 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. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/