On Thu, 29 Feb 2024 10:25:50 -0600 David Lechner <dlechner@xxxxxxxxxxxx> wrote: > This adds a new binding for the Analog Devices, Inc. AD7944, AD7985, and > AD7986 ADCs. > > Signed-off-by: David Lechner <dlechner@xxxxxxxxxxxx> Hi David, One trivial comment. If this is all that comes up I can tweak that whilst applying. Jonathan ... > + adi,spi-mode: > + $ref: /schemas/types.yaml#/definitions/string > + enum: [ single, chain ] > + description: | > + This property indicates the SPI wiring configuration. > + > + When this property is omitted, it is assumed that the device is using what > + the he datasheet calls "4-wire mode". This is the conventional SPI mode Stray "he"? > + used when there are multiple devices on the same bus. In this mode, the > + CNV line is used to initiate the conversion and the SDI line is connected > + to CS on the SPI controller. >