All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-02-17 19:15 ` Samuel Thibault
  0 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-02-17 19:15 UTC (permalink / raw)
  To: Dmitry Torokhov, Pavel Machek, Andrew Morton, David Herrmann,
	jslaby, Bryan Wu, rpurdie, linux-kernel, Evan Broder,
	Arnaud Patard, Peter Korsgaard, Sascha Hauer, Rob Clark,
	Niels de Vos, linux-arm-kernel, blogic, Pali Rohár

Here is an updated version to fix the initialization of the vt_led_work
queues before registering LEDs, and refresh against 3.19.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-02-17 19:15 ` Samuel Thibault
  0 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-02-17 19:15 UTC (permalink / raw)
  To: linux-arm-kernel

Here is an updated version to fix the initialization of the vt_led_work
queues before registering LEDs, and refresh against 3.19.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-02-17 19:15 ` Samuel Thibault
@ 2015-04-01 20:00   ` Pali Rohár
  -1 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-04-01 20:00 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Samuel Thibault, Pavel Machek, Andrew Morton, David Herrmann,
	jslaby, Bryan Wu, rpurdie, linux-kernel, Evan Broder,
	Arnaud Patard, Peter Korsgaard, Sascha Hauer, Rob Clark,
	Niels de Vos, linux-arm-kernel, blogic

