Re: [PATCH v6 1/2] dt-bindings: iio: light: add AMS AS73211 support

[Jonathan Cameron <jic23@xxxxxxxxxx>]

On Wed, 5 Aug 2020 07:57:43 +0200
Christian Eggers <ceggers@xxxxxxx> wrote:

> Add DT bindings for AMS AS73211 XYZ True Color Sensor.
> 
> Signed-off-by: Christian Eggers <ceggers@xxxxxxx>
> Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
Applied to the togreg branch of iio.git and pushed out as testing for the
autobuilders to poke at it.

Thanks,

Jonathan

> ---
>  .../bindings/iio/light/ams,as73211.yaml       | 54 +++++++++++++++++++
>  1 file changed, 54 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/light/ams,as73211.yaml
> 
> diff --git a/Documentation/devicetree/bindings/iio/light/ams,as73211.yaml b/Documentation/devicetree/bindings/iio/light/ams,as73211.yaml
> new file mode 100644
> index 000000000000..0e8cd02759b3
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/light/ams,as73211.yaml
> @@ -0,0 +1,54 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/light/ams,as73211.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: AMS AS73211 JENCOLOR(R) Digital XYZ Sensor
> +
> +maintainers:
> +  - Christian Eggers <ceggers@xxxxxxx>
> +
> +description: |
> +  XYZ True Color Sensor with I2C Interface
> +  https://ams.com/documents/20143/36005/AS73211_DS000556_3-01.pdf/a65474c0-b302-c2fd-e30a-c98df87616df
> +
> +properties:
> +  compatible:
> +    enum:
> +      - ams,as73211
> +
> +  reg:
> +    description:
> +      I2C address of the device (0x74...0x77).
> +    maxItems: 1
> +
> +  interrupts:
> +    description:
> +      Interrupt specifier for the READY interrupt generated by the device.
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        as73211@74 {
> +            compatible = "ams,as73211";
> +            reg = <0x74>;
> +            pinctrl-names = "default";
> +            pinctrl-0 = <&pinctrl_color_sensor>;
> +            interrupt-parent = <&gpio2>;
> +            interrupts = <19 IRQ_TYPE_EDGE_RISING>; /* READY */
> +        };
> +    };
> +...