[PATCH] dt-bindings: iio: isl29018: convert bindings to YAML format

[PATCH] dt-bindings: iio: isl29018: convert bindings to YAML format

[Brian Masney]

Convert the isl29018 device tree bindings to the new YAML format.

Signed-off-by: Brian Masney <masneyb@onstation.org>
---
I'm willing to be listed as the maintainer since this is one of the
drivers that I moved out out of staging unless one of the original
authors wants to be listed instead.

I added the BSD-2-Clause to the new binding document. I wrote the
original document and the only commit since then was from Rob in
commit 791d3ef2e111 ("dt-bindings: remove 'interrupt-parent' from
bindings")

 .../bindings/iio/light/isl29018.txt           | 27 ---------
 .../bindings/iio/light/isl29018.yaml          | 56 +++++++++++++++++++
 2 files changed, 56 insertions(+), 27 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/iio/light/isl29018.txt
 create mode 100644 Documentation/devicetree/bindings/iio/light/isl29018.yaml

diff --git a/Documentation/devicetree/bindings/iio/light/isl29018.txt b/Documentation/devicetree/bindings/iio/light/isl29018.txt
deleted file mode 100644
index b9bbde3e13ed..000000000000
--- a/Documentation/devicetree/bindings/iio/light/isl29018.txt
+++ /dev/null
@@ -1,27 +0,0 @@
-* ISL 29018/29023/29035 I2C ALS, Proximity, and Infrared sensor
-
-Required properties:
-
-  - compatible: Should be one of
-		"isil,isl29018"
-		"isil,isl29023"
-		"isil,isl29035"
-  - reg: the I2C address of the device
-
-Optional properties:
-
-  - interrupts: the sole interrupt generated by the device
-
-  Refer to interrupt-controller/interrupts.txt for generic interrupt client
-  node bindings.
-
-  - vcc-supply: phandle to the regulator that provides power to the sensor.
-
-Example:
-
-isl29018@44 {
-	compatible = "isil,isl29018";
-	reg = <0x44>;
-	interrupt-parent = <&gpio>;
-	interrupts = <TEGRA_GPIO(Z, 2) IRQ_TYPE_LEVEL_HIGH>;
-};
diff --git a/Documentation/devicetree/bindings/iio/light/isl29018.yaml b/Documentation/devicetree/bindings/iio/light/isl29018.yaml
new file mode 100644
index 000000000000..cbb00be8f359
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/light/isl29018.yaml
@@ -0,0 +1,56 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/light/isl29018.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: |
+  Intersil 29018/29023/29035 Ambient Light, Infrared Light, and Proximity Sensor
+
+maintainers:
+  - Brian Masney <masneyb@onstation.org>
+
+description: |
+  Ambient and infrared light sensing with proximity detection over an i2c
+  interface.
+
+  https://www.renesas.com/us/en/www/doc/datasheet/isl29018.pdf
+  https://www.renesas.com/us/en/www/doc/datasheet/isl29023.pdf
+  https://www.renesas.com/us/en/www/doc/datasheet/isl29035.pdf
+
+properties:
+  compatible:
+    enum:
+      - isil,isl29018
+      - isil,isl29023
+      - isil,isl29035
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  vcc-supply:
+    description: Regulator that provides power to the sensor
+
+required:
+  - compatible
+  - reg
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+
+    i2c {
+
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        sensor@44 {
+                compatible = "isil,isl29018";
+                reg = <0x44>;
+                interrupts-extended = <&msmgpio 61 IRQ_TYPE_LEVEL_HIGH>;
+        };
+    };
+...
-- 
2.20.1

Re: [PATCH] dt-bindings: iio: isl29018: convert bindings to YAML format

[Rob Herring]

On Sat, Apr 27, 2019 at 1:25 PM Brian Masney <masneyb@onstation.org> wrote:
>
> Convert the isl29018 device tree bindings to the new YAML format.
>
> Signed-off-by: Brian Masney <masneyb@onstation.org>
> ---
> I'm willing to be listed as the maintainer since this is one of the
> drivers that I moved out out of staging unless one of the original
> authors wants to be listed instead.
>
> I added the BSD-2-Clause to the new binding document. I wrote the
> original document and the only commit since then was from Rob in
> commit 791d3ef2e111 ("dt-bindings: remove 'interrupt-parent' from
> bindings")
>
>  .../bindings/iio/light/isl29018.txt           | 27 ---------
>  .../bindings/iio/light/isl29018.yaml          | 56 +++++++++++++++++++
>  2 files changed, 56 insertions(+), 27 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/light/isl29018.txt
>  create mode 100644 Documentation/devicetree/bindings/iio/light/isl29018.yaml

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

Re: [PATCH] dt-bindings: iio: isl29018: convert bindings to YAML format

[Jonathan Cameron]

On Mon, 29 Apr 2019 07:44:14 -0500
Rob Herring <robh+dt@kernel.org> wrote:

> On Sat, Apr 27, 2019 at 1:25 PM Brian Masney <masneyb@onstation.org> wrote:
> >
> > Convert the isl29018 device tree bindings to the new YAML format.
> >
> > Signed-off-by: Brian Masney <masneyb@onstation.org>
> > ---
> > I'm willing to be listed as the maintainer since this is one of the
> > drivers that I moved out out of staging unless one of the original
> > authors wants to be listed instead.
> >
> > I added the BSD-2-Clause to the new binding document. I wrote the
> > original document and the only commit since then was from Rob in
> > commit 791d3ef2e111 ("dt-bindings: remove 'interrupt-parent' from
> > bindings")
> >
> >  .../bindings/iio/light/isl29018.txt           | 27 ---------
> >  .../bindings/iio/light/isl29018.yaml          | 56 +++++++++++++++++++
> >  2 files changed, 56 insertions(+), 27 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/iio/light/isl29018.txt
> >  create mode 100644 Documentation/devicetree/bindings/iio/light/isl29018.yaml  
> 
> Reviewed-by: Rob Herring <robh@kernel.org>
Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to play with it.

Thanks,

Jonathan