On 12/10/2022 03:44, Rajat Khandelwal wrote: > The datasheet states that the slave address for the device is 0x20 > when the pins A0 and A1 are ground. The DT binding has been using > 0x10 as the value and I think it should be 0x20 as per datasheet. > > Signed-off-by: Rajat Khandelwal <rajat.khandelwal@xxxxxxxxxxxxxxx> > --- > > v2: > 1. Subject prefix added > 2. DT bindings validated No, still wrong. Nothing was validated. > > Documentation/devicetree/bindings/iio/dac/maxim,ds4424.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/iio/dac/maxim,ds4424.yaml b/Documentation/devicetree/bindings/iio/dac/maxim,ds4424.yaml > index 264fa7c5fe3a..1955b1a97cd6 100644 > --- a/Documentation/devicetree/bindings/iio/dac/maxim,ds4424.yaml > +++ b/Documentation/devicetree/bindings/iio/dac/maxim,ds4424.yaml > @@ -38,7 +38,7 @@ examples: > > dac@10 { > compatible = "maxim,ds4424"; > - reg = <0x10>; /* When A0, A1 pins are ground */ > + reg = <0x20>; /* When A0, A1 pins are ground */ Nothing improved here. Best regards, Krzysztof