On Fri, Sep 6, 2024 at 1:02 AM David Lechner <dlechner@xxxxxxxxxxxx> wrote: > > On 9/5/24 3:24 AM, Alexandru Ardelean wrote: > > The driver will support the AD7606C-16 and AD7606C-18. > > This change adds the compatible strings for these devices. > > > > The AD7606C-16,18 channels also support these (individually configurable) > > types of channels: > > - bipolar single-ended > > - unipolar single-ended > > - bipolar differential > > > > Signed-off-by: Alexandru Ardelean <aardelean@xxxxxxxxxxxx> > > --- > > ... > > > @@ -170,6 +220,17 @@ allOf: > > adi,conversion-start-gpios: > > maxItems: 1 > > > > + - if: > > + not: > > + properties: > > + compatible: > > + enum: > > + - adi,ad7606c-16 > > + - adi,ad7606c-18 > > + then: > > + patternProperties: > > + "^channel@[1-8]$": false > > + > > Technically, we should also have "^channel@[1-8]$": false > if adi,sw-mode is not set (not: required: [adi,sw-mode]). Yes. That's true. Maybe I'll add another condition here for that.