Commit 5a997fa6 authored Nov 16, 2024 by Carlos Song Committed by Wen Zhiwei Mar 11, 2025

iio: gyro: fxas21002c: Fix missing data update in trigger handler

stable inclusion
from stable-v6.6.72
commit 2df664d7b4f2550a8df894b4cbf3128ba6344381
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IBQN9L

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=2df664d7b4f2550a8df894b4cbf3128ba6344381



--------------------------------

commit fa13ac6cdf9b6c358e7d77c29fb60145c7a87965 upstream.

The fxas21002c_trigger_handler() may fail to acquire sample data because
the runtime PM enters the autosuspend state and sensor can not return
sample data in standby mode..

Resume the sensor before reading the sample data into the buffer within the
trigger handler. After the data is read, place the sensor back into the
autosuspend state.

Fixes: a0701b62 ("iio: gyro: add core driver for fxas21002c")
Signed-off-by: Carlos Song <carlos.song@nxp.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
Link: https://patch.msgid.link/20241116152945.4006374-1-Frank.Li@nxp.com


Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Wen Zhiwei <wenzhiwei@kylinos.cn>

parent 479180ee

drivers/iio/gyro/fxas21002c_core.c

+8 −1

Original line number	Diff line number	Diff line
		@@ -730,14 +730,21 @@ static irqreturn_t fxas21002c_trigger_handler(int irq, void *p)
		int ret;

		mutex_lock(&data->lock);
		ret = fxas21002c_pm_get(data);
		if (ret < 0)
		goto out_unlock;

		ret = regmap_bulk_read(data->regmap, FXAS21002C_REG_OUT_X_MSB,
		data->buffer, CHANNEL_SCAN_MAX * sizeof(s16));
		if (ret < 0)
		goto out_unlock;
		goto out_pm_put;

		iio_push_to_buffers_with_timestamp(indio_dev, data->buffer,
		data->timestamp);

		out_pm_put:
		fxas21002c_pm_put(data);

		out_unlock:
		mutex_unlock(&data->lock);