[-- Attachment #1: Type: Text/Plain, Size: 387 bytes --]

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.
> 
> Samuel

Hello! I would like to ask when will be this patch series merged 
into mainline kernel? Are there still some problems with it?

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-04-01 20:00   ` Pali Rohár
  0 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-04-01 20:00 UTC (permalink / raw)
  To: linux-arm-kernel

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.
> 
> Samuel

Hello! I would like to ask when will be this patch series merged 
into mainline kernel? Are there still some problems with it?

-- 
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: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150401/fb417235/attachment.sig>

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-01 20:00   ` Pali Rohár
@ 2015-04-01 21:11     ` Samuel Thibault
  -1 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-04-01 21:11 UTC (permalink / raw)
  To: Pali Rohár
  Cc: Dmitry Torokhov, Pavel Machek, Andrew Morton, David Herrmann,
	jslaby, Bryan Wu, rpurdie, linux-kernel, Evan Broder,
	Arnaud Patard, Peter Korsgaard, Sascha Hauer, Rob Clark,
	Niels de Vos, linux-arm-kernel, blogic

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.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-04-01 21:11     ` Samuel Thibault
  0 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-04-01 21:11 UTC (permalink / raw)
  To: linux-arm-kernel

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.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-01 21:11     ` Samuel Thibault
@ 2015-04-02 14:44       ` Pavel Machek
  -1 siblings, 0 replies; 30+ messages in thread
From: Pavel Machek @ 2015-04-02 14:44 UTC (permalink / raw)
  To: Samuel Thibault, Pali Rohár, Dmitry Torokhov, Andrew Morton,
	David Herrmann, jslaby, Bryan Wu, rpurdie, linux-kernel,
	Evan Broder, Arnaud Patard, Peter Korsgaard, Sascha Hauer,
	Rob Clark, Niels de Vos, linux-arm-kernel, blogic

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
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-04-02 14:44       ` Pavel Machek
  0 siblings, 0 replies; 30+ messages in thread
From: Pavel Machek @ 2015-04-02 14:44 UTC (permalink / raw)
  To: linux-arm-kernel

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
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-02 14:44       ` Pavel Machek
@ 2015-04-12 22:16         ` Pali Rohár
  -1 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-04-12 22:16 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Pavel Machek, Samuel Thibault, Andrew Morton, David Herrmann,
	jslaby, Bryan Wu, rpurdie, linux-kernel, Evan Broder,
	Arnaud Patard, Peter Korsgaard, Sascha Hauer, Rob Clark,
	Niels de Vos, linux-arm-kernel, blogic

[-- Attachment #1: Type: Text/Plain, Size: 895 bytes --]

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, ping. If everything is OK, can you merge this patch? If 
not can you comment it?

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-04-12 22:16         ` Pali Rohár
  0 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-04-12 22:16 UTC (permalink / raw)
  To: linux-arm-kernel

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, ping. If everything is OK, can you merge this patch? If 
not can you comment it?

-- 
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: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150413/c6493527/attachment.sig>

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-12 22:16         ` Pali Rohár
@ 2015-04-13  8:06           ` Samuel Thibault
  -1 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-04-13  8:06 UTC (permalink / raw)
  To: Pali Rohár
  Cc: Dmitry Torokhov, Pavel Machek, Andrew Morton, David Herrmann,
	jslaby, Bryan Wu, rpurdie, linux-kernel, Evan Broder,
	Arnaud Patard, Peter Korsgaard, Sascha Hauer, Rob Clark,
	Niels de Vos, linux-arm-kernel, blogic

FTR, the two patches apply and work fine with linux 4.0.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-04-13  8:06           ` Samuel Thibault
  0 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-04-13  8:06 UTC (permalink / raw)
  To: linux-arm-kernel

FTR, the two patches apply and work fine with linux 4.0.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-02 14:44       ` Pavel Machek
@ 2015-04-23 16:55         ` Pali Rohár
  -1 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-04-23 16:55 UTC (permalink / raw)
  To: Dmitry Torokhov, Pavel Machek
  Cc: Samuel Thibault, Andrew Morton, David Herrmann, jslaby, Bryan Wu,
	rpurdie, linux-kernel, Evan Broder, Arnaud Patard,
	Peter Korsgaard, Sascha Hauer, Rob Clark, Niels de Vos,
	linux-arm-kernel, blogic

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?

-- 
Pali Rohár
pali.rohar@gmail.com

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-04-23 16:55         ` Pali Rohár
  0 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-04-23 16:55 UTC (permalink / raw)
  To: linux-arm-kernel

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?

-- 
Pali Roh?r
pali.rohar at gmail.com

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-23 16:55         ` Pali Rohár
@ 2015-04-23 17:04           ` Dmitry Torokhov
  -1 siblings, 0 replies; 30+ messages in thread
From: Dmitry Torokhov @ 2015-04-23 17:04 UTC (permalink / raw)
  To: Pali Rohár
  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, blogic

On Thu, Apr 23, 2015 at 9:55 AM, Pali Rohár <pali.rohar@gmail.com> 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

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-04-23 17:04           ` Dmitry Torokhov
  0 siblings, 0 replies; 30+ messages in thread
From: Dmitry Torokhov @ 2015-04-23 17:04 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Apr 23, 2015 at 9:55 AM, Pali Roh?r <pali.rohar@gmail.com> 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

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-23 17:04           ` Dmitry Torokhov
@ 2015-05-02 22:44             ` Pali Rohár
  -1 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-05-02 22:44 UTC (permalink / raw)
  To: Samuel Thibault
  Cc: Dmitry Torokhov, Pavel Machek, 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, blogic

[-- Attachment #1: Type: Text/Plain, Size: 2261 bytes --]

On Thursday 23 April 2015 19:04:49 Dmitry Torokhov wrote:
> On Thu, Apr 23, 2015 at 9:55 AM, Pali Rohár
> <pali.rohar@gmail.com> 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@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-05-02 22:44             ` Pali Rohár
  0 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-05-02 22:44 UTC (permalink / raw)
  To: linux-arm-kernel

On Thursday 23 April 2015 19:04:49 Dmitry Torokhov wrote:
> On Thu, Apr 23, 2015 at 9:55 AM, Pali Roh?r
> <pali.rohar@gmail.com> 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: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150503/50985102/attachment.sig>

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-04-23 17:04           ` Dmitry Torokhov
@ 2015-06-05 15:28             ` Samuel Thibault
  -1 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-06-05 15:28 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Pali Rohár, Pavel Machek, 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, blogic

Hello,

Dmitry Torokhov, le Thu 23 Apr 2015 10:04:49 -0700, a écrit :
> 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.

What do you mean precisely by "engage"?  In the following, I guess
actually calling dev->event(EV_LED)

> 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.

This seems like only a workaround, perhaps it should rather be leds.c
which checks for dev->users before calling dev->event(EV_LED)?

> Another issue is that I do not think we should be introducing virtual
> VT leds. I believe LEDs should belong to real devices;

But then how to fix console-setup's bug? (it was actually the starter
for all this work)

See http://bugs.debian.org/514464

