Re: [PATCH] dt-bindings: display: Convert Allwinner display pipeline to schemas

2019-12-31 Thread Rob Herring
On Thu, Dec 19, 2019 at 1:47 AM Maxime Ripard  wrote:
>
> The Allwinner SoCs have a display engine composed of several controllers
> assembled differently depending on the SoC, the number and type of output
> they have, and the additional features they provide. A number of those are
> supported in Linux, with the matching bindings.
>
> Now that we have the DT validation in place, let's split into separate file
> and convert the device tree bindings for those controllers to schemas.
>
> Signed-off-by: Maxime Ripard 
> ---
>  .../allwinner,sun4i-a10-display-backend.yaml  | 268 
>  .../allwinner,sun4i-a10-display-engine.yaml   | 114 
>  .../allwinner,sun4i-a10-display-frontend.yaml | 116 
>  .../display/allwinner,sun4i-a10-hdmi.yaml | 159 +
>  .../display/allwinner,sun4i-a10-tcon.yaml | 560 +++
>  .../allwinner,sun4i-a10-tv-encoder.yaml   |  62 ++
>  .../display/allwinner,sun6i-a31-drc.yaml  | 115 
>  .../allwinner,sun8i-a83t-de2-mixer.yaml   |  96 +++
>  .../display/allwinner,sun8i-a83t-dw-hdmi.yaml | 251 +++
>  .../allwinner,sun8i-a83t-hdmi-phy.yaml|  77 +++
>  .../display/allwinner,sun8i-r40-tcon-top.yaml | 258 +++
>  .../display/allwinner,sun9i-a80-deu.yaml  | 110 +++
>  .../bindings/display/sunxi/sun4i-drm.txt  | 637 --
>  13 files changed, 2186 insertions(+), 637 deletions(-)
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-engine.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-frontend.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun4i-a10-hdmi.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tcon.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tv-encoder.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun6i-a31-drc.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-hdmi-phy.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun8i-r40-tcon-top.yaml
>  create mode 100644 
> Documentation/devicetree/bindings/display/allwinner,sun9i-a80-deu.yaml
>  delete mode 100644 
> Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt
>
> diff --git 
> a/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
>  
> b/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
> new file mode 100644
> index ..d941c9cf07df
> --- /dev/null
> +++ 
> b/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
> @@ -0,0 +1,268 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: 
> http://devicetree.org/schemas/display/allwinner,sun4i-a10-display-backend.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Allwinner A10 Display Engine Backend Device Tree Bindings
> +
> +maintainers:
> +  - Chen-Yu Tsai 
> +  - Maxime Ripard 
> +
> +description: |
> +  The display engine backend exposes layers and sprites to the system.
> +
> +properties:
> +  compatible:
> +enum:
> +  - allwinner,sun4i-a10-display-backend
> +  - allwinner,sun5i-a13-display-backend
> +  - allwinner,sun6i-a31-display-backend
> +  - allwinner,sun7i-a20-display-backend
> +  - allwinner,sun8i-a23-display-backend
> +  - allwinner,sun8i-a33-display-backend
> +  - allwinner,sun9i-a80-display-backend
> +
> +  reg:
> +minItems: 1
> +maxItems: 2
> +items:
> +  - description: Display Backend registers
> +  - description: SAT registers
> +
> +  reg-names:
> +minItems: 1
> +maxItems: 2
> +items:
> +  - const: be
> +  - const: sat
> +
> +  interrupts:
> +maxItems: 1
> +
> +  clocks:
> +minItems: 3
> +maxItems: 4
> +items:
> +  - description: The backend interface clock
> +  - description: The backend module clock
> +  - description: The backend DRAM clock
> +  - description: The SAT clock
> +
> +  clock-names:
> +minItems: 3
> +maxItems: 4
> +items:
> +  - const: ahb
> +  - const: mod
> +  - const: ram
> +  - const: sat
> +
> +  resets:
> +minItems: 1
> +maxItems: 2
> +items:
> +  - description: The Backend reset line
> +  - description: The SAT reset line
> +
> +  reset-names:
> +minItems: 1
> +maxItems: 2
> +items:
> +  - const: be
> +  - const: sat
> +
> +  # FIXME: This should be made required eventually once every SoC will
> +  # have the 

Re: [PATCH] dt-bindings: display: Convert Allwinner display pipeline to schemas

