[PATCH kdrive/ephyr v7 4/9] kdrive: update evdev keyboard LEDs (#22302)

[Laércio de Sousa]

Hi Peter!

I've just reviewed this patch. Assuming that the kdrive evdev keyboard was
correctly attached to master keyboard, I've just removed the kdrive.c part
of this patch, keeping the evdev.c part and... voilà! My Xephyr keyboard
LEDs are being correclty.

Since I've just resent my patch series without this one included, I'll
submit it separately.

Thank you very much for the advice!

Att.

2016-02-11 16:13 GMT-02:00 Laércio de Sousa <
laerciosousa at sme-mogidascruzes.sp.gov.br>:

> 2016-02-11 1:11 GMT-02:00 Peter Hutterer <peter.hutterer at who-t.net>:
>>
>> the idea behind only propagating LED changes to core keyboard is that you
>> may have multiple keyboards, not all attached to the same master keyboard.
>> If you toggle caps lock on one of them, you don't want it to toggle on an
>> otherwise unrelated keyboard. Likewise, there's the niche case of floating
>> slave devices, where you don't want logical keyboard LEDs to change other
>> keyboards.
>>
>> (...)
>>
>> so something else isn't right here. By default, xephyr keyboards should be
>> attached to a master keyboard and see the LEDs update. Where exactly does
>> the LED update stop?
>
> I'll drop this patch for the moment, since it's not critical to make
> multi-seat work properly with Xephyr. I'll investigate a bit more the
> problem of attaching Xephyr keyboard to a master keyboard and propose
> another patch in a second moment.
>
> Thanks, Peter!
> --
> *Laércio de Sousa*
> *Orientador de Informática*
> *Escola Municipal "Professor Eulálio Gruppi"*
> *Rua Ismael da Silva Mello, 559, Mogi Moderno*
> *Mogi das Cruzes - SPCEP 08717-390*
> Telefone: (11) 4726-8313
>



-- 
*Laércio de Sousa*
*Orientador de Informática*
*Escola Municipal "Professor Eulálio Gruppi"*
*Rua Ismael da Silva Mello, 559, Mogi Moderno*
*Mogi das Cruzes - SPCEP 08717-390*
Telefone: (11) 4726-8313
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.x.org/archives/xorg-devel/attachments/20160215/e45482a3/attachment.html>