Re: [PATCH 2/2] compositor: add a way to change the keyboard leds

2014-08-22 Thread Pekka Paalanen
On Fri, 22 Aug 2014 10:13:52 +0300 Giulio Camuffo wrote: > 2014-08-21 17:12 GMT+03:00 Pekka Paalanen : > > On Thu, 21 Aug 2014 08:37:17 +0100 > > Daniel Stone wrote: > > > >> Hi, > >> Last nitpick, sorry ... > >> > >> On Wednesday, August 20, 2014, Giulio Camuffo > >> wrote: > >> > > >> > +

Re: [PATCH 2/2] compositor: add a way to change the keyboard leds

2014-08-22 Thread Giulio Camuffo
2014-08-21 17:12 GMT+03:00 Pekka Paalanen : > On Thu, 21 Aug 2014 08:37:17 +0100 > Daniel Stone wrote: > >> Hi, >> Last nitpick, sorry ... >> >> On Wednesday, August 20, 2014, Giulio Camuffo >> wrote: >> > >> > + mods_depressed = >> > xkb_state_serialize_mods(keyboard->xkb_state.state, >> >

Re: [PATCH 2/2] compositor: add a way to change the keyboard leds

2014-08-21 Thread Pekka Paalanen
On Thu, 21 Aug 2014 08:37:17 +0100 Daniel Stone wrote: > Hi, > Last nitpick, sorry ... > > On Wednesday, August 20, 2014, Giulio Camuffo > wrote: > > > > + mods_depressed = > > xkb_state_serialize_mods(keyboard->xkb_state.state, > > + XKB_STAT

Re: [PATCH 2/2] compositor: add a way to change the keyboard leds

2014-08-21 Thread Daniel Stone
Hi, Last nitpick, sorry ... On Wednesday, August 20, 2014, Giulio Camuffo wrote: > > + mods_depressed = > xkb_state_serialize_mods(keyboard->xkb_state.state, > + XKB_STATE_DEPRESSED); > + mods_latched = xkb_state_serialize_mods(keyboard->x

Re: [PATCH 2/2] compositor: add a way to change the keyboard leds

2014-08-20 Thread Giulio Camuffo
2014-08-20 10:40 GMT+03:00 Daniel Stone : > Hi, > > On 19 August 2014 11:25, Giulio Camuffo wrote: >> >> This adds a function weston_keyboard_set_leds() which can be used >> to change the state of the num lock and the caps lock leds. >> Only the evdev backend supports this, since it doesn't make s

Re: [PATCH 2/2] compositor: add a way to change the keyboard leds

2014-08-20 Thread Daniel Stone
Hi, On 19 August 2014 11:25, Giulio Camuffo wrote: > This adds a function weston_keyboard_set_leds() which can be used > to change the state of the num lock and the caps lock leds. > Only the evdev backend supports this, since it doesn't make sense > for embedded sessions. > This is missing the

[PATCH 2/2] compositor: add a way to change the keyboard leds

2014-08-19 Thread Giulio Camuffo
This adds a function weston_keyboard_set_leds() which can be used to change the state of the num lock and the caps lock leds. Only the evdev backend supports this, since it doesn't make sense for embedded sessions. --- src/compositor.h | 3 +++ src/input.c | 43 ++