Re: [PATCH 1/2] evdev: turn off all the leds at startup

2014-08-20 Thread Daniel Stone
Hi,

On 19 August 2014 11:25, Giulio Camuffo giuliocamu...@gmail.com wrote:

 the internal state in xkbcommon is off for all the three leds, so
 make them be in sync.


This should only trigger if seat-keyboard is NULL beforehand (is the first
keyboard); if not, it should inherit LED state from the other attached
keyboards. Otherwise:
Reviewed-by: Daniel Stone dani...@collabora.com

Cheers,
Daniel
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


[PATCH 1/2] evdev: turn off all the leds at startup

2014-08-19 Thread Giulio Camuffo
the internal state in xkbcommon is off for all the three leds, so
make them be in sync.
---
 src/evdev.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/evdev.c b/src/evdev.c
index 888dfbd..a46ed87 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -581,6 +581,7 @@ evdev_configure_device(struct evdev_device *device)
device-seat_caps |= EVDEV_SEAT_KEYBOARD;
weston_log(input device %s, %s is a keyboard\n,
   device-devname, device-devnode);
+   evdev_led_update(device, 0);
}
if (has_touch  !has_button) {
weston_seat_init_touch(device-seat);
-- 
2.0.4

___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel