Re: [PATCH v6 5/5] iio: adc: ad9467: check for backend capabilities

["Nuno Sá" <noname.nuno@gmail.com>]

On Sun, 2026-02-08 at 19:30 +0000, Jonathan Cameron wrote:
> On Thu, 05 Feb 2026 12:24:11 +0000
> Tomas Melin <tomas.melin@vaisala.com> wrote:
> 
> > Add capability checks for operation with backends that do not necessarily
> > support full set of features, but are otherwise compatible with the device.
> > This ensures a fully functional device, but with limited capabilities.
> > 
> > Signed-off-by: Tomas Melin <tomas.melin@vaisala.com>
> I've mentioned before that I've started messing around with Chris Mason's
> set of Claude Code prompts and today I was trying out the new stuff mentioned in:
> https://lore.kernel.org/all/b187e0c1-1df8-4529-bfe4-0a1d65221adc@meta.com/
> 

Yeps, we're doing the same and adding the above to our CI. I guess an IIO skill will
be helpful long term.

> 
> Given this patch set is pretty much ready to merge I asked it to take a look
> and it came up with something that certainly merited me taking a look.
> 
>  Issue: In ad9467_chan_test_mode_write(), the testmode_on path is guarded
>   by IIO_BACKEND_CAP_CALIBRATION but the matching testmode_off path is not.
>   On backends without that capability, entering PN test mode succeeds but
>   exiting fails with -EOPNOTSUPP, leaving the ADC stuck in test mode.
> 
> I may be missing some protection path, but I think it is right and if you
> were to set up the test modes with a backend that doesn't support the
> relevant functionality you would indeed be able to turn them on but never
> off again.
> 
> So this might be the first bug in IIO that the LLM found and human's, including
> me, missed!  

Yes, that seems to be the case.

- Nuno Sá