[lkml]   [2010]   [Feb]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Route kbd leds through the generic leds layer (3rd version)
    Dmitry Torokhov, le Thu 25 Feb 2010 19:54:48 -0800, a écrit :
    > Right now on .33, if you write LED event to one event device while
    > legacy keyboard driver is inactive (in raw mode) then that write will
    > only affect that particular event device, not the rest of them. Your
    > patch changes that.

    Errr, I don't think so. My patch just puts a trigger layer between
    keyboard and the loop over devices that keyboard used to do. When the
    legacy keyboard driver is inactive, it doesn't trigger anything any
    more, and my patch becomes actually inactive, and other sources of LED
    events can send them to various input event devices independently. I've
    just tried it with Xorg (telling it to use only one keyboard) and it

    > Whether LED state should be shared between all input devices or should
    > be kept separate is a matter of policy and we try to keep policy in
    > userspace.


    > > On the contrary,
    > > it fixes the problem of proper caps lock with led feedback.
    > I am unaware of such issue, any pointers?

    In the very changelog of my patch :) #7063
    And also please see the thread “make CapsLock work as expected even
    for non-ASCII” on lkml.

    > > 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>.
    > I think that if we want to go LED route we shoudl start with adding led
    > devices to individual keyboard drivers first

    Mmm, thinking again, I think my patch could be rather easily converted
    into creating led instances for each device, with proper trigger
    defaults. Something like input-devicename::numlock? (devicename::numlock
    might get conflicts with non-input devices I guess?)

    That being said, usermode tools which want to set up the modifiers and
    connect the input LEDs to them need an easy and proper way to do so.
    Having central input::numlock and such still seems a good thing.

    > and then convert legacy
    > keyboard to use trigger instead of talking to input devices directly.

    This part is already done by my patch.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2010-02-26 11:37    [W:0.023 / U:303.116 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site