diff mbox series

dt-bindings: iio: accel: kionix,kxsd9 yaml conversion.

Message ID 20200628203205.49915-1-jic23@kernel.org
State New
Headers show
Series dt-bindings: iio: accel: kionix,kxsd9 yaml conversion. | expand

Commit Message

Jonathan Cameron June 28, 2020, 8:32 p.m. UTC
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>


Convert and flesh out a bit the binding docs for this simple 3 axis
accelerometer.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

---

Sorry, typo in iio mailing list in first try at sending this out.

.../bindings/iio/accel/kionix,kxsd9.txt       | 22 -------
 .../bindings/iio/accel/kionix,kxsd9.yaml      | 65 +++++++++++++++++++
 2 files changed, 65 insertions(+), 22 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.txt
 create mode 100644 Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml

-- 
2.27.0

Comments

Rob Herring (Arm) July 15, 2020, 8:25 p.m. UTC | #1
On Sun, 28 Jun 2020 21:32:05 +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> 

> Convert and flesh out a bit the binding docs for this simple 3 axis

> accelerometer.

> 

> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> ---

> 

> Sorry, typo in iio mailing list in first try at sending this out.

> 

> .../bindings/iio/accel/kionix,kxsd9.txt       | 22 -------

>  .../bindings/iio/accel/kionix,kxsd9.yaml      | 65 +++++++++++++++++++

>  2 files changed, 65 insertions(+), 22 deletions(-)

>  delete mode 100644 Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.txt

>  create mode 100644 Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml

> 


Reviewed-by: Rob Herring <robh@kernel.org>
Jonathan Cameron July 18, 2020, 4:27 p.m. UTC | #2
On Wed, 15 Jul 2020 14:25:14 -0600
Rob Herring <robh@kernel.org> wrote:

> On Sun, 28 Jun 2020 21:32:05 +0100, Jonathan Cameron wrote:

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

> > 

> > Convert and flesh out a bit the binding docs for this simple 3 axis

> > accelerometer.

> > 

> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> > ---

> > 

> > Sorry, typo in iio mailing list in first try at sending this out.

> > 

> > .../bindings/iio/accel/kionix,kxsd9.txt       | 22 -------

> >  .../bindings/iio/accel/kionix,kxsd9.yaml      | 65 +++++++++++++++++++

> >  2 files changed, 65 insertions(+), 22 deletions(-)

> >  delete mode 100644 Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.txt

> >  create mode 100644 Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml

> >   

> 

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

Applied.

Thanks,

J
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.txt b/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.txt
deleted file mode 100644
index b25bf3a77e0f..000000000000
--- a/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.txt
+++ /dev/null
@@ -1,22 +0,0 @@ 
-Kionix KXSD9 Accelerometer device tree bindings
-
-Required properties:
- - compatible: 		should be set to "kionix,kxsd9"
- - reg:			i2c slave address
-
-Optional properties:
- - vdd-supply:		The input supply for VDD
- - iovdd-supply:	The input supply for IOVDD
- - interrupts:		The movement detection interrupt
- - mount-matrix:	See mount-matrix.txt
-
-Example:
-
-kxsd9@18 {
-	compatible = "kionix,kxsd9";
-	reg = <0x18>;
-	interrupt-parent = <&foo>;
-	interrupts = <57 IRQ_TYPE_EDGE_FALLING>;
-	iovdd-supply = <&bar>;
-	vdd-supply = <&baz>;
-};
diff --git a/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml b/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml
new file mode 100644
index 000000000000..d61ab4fa3d71
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/accel/kionix,kxsd9.yaml
@@ -0,0 +1,65 @@ 
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/accel/kionix,kxsd9.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Kionix KXSD9 Accelerometer
+
+maintainers:
+  - Jonathan Cameron <jic23@kernel.org>
+
+description: |
+  3 axis 12 bit accelerometer with +-8G range on all axes. Also has a
+  12 bit auxiliary ADC channel. Interface is either SPI or I2C.
+
+properties:
+  compatible:
+    const: kionix,kxsd9
+
+  reg:
+    maxItems: 1
+
+  vdd-supply: true
+  iovdd-supply: true
+
+  interrupts:
+    maxItems: 1
+
+  mount-matrix:
+    description: an optional 3x3 mounting rotation matrix.
+
+required:
+  - compatible
+  - reg
+
+examples:
+  - |
+    # include <dt-bindings/interrupt-controller/irq.h>
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        accel@18 {
+            compatible = "kionix,kxsd9";
+            reg = <0x18>;
+            iovdd-supply = <&iovdd>;
+            vdd-supply = <&vdd>;
+            interrupts = <57 IRQ_TYPE_EDGE_FALLING>;
+            mount-matrix = "-0.984807753012208", "0",  "-0.173648177666930",
+                           "0",                  "-1", "0",
+                           "-0.173648177666930", "0",  "0.984807753012208";
+        };
+    };
+  - |
+    # include <dt-bindings/interrupt-controller/irq.h>
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+        accel@0 {
+            compatible = "kionix,kxsd9";
+            reg = <0>;
+            spi-max-frequency = <10000000>;
+        };
+    };
+...