Re: [PATCH v2 2/2] iio: adc: max1363: Reorder mode_list[] entries

[Jonathan Cameron <jic23@kernel.org>]

On Mon, 19 May 2025 08:02:33 +0300
Matti Vaittinen <mazziesaccount@gmail.com> wrote:

> On 16/05/2025 20:39, Fabio Estevam wrote:
> > From: Fabio Estevam <festevam@denx.de>
> > 
> > The IIO core issues warnings when a scan mask is a subset of a previous
> > entry in the available_scan_masks array.
> > 
> > On a board using a MAX11601, the following warning is observed:
> > 
> > max1363 1-0064: available_scan_mask 7 subset of 6. Never used
> > 
> > This occurs because the entries in the max11607_mode_list[] array are not
> > ordered correctly. To fix this, reorder the entries so that no scan mask is
> > a subset of an earlier one.
> > 
> > While at it, reorder the mode_list[] arrays for other supported chips as
> > well, to prevent similar warnings on different variants.
> > 
> > Cc: stable@vger.kernel.org
> > Fixes: 2718f15403fb ("iio: sanity check available_scan_masks array")  
> 
> Same comment as for patch1/2
> 
> Yours,
> 	-- Matti
Likewise, dropped the fixes tag.  Admittedly that's the somewhat lazy
option.  I'd less fussed than normal on this because the worst that
happens is we print warnings and sample more channels than strictly
necessary.

Jonathan