console-setup needs a way to tell which kbd modifier should toggle the
capslock LED on all the keyboards used by the VT. Thus the point of VT
leds, which people can use to decide the LED behavior of all keyboards,
including hotplugged ones etc.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-06-05 15:28             ` Samuel Thibault
  0 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-06-05 15:28 UTC (permalink / raw)
  To: linux-arm-kernel

Hello,

Dmitry Torokhov, le Thu 23 Apr 2015 10:04:49 -0700, a ?crit :
> 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.

What do you mean precisely by "engage"?  In the following, I guess
actually calling dev->event(EV_LED)

> 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.

This seems like only a workaround, perhaps it should rather be leds.c
which checks for dev->users before calling dev->event(EV_LED)?

> Another issue is that I do not think we should be introducing virtual
> VT leds. I believe LEDs should belong to real devices;

But then how to fix console-setup's bug? (it was actually the starter
for all this work)

See http://bugs.debian.org/514464

console-setup needs a way to tell which kbd modifier should toggle the
capslock LED on all the keyboards used by the VT. Thus the point of VT
leds, which people can use to decide the LED behavior of all keyboards,
including hotplugged ones etc.

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-06-05 15:28             ` Samuel Thibault
@ 2015-06-25 15:30               ` Samuel Thibault
  -1 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-06-25 15:30 UTC (permalink / raw)
  To: Dmitry Torokhov, Pali Rohár, Pavel Machek, 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, blogic

Hello,

Just to give an update to people who where Cc-ed at some point in the
discussion: a version reworked by Dmitry got pulled into Linus' tree, so
it should get into 4.2!

Thanks to everybody involved,
Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-06-25 15:30               ` Samuel Thibault
  0 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-06-25 15:30 UTC (permalink / raw)
  To: linux-arm-kernel

Hello,

Just to give an update to people who where Cc-ed at some point in the
discussion: a version reworked by Dmitry got pulled into Linus' tree, so
it should get into 4.2!

Thanks to everybody involved,
Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-06-25 15:30               ` Samuel Thibault
@ 2015-06-25 15:37                 ` Peter Korsgaard
  -1 siblings, 0 replies; 30+ messages in thread
From: Peter Korsgaard @ 2015-06-25 15:37 UTC (permalink / raw)
  To: Samuel Thibault
  Cc: Dmitry Torokhov, Pali Rohár, Pavel Machek, 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, blogic

>>>>> "Samuel" == Samuel Thibault <samuel.thibault@ens-lyon.org> writes:

 > Hello,
 > Just to give an update to people who where Cc-ed at some point in the
 > discussion: a version reworked by Dmitry got pulled into Linus' tree, so
 > it should get into 4.2!

 > Thanks to everybody involved,

Wee, thanks!

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-06-25 15:37                 ` Peter Korsgaard
  0 siblings, 0 replies; 30+ messages in thread
From: Peter Korsgaard @ 2015-06-25 15:37 UTC (permalink / raw)
  To: linux-arm-kernel

>>>>> "Samuel" == Samuel Thibault <samuel.thibault@ens-lyon.org> writes:

 > Hello,
 > Just to give an update to people who where Cc-ed at some point in the
 > discussion: a version reworked by Dmitry got pulled into Linus' tree, so
 > it should get into 4.2!

 > Thanks to everybody involved,

Wee, thanks!

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-06-25 15:30               ` Samuel Thibault
@ 2015-06-25 16:25                 ` Samuel Thibault
  -1 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-06-25 16:25 UTC (permalink / raw)
  To: Dmitry Torokhov, Pali Rohár, Pavel Machek, 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, blogic

Samuel Thibault, le Thu 25 Jun 2015 17:30:32 +0200, a écrit :
> Just to give an update to people who where Cc-ed at some point in the
> discussion: a version reworked by Dmitry got pulled into Linus' tree, so
> it should get into 4.2!

(5 years after first submission, but better late than never :) )

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-06-25 16:25                 ` Samuel Thibault
  0 siblings, 0 replies; 30+ messages in thread
From: Samuel Thibault @ 2015-06-25 16:25 UTC (permalink / raw)
  To: linux-arm-kernel

Samuel Thibault, le Thu 25 Jun 2015 17:30:32 +0200, a ?crit :
> Just to give an update to people who where Cc-ed at some point in the
> discussion: a version reworked by Dmitry got pulled into Linus' tree, so
> it should get into 4.2!

(5 years after first submission, but better late than never :) )

Samuel

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-06-25 16:25                 ` Samuel Thibault
@ 2015-06-25 16:53                   ` Dmitry Torokhov
  -1 siblings, 0 replies; 30+ messages in thread
From: Dmitry Torokhov @ 2015-06-25 16:53 UTC (permalink / raw)
  To: Samuel Thibault, Pali Rohár, Pavel Machek, 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, blogic

