> -----Original Message----- > From: David Lechner <dlechner@xxxxxxxxxxxx> > Sent: Friday, February 2, 2024 10:38 PM > To: Sa, Nuno <Nuno.Sa@xxxxxxxxxx> > Cc: linux-iio@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; Lars-Peter Clausen > <lars@xxxxxxxxxx>; Hennerich, Michael <Michael.Hennerich@xxxxxxxxxx>; > Jonathan Cameron <jic23@xxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; > Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>; Conor Dooley > <conor+dt@xxxxxxxxxx>; Frank Rowand <frowand.list@xxxxxxxxx>; Olivier > Moysan <olivier.moysan@xxxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx> > Subject: Re: [PATCH v8 2/7] dt-bindings: adc: axi-adc: update bindings for backend > framework > > > On Fri, Feb 2, 2024 at 9:10 AM Nuno Sa via B4 Relay > <devnull+nuno.sa.analog.com@xxxxxxxxxx> wrote: > > > > From: Nuno Sa <nuno.sa@xxxxxxxxxx> > > > > 'adi,adc-dev' is now deprecated and must not be used anymore. Hence, > > also remove it from being required. > > > > The reason why it's being deprecated is because the axi-adc CORE is now > > an IIO service provider hardware (IIO backends) for consumers to make use > > of. Before, the logic with 'adi,adc-dev' was the opposite (it was kind > > of consumer referencing other nodes/devices) and that proved to be wrong > > and to not scale. > > > > Now, IIO consumers of this hardware are expected to reference it using the > > io-backends property. Hence, the new '#io-backend-cells' is being added > > so the device is easily identified as a provider. > > > > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > > Signed-off-by: Nuno Sa <nuno.sa@xxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/iio/adc/adi,axi-adc.yaml | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,axi-adc.yaml > b/Documentation/devicetree/bindings/iio/adc/adi,axi-adc.yaml > > index 9996dd93f84b..add10b22dcac 100644 > > --- a/Documentation/devicetree/bindings/iio/adc/adi,axi-adc.yaml > > +++ b/Documentation/devicetree/bindings/iio/adc/adi,axi-adc.yaml > > @@ -39,12 +39,15 @@ properties: > > $ref: /schemas/types.yaml#/definitions/phandle > > description: > > A reference to a the actual ADC to which this FPGA ADC interfaces to. > > + deprecated: true > > + > > + '#io-backends-cells' > > Still missing the : here. Ahh crap!! I was so blindly assuming the error report was because the property could still be missing in the core schemas that I did not even paid attention. Thanks for this! But worst, the property is not even correct. #io-backends-cells -> #io-backend-cells Both here and in the example. Jonathan, sorry about this... Let me know if this is something you can fix or if you want me to spin another version. - Nuno Sá
