<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 21 August 2014 13:17, Giulio Camuffo <span dir="ltr"><<a href="mailto:giuliocamuffo@gmail.com" target="_blank">giuliocamuffo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2014-08-21 15:06 GMT+03:00 Daniel Stone <<a href="mailto:daniel@fooishbar.org">daniel@fooishbar.org</a>>:<br>
<div class="">> On 21 August 2014 08:58, Giulio Camuffo <<a href="mailto:giuliocamuffo@gmail.com">giuliocamuffo@gmail.com</a>> wrote:<br>>> Switching VT is another matter, because all the keyboard devices are<br>

>> removed so the xkb state is lost, so when returning to weston's vt we<br>
>> don't know anymore which leds are supposed to be on.<br>><br>
> Sure we do: the usual way is to release all keys on VT leave, resetting<br>
> latches but leaving locks as they are. So when they came back, just apply<br>
> the state that occurred as a result of that, rather than trying to maintain<br>
> a totally unknown state from whilst you were switched away, or reset it<br>
> completely.<br>
<br>
</div>I mean the current weston code doesn't know the state anymore. Sure<br>
that can be fixed, but I think it should be another patch.</blockquote><div><br></div><div>Oh yes, to be totally clear, that was really just an aside. It might be the correct way to solve the problem if the LEDs get clobbered on VT switches though, mind. ;)</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">>> This patch just fixes keyboard hotplugging, besides turning the leds<br>
>> off at startup.<br>
><br>
> Sure. I really, really don't like that timer though ... I'd rather just sit<br>
> this one out whilst we work out the correct thing to do though. If you've<br>
> got some time on your hands, looking at the VT-enter path would be good I<br>
> think; even if it doesn't make the first release, I think it'd make a good<br>
> stable-branch candidate.<br>
<br>
</div>I agree the timer is ugly, but I have other priorities at the moment<br>
honestly, so I'm not going to spend big amount of time on this, at<br>
least yet. I'm fine with leaving this one be and getting only the<br>
other two in.<br></blockquote><div><br></div><div>No problem. We can come back to it later. Thanks for dealing with all my bikeshedding!</div><div><br></div><div>-d </div></div></div></div>