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

From: Jonathan Cameron
Date: Thu Aug 06 2020 - 13:44:48 EST


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 */
> + };
> + };
> +...