Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2017-06-11 Thread Jonathan Cameron
On Wed, 7 Jun 2017 18:10:02 -0700 Gwendal Grignou wrote: Hi Gwendal, > [+Nick] > Looking into it again, given that overloading IIO_ACTIVITY is not an > option, we need another channel type for these type of events. (single > tap, double tap). Could represent them as an any

Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2017-06-11 Thread Jonathan Cameron
On Wed, 7 Jun 2017 18:10:02 -0700 Gwendal Grignou wrote: Hi Gwendal, > [+Nick] > Looking into it again, given that overloading IIO_ACTIVITY is not an > option, we need another channel type for these type of events. (single > tap, double tap). Could represent them as an any access event... >

Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2017-06-07 Thread Gwendal Grignou
[+Nick] Looking into it again, given that overloading IIO_ACTIVITY is not an option, we need another channel type for these type of events. (single tap, double tap). Also to reduce the CPU load at idle, instead of polling an accelerometer to detect orientation change, we are looking at using

Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2017-06-07 Thread Gwendal Grignou
[+Nick] Looking into it again, given that overloading IIO_ACTIVITY is not an option, we need another channel type for these type of events. (single tap, double tap). Also to reduce the CPU load at idle, instead of polling an accelerometer to detect orientation change, we are looking at using

Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2016-12-19 Thread Jonathan Cameron
On 06/12/16 11:25, Peter Meerwald-Stadler wrote: > >> This adds the IIO_MOD_DOUBLE_TAP entry to the iio_modifier enum and the >> corresponding "double_tap" string to the iio_modifier_names array. > > I don't think we should have gestures as channel modifiers Agreed, though treating it like other

Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2016-12-19 Thread Jonathan Cameron
On 06/12/16 11:25, Peter Meerwald-Stadler wrote: > >> This adds the IIO_MOD_DOUBLE_TAP entry to the iio_modifier enum and the >> corresponding "double_tap" string to the iio_modifier_names array. > > I don't think we should have gestures as channel modifiers Agreed, though treating it like other

Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2016-12-06 Thread Peter Meerwald-Stadler
> This adds the IIO_MOD_DOUBLE_TAP entry to the iio_modifier enum and the > corresponding "double_tap" string to the iio_modifier_names array. I don't think we should have gestures as channel modifiers probably a middle layer between IIO and input subsystem is needed (could be in userspace?)

Re: [PATCH 2/3] iio: core: Add double tap as possible gesture

2016-12-06 Thread Peter Meerwald-Stadler
> This adds the IIO_MOD_DOUBLE_TAP entry to the iio_modifier enum and the > corresponding "double_tap" string to the iio_modifier_names array. I don't think we should have gestures as channel modifiers probably a middle layer between IIO and input subsystem is needed (could be in userspace?)

[PATCH 2/3] iio: core: Add double tap as possible gesture

2016-12-05 Thread Thierry Escande
From: Gwendal Grignou This adds the IIO_MOD_DOUBLE_TAP entry to the iio_modifier enum and the corresponding "double_tap" string to the iio_modifier_names array. Signed-off-by: Gwendal Grignou Signed-off-by: Thierry Escande

[PATCH 2/3] iio: core: Add double tap as possible gesture

2016-12-05 Thread Thierry Escande
From: Gwendal Grignou This adds the IIO_MOD_DOUBLE_TAP entry to the iio_modifier enum and the corresponding "double_tap" string to the iio_modifier_names array. Signed-off-by: Gwendal Grignou Signed-off-by: Thierry Escande --- drivers/iio/industrialio-core.c | 1 +