On 2025-04-17 17:19, Frank Li wrote:
> Convert fsl,dcu.txt to yaml format.
> 
> Additional changes:
> - remove label in example.
> - change node to display-controller in example.
> - use 32bit address in example.
> 
> Signed-off-by: Frank Li <frank...@nxp.com>

Reviewed-by: Stefan Agner <ste...@agner.ch>

> ---
>  .../devicetree/bindings/display/fsl,dcu.txt   | 34 ----------
>  .../bindings/display/fsl,ls1021a-dcu.yaml     | 68 +++++++++++++++++++
>  MAINTAINERS                                   |  2 +-
>  3 files changed, 69 insertions(+), 35 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/display/fsl,dcu.txt
>  create mode 100644 
> Documentation/devicetree/bindings/display/fsl,ls1021a-dcu.yaml
> 
> diff --git a/Documentation/devicetree/bindings/display/fsl,dcu.txt 
> b/Documentation/devicetree/bindings/display/fsl,dcu.txt
> deleted file mode 100644
> index 63ec2a624aa94..0000000000000
> --- a/Documentation/devicetree/bindings/display/fsl,dcu.txt
> +++ /dev/null
> @@ -1,34 +0,0 @@
> -Device Tree bindings for Freescale DCU DRM Driver
> -
> -Required properties:
> -- compatible:                Should be one of
> -     * "fsl,ls1021a-dcu".
> -     * "fsl,vf610-dcu".
> -
> -- reg:                       Address and length of the register set for dcu.
> -- clocks:            Handle to "dcu" and "pix" clock (in the order below)
> -                     This can be the same clock (e.g. LS1021a)
> -                     See ../clocks/clock-bindings.txt for details.
> -- clock-names:               Should be "dcu" and "pix"
> -                     See ../clocks/clock-bindings.txt for details.
> -- big-endian         Boolean property, LS1021A DCU registers are big-endian.
> -- port                       Video port for the panel output
> -
> -Optional properties:
> -- fsl,tcon:          The phandle to the timing controller node.
> -
> -Examples:
> -dcu: dcu@2ce0000 {
> -     compatible = "fsl,ls1021a-dcu";
> -     reg = <0x0 0x2ce0000 0x0 0x10000>;
> -     clocks = <&platform_clk 0>, <&platform_clk 0>;
> -     clock-names = "dcu", "pix";
> -     big-endian;
> -     fsl,tcon = <&tcon>;
> -
> -     port {
> -             dcu_out: endpoint {
> -                     remote-endpoint = <&panel_out>;
> -          };
> -     };
> -};
> diff --git a/Documentation/devicetree/bindings/display/fsl,ls1021a-dcu.yaml 
> b/Documentation/devicetree/bindings/display/fsl,ls1021a-dcu.yaml
> new file mode 100644
> index 0000000000000..7246c89271a8d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/fsl,ls1021a-dcu.yaml
> @@ -0,0 +1,68 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/display/fsl,ls1021a-dcu.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Freescale DCU DRM Driver
> +
> +maintainers:
> +  - Frank Li <frank...@nxp.com>
> +
> +properties:
> +  compatible:
> +    enum:
> +      - fsl,ls1021a-dcu
> +      - fsl,vf610-dcu
> +
> +  reg:
> +    maxItems: 1
> +
> +  clocks:
> +    maxItems: 2
> +
> +  clock-names:
> +    items:
> +      - const: dcu
> +      - const: pix
> +
> +  big-endian: true
> +
> +  port:
> +    $ref: /schemas/graph.yaml#/$defs/port-base
> +    unevaluatedProperties: false
> +    description: Video port for the panel output
> +
> +    properties:
> +      endpoint:
> +        $ref: /schemas/media/video-interfaces.yaml#
> +        unevaluatedProperties: false
> +
> +  fsl,tcon:
> +    $ref: /schemas/types.yaml#/definitions/phandle
> +    description: The phandle to the timing controller node.
> +
> +required:
> +  - compatible
> +  - reg
> +  - clocks
> +  - clock-names
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    display-controller@2ce0000 {
> +        compatible = "fsl,ls1021a-dcu";
> +        reg = <0x2ce0000 0x10000>;
> +        clocks = <&platform_clk 0>, <&platform_clk 0>;
> +        clock-names = "dcu", "pix";
> +        big-endian;
> +        fsl,tcon = <&tcon>;
> +
> +        port {
> +            endpoint {
> +                remote-endpoint = <&panel_out>;
> +            };
> +        };
> +    };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 65e0716554203..02504134e1f5d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7894,7 +7894,7 @@ M:      Alison Wang <alison.w...@nxp.com>
>  L:   dri-devel@lists.freedesktop.org
>  S:   Supported
>  T:   git https://gitlab.freedesktop.org/drm/misc/kernel.git
> -F:   Documentation/devicetree/bindings/display/fsl,dcu.txt
> +F:   Documentation/devicetree/bindings/display/fsl,ls1021a-dcu.yaml
>  F:   Documentation/devicetree/bindings/display/fsl,tcon.txt
>  F:   drivers/gpu/drm/fsl-dcu/

Reply via email to