Re: [PATCH v7 03/10] iio: adc: add helpers for parsing ADC nodes

[Matti Vaittinen]

On 16/03/2025 11:38, Jonathan Cameron wrote:
> On Thu, 13 Mar 2025 09:18:18 +0200
> Matti Vaittinen <mazziesaccount@xxxxxxxxx> wrote:
>
> > There are ADC ICs which may have some of the AIN pins usable for other
> > functions. These ICs may have some of the AIN pins wired so that they
> > should not be used for ADC.
> >
> > (Preferred?) way for marking pins which can be used as ADC inputs is to
> > add corresponding channels@N nodes in the device tree as described in
> > the ADC binding yaml.
> >
> > Add couple of helper functions which can be used to retrieve the channel
> > information from the device node.
> I suspect we'll need the addition of an optional trailing timestamp
> channel at somepoint. But we can add that when we need it as only
> matters for drivers doing iio_push_to_buffers_with_timestamp()

This is true. That'll enable using this for devices with buffers - which 
is not possible right now as most buffer users do timestamps. I'll leave 
adding the parameter to first buffered user though, but I think it's 
good to say out loud this is doable :) Thanks!

> Otherwise no additional comments from me.

Yours,
	-- Matti