]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
iio: hid-sensor-trigger: Fix sometimes not powering up the sensor after resume
authorHans de Goede <hdegoede@redhat.com>
Sat, 14 Apr 2018 15:09:09 +0000 (17:09 +0200)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 21 Apr 2018 14:52:35 +0000 (15:52 +0100)
commit6f92253024d9d947a4f454654840ce479e251376
tree05d1eba07ddb9e26f346850aac1420eac7234f97
parentf0c8d1f6dc8eac5a1fbf441c8e080721a7b6c0ff
iio: hid-sensor-trigger: Fix sometimes not powering up the sensor after resume

hid_sensor_set_power_work() powers the sensors back up after a resume
based on the user_requested_state atomic_t.

But hid_sensor_power_state() treats this as a boolean flag, leading to
the following problematic scenario:

1) Some app starts using the iio-sensor in buffered / triggered mode,
   hid_sensor_data_rdy_trigger_set_state(true) gets called, setting
   user_requested_state to 1.
2) Something directly accesses a _raw value through sysfs, leading
   to a call to hid_sensor_power_state(true) followed by
   hid_sensor_power_state(false) call, this sets user_requested_state
   to 1 followed by setting it to 0.
3) Suspend/resume the machine, hid_sensor_set_power_work() now does
   NOT power the sensor back up because user_requested_state (wrongly)
   is 0. Which stops the app using the sensor in buffered mode from
   receiving any new values.

This commit changes user_requested_state to a counter tracking how many
times hid_sensor_power_state(true) was called instead, fixing this.

Cc: Bastien Nocera <hadess@hadess.net>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/common/hid-sensors/hid-sensor-trigger.c