2019-12-30 Thread Maxime Ripard
On Thu, Dec 19, 2019 at 09:47:55AM +0100, Maxime Ripard wrote:
> The Allwinner SoCs have a display engine composed of several controllers
> assembled differently depending on the SoC, the number and type of output
> they have, and the additional features they provide. A number of those are
> supported in Linux, with the matching bindings.
>
> Now that we have the DT validation in place, let's split into separate file
> and convert the device tree bindings for those controllers to schemas.
>
> Signed-off-by: Maxime Ripard 

Ping?

Maxime
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH] dt-bindings: display: Convert Allwinner display pipeline to schemas

2019-12-23 Thread Maxime Ripard
The Allwinner SoCs have a display engine composed of several controllers
assembled differently depending on the SoC, the number and type of output
they have, and the additional features they provide. A number of those are
supported in Linux, with the matching bindings.

Now that we have the DT validation in place, let's split into separate file
and convert the device tree bindings for those controllers to schemas.

Signed-off-by: Maxime Ripard 
---
 .../allwinner,sun4i-a10-display-backend.yaml  | 268 
 .../allwinner,sun4i-a10-display-engine.yaml   | 114 
 .../allwinner,sun4i-a10-display-frontend.yaml | 116 
 .../display/allwinner,sun4i-a10-hdmi.yaml | 159 +
 .../display/allwinner,sun4i-a10-tcon.yaml | 560 +++
 .../allwinner,sun4i-a10-tv-encoder.yaml   |  62 ++
 .../display/allwinner,sun6i-a31-drc.yaml  | 115 
 .../allwinner,sun8i-a83t-de2-mixer.yaml   |  96 +++
 .../display/allwinner,sun8i-a83t-dw-hdmi.yaml | 251 +++
 .../allwinner,sun8i-a83t-hdmi-phy.yaml|  77 +++
 .../display/allwinner,sun8i-r40-tcon-top.yaml | 258 +++
 .../display/allwinner,sun9i-a80-deu.yaml  | 110 +++
 .../bindings/display/sunxi/sun4i-drm.txt  | 637 --
 13 files changed, 2186 insertions(+), 637 deletions(-)
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-engine.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-frontend.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun4i-a10-hdmi.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tcon.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tv-encoder.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun6i-a31-drc.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-hdmi-phy.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun8i-r40-tcon-top.yaml
 create mode 100644 
Documentation/devicetree/bindings/display/allwinner,sun9i-a80-deu.yaml
 delete mode 100644 
Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt

diff --git 
a/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
 
b/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
new file mode 100644
index ..d941c9cf07df
--- /dev/null
+++ 
b/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-display-backend.yaml
@@ -0,0 +1,268 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: 
http://devicetree.org/schemas/display/allwinner,sun4i-a10-display-backend.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Allwinner A10 Display Engine Backend Device Tree Bindings
+
+maintainers:
+  - Chen-Yu Tsai 
+  - Maxime Ripard 
+
+description: |
+  The display engine backend exposes layers and sprites to the system.
+
+properties:
+  compatible:
+enum:
+  - allwinner,sun4i-a10-display-backend
+  - allwinner,sun5i-a13-display-backend
+  - allwinner,sun6i-a31-display-backend
+  - allwinner,sun7i-a20-display-backend
+  - allwinner,sun8i-a23-display-backend
+  - allwinner,sun8i-a33-display-backend
+  - allwinner,sun9i-a80-display-backend
+
+  reg:
+minItems: 1
+maxItems: 2
+items:
+  - description: Display Backend registers
+  - description: SAT registers
+
+  reg-names:
+minItems: 1
+maxItems: 2
+items:
+  - const: be
+  - const: sat
+
+  interrupts:
+maxItems: 1
+
+  clocks:
+minItems: 3
+maxItems: 4
+items:
+  - description: The backend interface clock
+  - description: The backend module clock
+  - description: The backend DRAM clock
+  - description: The SAT clock
+
+  clock-names:
+minItems: 3
+maxItems: 4
+items:
+  - const: ahb
+  - const: mod
+  - const: ram
+  - const: sat
+
+  resets:
+minItems: 1
+maxItems: 2
+items:
+  - description: The Backend reset line
+  - description: The SAT reset line
+
+  reset-names:
+minItems: 1
+maxItems: 2
+items:
+  - const: be
+  - const: sat
+
+  # FIXME: This should be made required eventually once every SoC will
+  # have the MBUS declared.
+  interconnects:
+maxItems: 1
+
+  # FIXME: This should be made required eventually once every SoC will
+  # have the MBUS declared.
+  interconnect-names:
+const: dma-mem
+
+  ports:
+type: object
+description: |
+  A ports node with endpoint definitions as defined in
+