[PATCH 0/6] dt-bindings: Add some missing IIO related binding docs.

[PATCH 0/6] dt-bindings: Add some missing IIO related binding docs.

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

First of what will probably be one of several sets to slowly but surely
add yaml binding docs for all the defacto bindings hiding in IIO
drivers.  Ultimate I'd like to clear the backlog of these and so go
forward with full documenation, but there may be some cases we just decide
are too complex to bother.  The fsl,mma955x parts are ignored (for now) on
that basis (I think we have all the other accelerometers covered now)

Some of these date back to board file days and some were added by people
only interested in ACPI, but all can be instantiated from dt-bindings.
A few may simply have slipped through the net during review.

Testing done with hacked up qemu when anything was non trivial.

Jonathan Cameron (6):
  dt-bindings:iio:accel:adis16201 and adis16209 bindings
  dt-bindings:iio:accel:bosch,bma220 device tree binding documentation
  dt-bindings:iio:accel:fsl,mma7455 binding doc
  dt-bindings:trivial-devices: Add memsic,mxc4005/mxc6255/mxc6655
    entries
  dt-bindings:trivial-devices: Add sensortek,stk8312 and
    sensortek,s8ba50
  dt-bindings:iio:adc:adi,ad7298 document bindings

 .../bindings/iio/accel/adi,adis16201.yaml     | 55 +++++++++++++
 .../bindings/iio/accel/bosch,bma220.yaml      | 50 +++++++++++
 .../bindings/iio/accel/fsl,mma7455.yaml       | 82 +++++++++++++++++++
 .../bindings/iio/adc/adi,ad7298.yaml          | 48 +++++++++++
 .../devicetree/bindings/trivial-devices.yaml  | 10 +++
 5 files changed, 245 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml
 create mode 100644 Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml
 create mode 100644 Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
 create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7298.yaml

-- 
2.31.1

[PATCH 1/6] dt-bindings:iio:accel:adis16201 and adis16209 bindings

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

These two devices have different internal characterstics, but their
external connectivity and as a result device tree descriptions are
identical.

Note that neither driver in Linux currently has an of_match_table
but instead rely on matching via name alone.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 .../bindings/iio/accel/adi,adis16201.yaml     | 55 +++++++++++++++++++
 1 file changed, 55 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml
new file mode 100644
index 000000000000..6f8f8a6258fe
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml
@@ -0,0 +1,55 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/accel/adi,adis16201.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: ADIS16201 Dual Axis Inclinometer and similar
+
+maintainers:
+  - Jonathan Cameron <Jonathan.Cameron@huawei.com>
+
+description: |
+  Two similar parts from external interface point of view.
+  SPI interface.
+    https://www.analog.com/en/products/adis16201.html
+    https://www.analog.com/en/products/adis16209.html
+
+properties:
+  compatible:
+    enum:
+      - adi,adis16201
+      - adi,adis16209
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  spi-max-frequency: true
+
+  vdd-supply: true
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+    spi0 {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        accelerometer@0 {
+            compatible = "adi,adis16201";
+            reg = <0>;
+            spi-max-frequency = <2500000>;
+            interrupt-parent = <&gpio0>;
+            interrupts = <0 IRQ_TYPE_LEVEL_HIGH>;
+        };
+    };
+...
-- 
2.31.1

Re: [PATCH 1/6] dt-bindings:iio:accel:adis16201 and adis16209 bindings

[Rob Herring]

On Thu, Apr 01, 2021 at 06:41:07PM +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> These two devices have different internal characterstics, but their

typo: characteristics

> external connectivity and as a result device tree descriptions are
> identical.
> 
> Note that neither driver in Linux currently has an of_match_table
> but instead rely on matching via name alone.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
>  .../bindings/iio/accel/adi,adis16201.yaml     | 55 +++++++++++++++++++
>  1 file changed, 55 insertions(+)

Otherwise,

Reviewed-by: Rob Herring <robh@kernel.org>

