Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-12 Thread Srinivas Pandruvada
On Sat, 2017-08-12 at 14:58 +0200, Bastien Nocera wrote: > On Sat, 2017-08-12 at 13:16 +0100, Jonathan Cameron wrote: > > > > On Fri, 11 Aug 2017 16:04:30 +0200 > > Bastien Nocera wrote: > > > > > > > > Woot! > > > [...] > > > > > > > > Signed-off-by: Srinivas Pandruvada

Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-12 Thread Srinivas Pandruvada
On Sat, 2017-08-12 at 14:58 +0200, Bastien Nocera wrote: > On Sat, 2017-08-12 at 13:16 +0100, Jonathan Cameron wrote: > > > > On Fri, 11 Aug 2017 16:04:30 +0200 > > Bastien Nocera wrote: > > > > > > > > Woot! > > > [...] > > > > > > > > Signed-off-by: Srinivas Pandruvada > > > nt > > > >

Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-12 Thread Bastien Nocera
On Sat, 2017-08-12 at 13:16 +0100, Jonathan Cameron wrote: > On Fri, 11 Aug 2017 16:04:30 +0200 > Bastien Nocera wrote: > > > Woot! > > > > On Thu, 2017-08-10 at 16:24 -0700, Srinivas Pandruvada wrote: > > > It has been reported for a while that with iio-sensor-proxy > > >

Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-12 Thread Bastien Nocera
On Sat, 2017-08-12 at 13:16 +0100, Jonathan Cameron wrote: > On Fri, 11 Aug 2017 16:04:30 +0200 > Bastien Nocera wrote: > > > Woot! > > > > On Thu, 2017-08-10 at 16:24 -0700, Srinivas Pandruvada wrote: > > > It has been reported for a while that with iio-sensor-proxy > > > service the > > >

Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-12 Thread Jonathan Cameron
On Fri, 11 Aug 2017 16:04:30 +0200 Bastien Nocera wrote: > Woot! > > On Thu, 2017-08-10 at 16:24 -0700, Srinivas Pandruvada wrote: > > It has been reported for a while that with iio-sensor-proxy service the > > rotation only works after one suspend/resume cycle. This required

Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-12 Thread Jonathan Cameron
On Fri, 11 Aug 2017 16:04:30 +0200 Bastien Nocera wrote: > Woot! > > On Thu, 2017-08-10 at 16:24 -0700, Srinivas Pandruvada wrote: > > It has been reported for a while that with iio-sensor-proxy service the > > rotation only works after one suspend/resume cycle. This required a wait > > in the

Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-11 Thread Bastien Nocera
Woot! On Thu, 2017-08-10 at 16:24 -0700, Srinivas Pandruvada wrote: > It has been reported for a while that with iio-sensor-proxy service the > rotation only works after one suspend/resume cycle. This required a wait > in the systemd unit file to avoid race. I found a Yoga 900 where I could >

Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-11 Thread Bastien Nocera
Woot! On Thu, 2017-08-10 at 16:24 -0700, Srinivas Pandruvada wrote: > It has been reported for a while that with iio-sensor-proxy service the > rotation only works after one suspend/resume cycle. This required a wait > in the systemd unit file to avoid race. I found a Yoga 900 where I could >

[RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-10 Thread Srinivas Pandruvada
It has been reported for a while that with iio-sensor-proxy service the rotation only works after one suspend/resume cycle. This required a wait in the systemd unit file to avoid race. I found a Yoga 900 where I could reproduce this. The problem scenerio is: - During sensor driver init, enable

[RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

2017-08-10 Thread Srinivas Pandruvada
It has been reported for a while that with iio-sensor-proxy service the rotation only works after one suspend/resume cycle. This required a wait in the systemd unit file to avoid race. I found a Yoga 900 where I could reproduce this. The problem scenerio is: - During sensor driver init, enable