Re: [PATCH v1 3/3] dt-bindings: iio: adc: add ADS7924

From: Hugo Villeneuve
Date: Fri Dec 23 2022 - 12:56:29 EST


On Fri, 23 Dec 2022 14:09:08 +0000
Jonathan Cameron <jic23@xxxxxxxxxx> wrote:

> >
> > > + description:
> > > + Child nodes needed for each channel that the platform uses.
> >
> > I cannot understand this sentence at all. Instead describe the hardware
> > you are here representing. What's this?
>
> Needs rewording, but basically - "Which pins are connected to anything?"

Hi Jonathan and Krzysztof,
when writing this driver a few months ago, I used as a starting point the C source file and the YAML file of the TI ADS1015 driver.

I cannot say that I fully understand the sentence myself :) So I will reword it.

I will also take care of the other dt-bindings comments for version 2.

Thank you for your feedback.

Hugo V.

--
Hugo Villeneuve <hugo@xxxxxxxxxxx>