iio: imu: inv_icm42600: Avoid configuring if already pm_runtime suspended
commit 466f7a2fef2a4e426f809f79845a1ec1aeb558f4 upstream.
Do as in suspend, skip resume configuration steps if the device is already
pm_runtime suspended. This avoids reconfiguring a device that is already
in the correct low-power state and ensures that pm_runtime handles the
power state transitions properly.
Fixes: 31c24c1e93 ("iio: imu: inv_icm42600: add core of new inv_icm42600 driver")
Signed-off-by: Sean Nyekjaer <sean@geanix.com>
Link: https://patch.msgid.link/20250901-icm42pmreg-v3-3-ef1336246960@geanix.com
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
4f4b7a5a9d
commit
57fe5dae7a
@@ -837,17 +837,15 @@ static int inv_icm42600_suspend(struct device *dev)
|
||||
struct device *accel_dev;
|
||||
bool wakeup;
|
||||
int accel_conf;
|
||||
int ret;
|
||||
int ret = 0;
|
||||
|
||||
mutex_lock(&st->lock);
|
||||
|
||||
st->suspended.gyro = st->conf.gyro.mode;
|
||||
st->suspended.accel = st->conf.accel.mode;
|
||||
st->suspended.temp = st->conf.temp_en;
|
||||
if (pm_runtime_suspended(dev)) {
|
||||
ret = 0;
|
||||
if (pm_runtime_suspended(dev))
|
||||
goto out_unlock;
|
||||
}
|
||||
|
||||
/* disable FIFO data streaming */
|
||||
if (st->fifo.on) {
|
||||
@@ -900,10 +898,13 @@ static int inv_icm42600_resume(struct device *dev)
|
||||
struct inv_icm42600_sensor_state *accel_st = iio_priv(st->indio_accel);
|
||||
struct device *accel_dev;
|
||||
bool wakeup;
|
||||
int ret;
|
||||
int ret = 0;
|
||||
|
||||
mutex_lock(&st->lock);
|
||||
|
||||
if (pm_runtime_suspended(dev))
|
||||
goto out_unlock;
|
||||
|
||||
/* check wakeup capability */
|
||||
accel_dev = &st->indio_accel->dev;
|
||||
wakeup = st->apex.on && device_may_wakeup(accel_dev);
|
||||
|
||||
Reference in New Issue
Block a user