[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240620-custody-jailbreak-6540620d6570@wendy>
Date: Thu, 20 Jun 2024 09:51:03 +0100
From: Conor Dooley <conor.dooley@...rochip.com>
To: Olivier MOYSAN <olivier.moysan@...s.st.com>
CC: Conor Dooley <conor@...nel.org>, Arnaud Pouliquen
<arnaud.pouliquen@...s.st.com>, Jonathan Cameron <jic23@...nel.org>,
Lars-Peter Clausen <lars@...afoo.de>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Maxime Coquelin <mcoquelin.stm32@...il.com>, Alexandre Torgue
<alexandre.torgue@...s.st.com>, Fabrice Gasnier
<fabrice.gasnier@...s.st.com>, <alsa-devel@...a-project.org>,
<linux-iio@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-stm32@...md-mailman.stormreply.com>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 4/8] dt-bindings: iio: dfsdm: move to backend framework
On Thu, Jun 20, 2024 at 10:03:44AM +0200, Olivier MOYSAN wrote:
> On 6/18/24 20:10, Conor Dooley wrote:
> > On Tue, Jun 18, 2024 at 06:08:30PM +0200, Olivier Moysan wrote:
> > > allOf:
> > > - if:
> > > properties:
> > > @@ -199,9 +264,19 @@ patternProperties:
> > > description:
> > > From common IIO binding. Used to pipe external sigma delta
> > > modulator or internal ADC output to DFSDM channel.
> > > + deprecated: true
> > > - required:
> > > - - io-channels
> > > + if:
> > > + required:
> > > + - st,adc-channels
> > > + then:
> > > + required:
> > > + - io-channels
> > > +
> > > + patternProperties:
> > > + "^channel@([0-9]|1[0-9])$":
> > > + required:
> > > + - io-backends
> >
> > Why is this here, rather than with reg above? Only some channels require
> > a backend?
>
> The io-backends property is required only when we use st,stm32-dfsdm-adc
> compatible. In other words, when we are in an analog use case. In this case
> the channel is a consumer of a backend (typically a sd modulator)
> In an audio use case (compatible st,stm32-dfsdm-dmic) the backend is not
> required.
Ahh, I think the hunks and indent confused me here. What you're doing is
making io-backends required based on the compatible, but what I thought
you were doing was trying to make io-backends required in channels if
st,adc-channels was set.
Thanks for the explanation,
Conor.
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists