Re: [PATCH] dt-bindings: input: touchscreen: st1232: Convert to json-schema

[Martin Kepplinger <martink@xxxxxxxxx>]

Am Freitag, dem 11.06.2021 um 15:30 +0200 schrieb Geert Uytterhoeven:
> Convert the Sitronix st1232/st1633 touchscreen controller Device Tree
> binding documentation to json-schema.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
>  .../input/touchscreen/sitronix,st1232.yaml    | 50
> +++++++++++++++++++
>  .../input/touchscreen/sitronix-st1232.txt     | 28 -----------
>  2 files changed, 50 insertions(+), 28 deletions(-)
>  create mode 100644
> Documentation/devicetree/bindings/input/touchscreen/sitronix,st1232.y
> aml
>  delete mode 100644
> Documentation/devicetree/bindings/input/touchscreen/sitronix-
> st1232.txt
> 
> diff --git
> a/Documentation/devicetree/bindings/input/touchscreen/sitronix,st1232
> .yaml
> b/Documentation/devicetree/bindings/input/touchscreen/sitronix,st1232
> .yaml
> new file mode 100644
> index 0000000000000000..1d8ca19fd37ae3fc
> --- /dev/null
> +++
> b/Documentation/devicetree/bindings/input/touchscreen/sitronix,st1232
> .yaml
> @@ -0,0 +1,50 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id:
> http://devicetree.org/schemas/input/touchscreen/sitronix,st1232.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Sitronix st1232 or st1633 touchscreen controller
> +
> +maintainers:
> +  - Bastian Hecht <hechtb@xxxxxxxxx>
> +
> +allOf:
> +  - $ref: touchscreen.yaml#
> +
> +properties:
> +  compatible:
> +    enum:
> +      - sitronix,st1232
> +      - sitronix,st1633
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  gpios:
> +    description: A phandle to the reset GPIO
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    i2c {
> +            #address-cells = <1>;
> +            #size-cells = <0>;
> +
> +            touchscreen@55 {
> +                    compatible = "sitronix,st1232";
> +                    reg = <0x55>;
> +                    interrupts = <2 0>;
> +                    gpios = <&gpio1 166 0>;
> +            };
> +    };
> diff --git
> a/Documentation/devicetree/bindings/input/touchscreen/sitronix-
> st1232.txt
> b/Documentation/devicetree/bindings/input/touchscreen/sitronix-
> st1232.txt
> deleted file mode 100644
> index 019373253b28c08c..0000000000000000
> --- a/Documentation/devicetree/bindings/input/touchscreen/sitronix-
> st1232.txt
> +++ /dev/null
> @@ -1,28 +0,0 @@
> -* Sitronix st1232 or st1633 touchscreen controller
> -
> -Required properties:
> -- compatible: must contain one of
> -  * "sitronix,st1232"
> -  * "sitronix,st1633"
> -- reg: I2C address of the chip
> -- interrupts: interrupt to which the chip is connected
> -
> -Optional properties:
> -- gpios: a phandle to the reset GPIO
> -
> -For additional optional properties see: touchscreen.txt
> -
> -Example:
> -
> -       i2c@00000000 {
> -               /* ... */
> -
> -               touchscreen@55 {
> -                       compatible = "sitronix,st1232";
> -                       reg = <0x55>;
> -                       interrupts = <2 0>;
> -                       gpios = <&gpio1 166 0>;
> -               };
> -
> -               /* ... */
> -       };

Acked-by: Martin Kepplinger <martink@xxxxxxxxx>

thank you!