[PATCH 2/6] dt-bindings:iio:accel:bosch,bma220 device tree binding documentation

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Linux driver currently relies on the driver.name to match to the
compatible, but that doesn't mean it isn't a good idea to document it.

Only real complexity is the 3 separate power supplies.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 .../bindings/iio/accel/bosch,bma220.yaml      | 50 +++++++++++++++++++
 1 file changed, 50 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml b/Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml
new file mode 100644
index 000000000000..942b23ad0712
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml
@@ -0,0 +1,50 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/accel/bosch,bma220.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Bosch BMA220 Trixial Acceleration Sensor
+
+maintainers:
+  - Jonathan Cameron <Jonathan.Cameron@huawei.com>
+
+properties:
+  compatible:
+    enum:
+      - bosch,bma220
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  spi-max-frequency: true
+
+  vdda-supply: true
+  vddd-supply: true
+  vddio-supply: true
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+    spi0 {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        accelerometer@0 {
+            compatible = "bosch,bma220";
+            reg = <0>;
+            spi-max-frequency = <2500000>;
+            interrupt-parent = <&gpio0>;
+            interrupts = <0 IRQ_TYPE_LEVEL_HIGH>;
+        };
+    };
+...
-- 
2.31.1

Re: [PATCH 2/6] dt-bindings:iio:accel:bosch,bma220 device tree binding documentation

[Rob Herring]

On Thu, 01 Apr 2021 18:41:08 +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> Linux driver currently relies on the driver.name to match to the
> compatible, but that doesn't mean it isn't a good idea to document it.
> 
> Only real complexity is the 3 separate power supplies.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
>  .../bindings/iio/accel/bosch,bma220.yaml      | 50 +++++++++++++++++++
>  1 file changed, 50 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

[PATCH 3/6] dt-bindings:iio:accel:fsl,mma7455 binding doc

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Binding documentation for this driver was missing.
Note that the binding and some in tree dts files specifiy interrupts
etc but the driver is fairly simple and does not yet make use of them.

Listed both Joachim and myself as maintainers for this binding on basis
it's Joachim's driver but I don't want to volunteer Joachim too
strongly to look after this doc.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Joachim Eastwood <manabian@gmail.com>
---
 .../bindings/iio/accel/fsl,mma7455.yaml       | 82 +++++++++++++++++++
 1 file changed, 82 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
new file mode 100644
index 000000000000..7c8f8bdc2333
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
@@ -0,0 +1,82 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/accel/fsl,mma7455.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Freescale MMA7455 and MMA7456 three axis accelerometers
+
+maintainers:
+  - Joachim Eastwood <manabian@gmail.com>
+  - Jonathan Cameron <jic23@kernel.org>
+
+description:
+  Devices support both SPI and I2C interfaces.
+
+properties:
+  compatible:
+    enum:
+      - fsl,mma7455
+      - fsl,mma7456
+  reg:
+    maxItems: 1
+
+  avdd-supply: true
+  vddio-supply: true
+
+  interrupts:
+    minItems: 1
+    maxItems: 2
+
+  interrupt-names:
+    description:
+      Data ready is only available on INT1, but events can use either or
+      both pins.  If not specified, first element assumed to correspond
+      to INT1 and second (where present) to INT2.
+    minItems: 1
+    maxItems: 2
+    items:
+      enum:
+        - "INT1"
+        - "INT2"
+
+  spi-max-frequency: true
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    # include <dt-bindings/interrupt-controller/irq.h>
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        accelerometer@18 {
+            compatible = "fsl,mma7455";
+            reg = <0x18>;
+            vddio-supply = <&iovdd>;
+            avdd-supply = <&avdd>;
+            interrupts = <57 IRQ_TYPE_EDGE_FALLING>, <58 IRQ_TYPE_EDGE_FALLING>;
+            interrupt-names = "INT2", "INT1";
+        };
+    };
+  - |
+    # include <dt-bindings/interrupt-controller/irq.h>
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+        accelerometer@0 {
+            compatible = "fsl,mma7456";
+            reg = <0>;
+            spi-max-frequency = <10000000>;
+            vddio-supply = <&iovdd>;
+            avdd-supply = <&avdd>;
+            interrupts = <57 IRQ_TYPE_EDGE_FALLING>;
+            interrupt-names = "INT1";
+        };
+    };
+...
-- 
2.31.1

Re: [PATCH 3/6] dt-bindings:iio:accel:fsl,mma7455 binding doc

[Rob Herring]

On Thu, 01 Apr 2021 18:41:09 +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> Binding documentation for this driver was missing.
> Note that the binding and some in tree dts files specifiy interrupts
> etc but the driver is fairly simple and does not yet make use of them.
> 
> Listed both Joachim and myself as maintainers for this binding on basis
> it's Joachim's driver but I don't want to volunteer Joachim too
> strongly to look after this doc.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> Cc: Joachim Eastwood <manabian@gmail.com>
> ---
>  .../bindings/iio/accel/fsl,mma7455.yaml       | 82 +++++++++++++++++++
>  1 file changed, 82 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

[PATCH 4/6] dt-bindings:trivial-devices: Add memsic,mxc4005/mxc6255/mxc6655 entries

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Simple devices with I2C interface and single interrupt line so
entries in trivial devices are sufficient.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index a327130d1faa..c7d49a720afa 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -166,8 +166,14 @@ properties:
           - meas,tsys01
             # MEMSIC magnetometer
           - memsic,mmc35240
+            # MEMSIC 3-axis accelerometer
+          - memsic,mx4005
             # MEMSIC 2-axis 8-bit digital accelerometer
           - memsic,mxc6225
+            # MEMSIC 2-axis 8-bit digital accelerometer
+          - memsic,mxc6255
+            # MEMSIC 3-axis accelerometer
+          - memsic,mxc6655
             # Microchip differential I2C ADC, 1 Channel, 18 bit
           - microchip,mcp3421
             # Microchip differential I2C ADC, 2 Channel, 18 bit
-- 
2.31.1

Re: [PATCH 4/6] dt-bindings:trivial-devices: Add memsic,mxc4005/mxc6255/mxc6655 entries

[Rob Herring]

On Thu, 01 Apr 2021 18:41:10 +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> Simple devices with I2C interface and single interrupt line so
> entries in trivial devices are sufficient.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
>  Documentation/devicetree/bindings/trivial-devices.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

[PATCH 5/6] dt-bindings:trivial-devices: Add sensortek,stk8312 and sensortek,s8ba50

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Two simple devices. The stk8ba50 datasheet could be found via
google, but I only have the driver for the 8312.

Given they both seem to be 3 axis devices with a single interrupt
line, add them to trivial-devices.yaml

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index c7d49a720afa..92ea6b960e16 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -258,6 +258,10 @@ properties:
           - sensirion,sgpc3
             # Sensirion multi-pixel gas sensor with I2C interface
           - sensirion,sgp30
+            # Sensortek 3 axis accelerometer
+          - sensortek,stk8312
+            # Sensortek 3 axis accelerometer
+          - sensortek,stk8ba50
             # SGX Sensortech VZ89X Sensors
           - sgx,vz89x
             # Relative Humidity and Temperature Sensors
-- 
2.31.1

Re: [PATCH 5/6] dt-bindings:trivial-devices: Add sensortek,stk8312 and sensortek,s8ba50

[Rob Herring]

On Thu, 01 Apr 2021 18:41:11 +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> Two simple devices. The stk8ba50 datasheet could be found via
> google, but I only have the driver for the 8312.
> 
> Given they both seem to be 3 axis devices with a single interrupt
> line, add them to trivial-devices.yaml
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
>  Documentation/devicetree/bindings/trivial-devices.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

[PATCH 6/6] dt-bindings:iio:adc:adi,ad7298 document bindings

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

The device has a tsens-busy pin, but it's both fiddly and currently
ignored by the Linux driver.  Given it's not clear whether the binding
should be an interrupt, or a GPIO I have left that out for now.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Michael Hennerich <michael.hennerich@analog.com>
---
 .../bindings/iio/adc/adi,ad7298.yaml          | 48 +++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7298.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7298.yaml
new file mode 100644
index 000000000000..ca414bb396c5
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7298.yaml
@@ -0,0 +1,48 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+# Copyright 2019 Analog Devices Inc.
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/adi,ad7298.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Analog Devices AD7298 ADC
+
+maintainers:
+  - Michael Hennerich <michael.hennerich@analog.com>
+
+description: |
+  Bindings for the Analog Devices AD7298 ADC device. Datasheet can be
+  found here:
+    https://www.analog.com/en/products/ad7298.html
+
+properties:
+  compatible:
+    const: adi,ad7298
+
+  reg:
+    maxItems: 1
+
+  vref-supply: true
+  vdd-supply: true
+  spi-max-frequency: true
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+      #address-cells = <1>;
+      #size-cells = <0>;
+
+      adc@0 {
+        compatible = "adi,ad7298";
+        reg = <0>;
+        spi-max-frequency = <5000000>;
+        vref-supply = <&adc_vref>;
+      };
+    };
+...
-- 
2.31.1

Re: [PATCH 6/6] dt-bindings:iio:adc:adi,ad7298 document bindings

[Rob Herring]

On Thu, 01 Apr 2021 18:41:12 +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> The device has a tsens-busy pin, but it's both fiddly and currently
> ignored by the Linux driver.  Given it's not clear whether the binding
> should be an interrupt, or a GPIO I have left that out for now.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> Cc: Michael Hennerich <michael.hennerich@analog.com>
> ---
>  .../bindings/iio/adc/adi,ad7298.yaml          | 48 +++++++++++++++++++
>  1 file changed, 48 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

Re: [PATCH 0/6] dt-bindings: Add some missing IIO related binding docs.

[Jonathan Cameron]

On Thu,  1 Apr 2021 18:41:06 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> First of what will probably be one of several sets to slowly but surely
> add yaml binding docs for all the defacto bindings hiding in IIO
> drivers.  Ultimate I'd like to clear the backlog of these and so go
> forward with full documenation, but there may be some cases we just decide
> are too complex to bother.  The fsl,mma955x parts are ignored (for now) on
> that basis (I think we have all the other accelerometers covered now)
> 
> Some of these date back to board file days and some were added by people
> only interested in ACPI, but all can be instantiated from dt-bindings.
> A few may simply have slipped through the net during review.
> 
> Testing done with hacked up qemu when anything was non trivial.

Series applied with typo fix in patch 1 description that Rob pointed out.

Thanks,

Jonathan

> 
> Jonathan Cameron (6):
>   dt-bindings:iio:accel:adis16201 and adis16209 bindings
>   dt-bindings:iio:accel:bosch,bma220 device tree binding documentation
>   dt-bindings:iio:accel:fsl,mma7455 binding doc
>   dt-bindings:trivial-devices: Add memsic,mxc4005/mxc6255/mxc6655
>     entries
>   dt-bindings:trivial-devices: Add sensortek,stk8312 and
>     sensortek,s8ba50
>   dt-bindings:iio:adc:adi,ad7298 document bindings
> 
>  .../bindings/iio/accel/adi,adis16201.yaml     | 55 +++++++++++++
>  .../bindings/iio/accel/bosch,bma220.yaml      | 50 +++++++++++
>  .../bindings/iio/accel/fsl,mma7455.yaml       | 82 +++++++++++++++++++
>  .../bindings/iio/adc/adi,ad7298.yaml          | 48 +++++++++++
>  .../devicetree/bindings/trivial-devices.yaml  | 10 +++
>  5 files changed, 245 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml
>  create mode 100644 Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml
>  create mode 100644 Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml
>  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7298.yaml
>