On Thu, Jun 25, 2015 at 06:25:56PM +0200, Samuel Thibault wrote:
> Samuel Thibault, le Thu 25 Jun 2015 17:30:32 +0200, a écrit :
> > Just to give an update to people who where Cc-ed at some point in the
> > discussion: a version reworked by Dmitry got pulled into Linus' tree, so
> > it should get into 4.2!
> 
> (5 years after first submission, but better late than never :) )

Just like a good wine it had to mature a bit :)

Seriously though, I am sorry it took so long, I just had hard time
wrapping my head around of how to do it cleanly.

Thanks.

-- 
Dmitry

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-06-25 16:53                   ` Dmitry Torokhov
  0 siblings, 0 replies; 30+ messages in thread
From: Dmitry Torokhov @ 2015-06-25 16:53 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Jun 25, 2015 at 06:25:56PM +0200, Samuel Thibault wrote:
> Samuel Thibault, le Thu 25 Jun 2015 17:30:32 +0200, a ?crit :
> > Just to give an update to people who where Cc-ed at some point in the
> > discussion: a version reworked by Dmitry got pulled into Linus' tree, so
> > it should get into 4.2!
> 
> (5 years after first submission, but better late than never :) )

Just like a good wine it had to mature a bit :)

Seriously though, I am sorry it took so long, I just had hard time
wrapping my head around of how to do it cleanly.

Thanks.

-- 
Dmitry

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
  2015-06-25 15:30               ` Samuel Thibault
@ 2015-06-26  8:09                 ` Pali Rohár
  -1 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-06-26  8:09 UTC (permalink / raw)
  To: Samuel Thibault, Dmitry Torokhov, Pavel Machek, 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, blogic

On Thursday 25 June 2015 17:30:32 Samuel Thibault wrote:
> Hello,
> 
> Just to give an update to people who where Cc-ed at some point in the
> discussion: a version reworked by Dmitry got pulled into Linus' tree, so
> it should get into 4.2!
> 
> Thanks to everybody involved,
> Samuel

Thanks! I'm very very happy to hear this!

-- 
Pali Rohár
pali.rohar@gmail.com

^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer
@ 2015-06-26  8:09                 ` Pali Rohár
  0 siblings, 0 replies; 30+ messages in thread
From: Pali Rohár @ 2015-06-26  8:09 UTC (permalink / raw)
  To: linux-arm-kernel

On Thursday 25 June 2015 17:30:32 Samuel Thibault wrote:
> Hello,
> 
> Just to give an update to people who where Cc-ed at some point in the
> discussion: a version reworked by Dmitry got pulled into Linus' tree, so
> it should get into 4.2!
> 
> Thanks to everybody involved,
> Samuel

Thanks! I'm very very happy to hear this!

-- 
Pali Roh?r
pali.rohar at gmail.com

^ permalink raw reply	[flat|nested] 30+ messages in thread

end of thread, other threads:[~2015-06-26  8:10 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-17 19:15 [PATCHv7 0/2] INPUT: Route keyboard LEDs through the generic LEDs layer Samuel Thibault
2015-02-17 19:15 ` Samuel Thibault
2015-04-01 20:00 ` Pali Rohár
2015-04-01 20:00   ` Pali Rohár
2015-04-01 21:11   ` Samuel Thibault
2015-04-01 21:11     ` Samuel Thibault
2015-04-02 14:44     ` Pavel Machek
2015-04-02 14:44       ` Pavel Machek
2015-04-12 22:16       ` Pali Rohár
2015-04-12 22:16         ` Pali Rohár
2015-04-13  8:06         ` Samuel Thibault
2015-04-13  8:06           ` Samuel Thibault
2015-04-23 16:55       ` Pali Rohár
2015-04-23 16:55         ` Pali Rohár
2015-04-23 17:04         ` Dmitry Torokhov
2015-04-23 17:04           ` Dmitry Torokhov
2015-05-02 22:44           ` Pali Rohár
2015-05-02 22:44             ` Pali Rohár
2015-06-05 15:28           ` Samuel Thibault
2015-06-05 15:28             ` Samuel Thibault
2015-06-25 15:30             ` Samuel Thibault
2015-06-25 15:30               ` Samuel Thibault
2015-06-25 15:37               ` Peter Korsgaard
2015-06-25 15:37                 ` Peter Korsgaard
2015-06-25 16:25               ` Samuel Thibault
2015-06-25 16:25                 ` Samuel Thibault
2015-06-25 16:53                 ` Dmitry Torokhov
2015-06-25 16:53                   ` Dmitry Torokhov
2015-06-26  8:09               ` Pali Rohár
2015-06-26  8:09                 ` Pali Rohár

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.