Re: [PATCH togreg v2 0/2] iio: imu: inv_icm42607: fix PM error handling

Andy Shevchenko <[email protected]>
Newsgroups org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel
Organization Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo
Message-ID <[email protected]>
On Tue, Aug 11, 2026 at 10:03:43AM +0800, Linmao Li wrote:
> The recently queued ICM-42607 PM support has two error paths that can leave
> the PM core's state inconsistent with the device.
> 
> Patch 1 propagates sensor shutdown failures from runtime suspend, matching
> the behavior of the sibling ICM-42600 driver. Patch 2 ensures that system
> resume restores runtime PM management on both of its error paths, so that
> a failed resume does not leave runtime PM disabled for good.
> 
> Changes since v1:
> - Patch 2: split the device side of inv_icm42607_resume() into a helper so
>   the PM bookkeeping stays in the wrapper, per Andy's review.  No
>   functional change.
> - Rebased onto the current togreg head.
> - Patch 1 is unchanged.
> 
> The Fixes commit is in iio.git togreg and has been included in
> linux-next. It has not reached mainline.

> Based on iio.git togreg at 350d1fb9204b.

Unneeded info since you are correctly used --base and we see that below.

> Both patches were compile-tested with W=1 and checked with smatch. No
> ICM-42607 hardware or fault-injection setup was available.
> 
> Linmao Li (2):
>   iio: imu: inv_icm42607: propagate runtime suspend errors
>   iio: imu: inv_icm42607: restore runtime PM on system resume errors
> 
>  .../iio/imu/inv_icm42607/inv_icm42607_core.c  | 41 ++++++++++++++-----
>  1 file changed, 31 insertions(+), 10 deletions(-)
> 
> 
> base-commit: 350d1fb9204b13c5f95e511e98b8bcb47574d425

-- 
With Best Regards,
Andy Shevchenko
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.