Re: [PATCH v2 0/2] iio: light: Report the available lists via read_avail()

[Jonathan Cameron]

On Mon,  9 Nov 2026 14:07:36 +0900
Arka Mondal <arka@xxxxxxxxxxxxxx> wrote:

> These are patches 1 and 3 of the v1 series. Patches 2, 4, 5 and 6 are
> applied and are not resent.
> 
> Changes since v1:
> 
> - isl29125: the available bit goes in info_mask_shared_by_all_available
>   instead of info_mask_shared_by_type_available, which is the sharing
>   level that names the file scale_available, so the attribute keeps the
>   name it has and no ABI changes. The Fixes: tag is gone with it, and
>   the patch is a cleanup rather than a fix.
> 
> - max44009: the list is IIO_VAL_INT_PLUS_MICRO instead of
>   IIO_VAL_INT_PLUS_NANO. All eight values are a whole number of
>   microseconds, so three zeros go and nothing is rounded.
> 
> I loaded both drivers against CONFIG_I2C_STUB under qemu and read the
> attributes. isl29125 shows in_intensity_scale and scale_available, no
> in_intensity_scale_available, and scale_available still reads
> 0.005722 0.152590. max44009 shows in_illuminance_integration_time_available
> reading 0.800000 0.400000 0.200000 0.100000 0.050000 0.025000 0.012500 0.006250,
> and no unprefixed name.
> 
> Link: https://lore.kernel.org/r/20261102074700.63075-1-arka@xxxxxxxxxxxxxx

Applied both to the testing branch of iio.git.
I added a note on the ABI break in patch 2 being a fix of buggy ABI and
cross our fingers case

Thanks

Jonathan

> 
> Arka Mondal (2):
>   iio: light: isl29125: Report the scale list via read_avail()
>   iio: light: max44009: Report the integration time via read_avail()
> 
>  drivers/iio/light/isl29125.c | 29 +++++++++++++++-------
>  drivers/iio/light/max44009.c | 48 ++++++++++++++++++++----------------
>  2 files changed, 47 insertions(+), 30 deletions(-)
> 
> 
> base-commit: d135ab453a7400a34930bdda1ef4bdc8acfac80e