From: Samuel Thibault <samuel.thibault@ens-lyon.org> To: Dmitry Torokhov <dmitry.torokhov@gmail.com> Cc: "H. Peter Anvin" <hpa@zytor.com>, Pavel Machek <pavel@ucw.cz>, Alexey Dobriyan <adobriyan@gmail.com>, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, alan@lxorguk.ukuu.org.uk, mgarski@post.pl, linux-input@vger.kernel.org, Vojtech Pavlik <vojtech@suse.cz>, Richard Purdie <rpurdie@rpsys.net> Subject: Re: [PATCH] Route kbd leds through the generic leds layer (3rd version) Date: Thu, 25 Feb 2010 22:44:03 +0100 [thread overview] Message-ID: <20100225214403.GB9378@const.homenet.telecomitalia.it> (raw) In-Reply-To: <20100225102056.GG10823@core.coreip.homeip.net> Dmitry Torokhov, le Thu 25 Feb 2010 02:20:56 -0800, a écrit : > I am aunsure about this patch. It ties all LEDs together For now, yes. There could be an additional per-device layer that the user could select instead, but my patch doesn't prevent that. > whereas currently it is possible to operate keyboards and their leds > independently You mean through the input layer? Technically, yes, but at least my patch is an improvement over what exists right now: at first read of your sentence I thought "of course not! keyboard.c does it for all devices!", so it's not a regression in that concern. On the contrary, it fixes the problem of proper caps lock with led feedback. Being able to assign only some of the devices to the linux console would indeed probably be good, but to me it's just a refinement. Users a priori assume all keyboards get their leds updated, so my patch makes sense. And it won't prevent a further patch that, in addition to input::<led> leds, adds per-device leds, which the user could use instead of input::<led>. Samuel
WARNING: multiple messages have this Message-ID (diff)
From: Samuel Thibault <samuel.thibault@ens-lyon.org> To: Dmitry Torokhov <dmitry.torokhov@gmail.com> Cc: "H. Peter Anvin" <hpa@zytor.com>, Pavel Machek <pavel@ucw.cz>, Alexey Dobriyan <adobriyan@gmail.com>, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, alan@lxorguk.ukuu.org.uk, mgarski@post.pl, linux-input@vger.kernel.org, Vojtech Pavlik <vojtech@suse.cz>, Richard Purdie <rpurdie@rpsys.net> Subject: Re: [PATCH] Route kbd leds through the generic leds layer (3rd version) Date: Thu, 25 Feb 2010 22:44:03 +0100 [thread overview] Message-ID: <20100225214403.GB9378@const.homenet.telecomitalia.it> (raw) In-Reply-To: <20100225102056.GG10823@core.coreip.homeip.net> Dmitry Torokhov, le Thu 25 Feb 2010 02:20:56 -0800, a écrit : > I am aunsure about this patch. It ties all LEDs together For now, yes. There could be an additional per-device layer that the user could select instead, but my patch doesn't prevent that. > whereas currently it is possible to operate keyboards and their leds > independently You mean through the input layer? Technically, yes, but at least my patch is an improvement over what exists right now: at first read of your sentence I thought "of course not! keyboard.c does it for all devices!", so it's not a regression in that concern. On the contrary, it fixes the problem of proper caps lock with led feedback. Being able to assign only some of the devices to the linux console would indeed probably be good, but to me it's just a refinement. Users a priori assume all keyboards get their leds updated, so my patch makes sense. And it won't prevent a further patch that, in addition to input::<led> leds, adds per-device leds, which the user could use instead of input::<led>. Samuel -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2010-02-25 21:44 UTC|newest] Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top 2010-02-24 1:20 [PATCH] Route kbd leds through the generic leds layer Samuel Thibault 2010-02-25 1:38 ` [PATCH] Route kbd leds through the generic leds layer (3rd version) Samuel Thibault 2010-02-25 1:38 ` Samuel Thibault 2010-02-25 10:20 ` Dmitry Torokhov 2010-02-25 10:20 ` Dmitry Torokhov 2010-02-25 12:23 ` Richard Purdie 2010-02-25 21:47 ` Samuel Thibault 2010-02-25 21:47 ` Samuel Thibault 2010-02-25 21:44 ` Samuel Thibault [this message] 2010-02-25 21:44 ` Samuel Thibault 2010-02-26 3:54 ` Dmitry Torokhov 2010-02-26 3:54 ` Dmitry Torokhov 2010-02-26 10:35 ` Samuel Thibault 2010-02-26 10:35 ` Samuel Thibault 2010-03-06 6:54 ` Pavel Machek 2010-03-06 6:54 ` Pavel Machek 2010-03-06 12:30 ` Samuel Thibault 2010-03-06 12:30 ` Samuel Thibault 2010-03-07 8:25 ` Pavel Machek 2010-03-07 8:25 ` Pavel Machek 2010-03-07 12:06 ` Samuel Thibault 2010-03-07 12:06 ` Samuel Thibault 2010-03-06 6:52 ` Pavel Machek 2010-03-06 6:52 ` Pavel Machek
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20100225214403.GB9378@const.homenet.telecomitalia.it \ --to=samuel.thibault@ens-lyon.org \ --cc=adobriyan@gmail.com \ --cc=akpm@linux-foundation.org \ --cc=alan@lxorguk.ukuu.org.uk \ --cc=dmitry.torokhov@gmail.com \ --cc=hpa@zytor.com \ --cc=linux-input@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mgarski@post.pl \ --cc=pavel@ucw.cz \ --cc=rpurdie@rpsys.net \ --cc=vojtech@suse.cz \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.