Re: [PATCH v3 1/1] dt-bindings: iio: adc: add AD7816/AD7817/AD7818 binding

From: Jonathan Cameron

Date: Tue May 12 2026 - 07:26:43 EST


On Mon, 11 May 2026 11:31:13 -0500
David Lechner <dlechner@xxxxxxxxxxxx> wrote:

> On 5/11/26 11:23 AM, Denny Lin wrote:
> > On Mon, May 11, 2026 at 7:51 AM David Lechner <dlechner@xxxxxxxxxxxx> wrote:
> >>
> >> Usually, we want to be moving the drivers out of staging when
> >> adding the DT bindings so that it all gets reviewed for correctness
> >> at the same time.
> >>
> >> Drivers in staging are usually there because they are doing things
> >> that are not actually acceptable for the final stable mainline
> >> version of a driver and need more work.
> >>
> >
> > Thanks for the explanation.
> >
> > I took a look at the driver and noticed it still uses the older
> > IIO sysfs-based interface rather than the iio_chan_spec + read_raw
> > model.
> >
> > Would you recommend updating the driver first, or is it okay to
> > proceed with the binding separately?
> >
> > Thanks,
> > Hungyu
>
> ADI's website says that these parts are obsolete, so unless anyone is
> actually still using these with a mainline kernel, I would suggest that
> we drop the driver rather than spending time fixing it up.

The ad7817 isn't obsolete yet unfortunately - that's the only reason I didn't
drop this last time I checked..

Any Analog folk want to take a view on this one - particularly if you
can check if it will be going obsolete soon. Good not to waste Denny's
time on this if so!

J

>
> In any case, unless you have the hardware for testing, it would be
> best not to make significant changes. There is a non-standard "oti"
> attribute that we would need to figure out what to do with. This is
> likely why the driver is in staging in the first place.