Thread (9 messages) 9 messages, 4 authors, 2026-02-03

RE: [PATCH 1/2] dt-bindings: iio: adc: ad4080: add AD4880 support

From: "Miclaus, Antoniu" <Antoniu.Miclaus@analog.com>
Date: 2026-02-03 15:43:19
Also in: linux-iio, lkml

...
quoted
+    then:
+      properties:
+        io-backends:
+          minItems: 2
+      required:
+        - adi,aux-spi-cs
+    else:
+      properties:
+        io-backends:
+          maxItems: 1
+
 additionalProperties: false

 examples:
@@ -98,4 +127,22 @@ examples:
           io-backends = <&iio_backend>;
         };
     };
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+          compatible = "adi,ad4880";
+          reg = <0>;
+          adi,aux-spi-cs = <1>;
We can already have multiple reg for a multiple CS device.

             reg = <0>, <1>;

So we shouldn't need a new adi,aux-spi-cs property for that.
Sure, that was the initial plan but for some reason going for multiple regs gets me to this:

[   11.396833]  spi_new_ancillary_device from ad4080_probe+0x4b8/0x5a4
[   11.396861]  ad4080_probe from really_probe+0xc8/0x2c8
[   11.397281] ad4080 spi1.0: chipselect 1 already in use
[   11.397297] ad4080 spi1.0: failed to register ancillary device
[   11.397317] ad4080 spi1.0: probe with driver ad4080 failed with error -16

While the aux-spi-cs works fine. Any tips?
quoted
+          spi-max-frequency = <10000000>;
+          vdd33-supply = <&vdd33>;
+          vddldo-supply = <&vddldo>;
+          vrefin-supply = <&vrefin>;
+          clocks = <&cnv>;
+          clock-names = "cnv";
+          io-backends = <&iio_backend_cha>, <&iio_backend_chb>;
+        };
+    };
 ...
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help