Re: [PATCH v7 4/8] iio: magnetometer: ak8975: consistently use 'data' parameter

[Jonathan Cameron <jic23@kernel.org>]

On Mon, 11 May 2026 13:26:13 +0200
Joshua Crofts via B4 Relay <devnull+joshua.crofts1.gmail.com@kernel.org> wrote:

> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> Some of the functions use 'client', some use 'data', and some use both.
> Refactor the driver to consistently use 'data' in all cases.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Reviewed-by: Nuno Sá <nuno.sa@analog.com>
> Signed-off-by: Joshua Crofts <joshua.crofts1@gmail.com>
Note that whilst I've applied this sashiko pointed out some more
while I was here issues.

The one about not ensuring the power is on for buffered captures
is a pretty major bug :( 

https://sashiko.dev/#/patchset/20260511-magnetometer-fixes-post-pickup-v7-0-9d910faa28b6%40gmail.com

Which is a pain because we have lots of cleanup on this driver already
and I don't want to rip it out.  The fix is pretty simple though
(add a preenable, postdissable pair of opts and pm_runtime_get_sync()
/ put_autosuspend() as appropriate.

So if you don't mind please put together that patch and I'll apply it
via the testing/togreg tree and we'll do a manual backport later.

Jonathan