[PATCH v4 0/2] iio: gyro: mpu3050: fix pm_runtime error handling

Antoniu Miclaus posted 2 patches 1 month ago
drivers/iio/gyro/mpu3050-core.c | 18 +++++++++++++-----
drivers/iio/gyro/mpu3050-i2c.c  |  3 +--
2 files changed, 14 insertions(+), 7 deletions(-)
[PATCH v4 0/2] iio: gyro: mpu3050: fix pm_runtime error handling
Posted by Antoniu Miclaus 1 month ago
Fix pm_runtime error handling across the mpu3050 driver. The existing
code uses pm_runtime_get_sync() without checking return values, allowing
operations to proceed even when the device fails to resume.

Replace pm_runtime_get_sync() with pm_runtime_resume_and_get() and
propagate errors appropriately. Unlike pm_runtime_get_sync(), the usage
count is not incremented on error.

Changes in v4:
- Reword commit messages to correctly attribute the unchecked return
  value to the caller rather than pm_runtime_get_sync() itself
- Simplify preenable by collapsing the trailing return statements

Antoniu Miclaus (2):
  iio: gyro: mpu3050-i2c: fix pm_runtime error handling
  iio: gyro: mpu3050-core: fix pm_runtime error handling

 drivers/iio/gyro/mpu3050-core.c | 18 +++++++++++++-----
 drivers/iio/gyro/mpu3050-i2c.c  |  3 +--
 2 files changed, 14 insertions(+), 7 deletions(-)

-- 
2.43.0
Re: [PATCH v4 0/2] iio: gyro: mpu3050: fix pm_runtime error handling
Posted by Jonathan Cameron 3 weeks, 5 days ago
On Mon, 16 Feb 2026 11:57:54 +0200
Antoniu Miclaus <antoniu.miclaus@analog.com> wrote:

> Fix pm_runtime error handling across the mpu3050 driver. The existing
> code uses pm_runtime_get_sync() without checking return values, allowing
> operations to proceed even when the device fails to resume.
> 
> Replace pm_runtime_get_sync() with pm_runtime_resume_and_get() and
> propagate errors appropriately. Unlike pm_runtime_get_sync(), the usage
> count is not incremented on error.
Applied to the fixes-togreg branch of iio.git and marked for
stable.  Thanks

Jonathan

> 
> Changes in v4:
> - Reword commit messages to correctly attribute the unchecked return
>   value to the caller rather than pm_runtime_get_sync() itself
> - Simplify preenable by collapsing the trailing return statements
> 
> Antoniu Miclaus (2):
>   iio: gyro: mpu3050-i2c: fix pm_runtime error handling
>   iio: gyro: mpu3050-core: fix pm_runtime error handling
> 
>  drivers/iio/gyro/mpu3050-core.c | 18 +++++++++++++-----
>  drivers/iio/gyro/mpu3050-i2c.c  |  3 +--
>  2 files changed, 14 insertions(+), 7 deletions(-)
>