Re: [PATCH v4 1/2] dt-bindings: display: convert rockchip vop bindings to yaml

2020-03-31 Thread Rob Herring
On Wed, 25 Mar 2020 11:38:27 +0100, Johan Jonker wrote:
> Current dts files with 'vop' nodes are manually verified.
> In order to automate this process rockchip-vop.txt
> has to be converted to yaml.
> 
> Signed-off-by: Johan Jonker 
> ---
> Changes v4:
>   Change description
>   Replace compatible oneOf by enum
>   Change interrupts description
>   Remove resets minItems
> 
> Changes v3:
>   Change description
> 
> Changes v2:
>   No new properties
> ---
>  .../bindings/display/rockchip/rockchip-vop.txt |  74 
>  .../bindings/display/rockchip/rockchip-vop.yaml| 124 
> +
>  2 files changed, 124 insertions(+), 74 deletions(-)
>  delete mode 100644 
> Documentation/devicetree/bindings/display/rockchip/rockchip-vop.txt
>  create mode 100644 
> Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> 

Reviewed-by: Rob Herring 
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH v4 1/2] dt-bindings: display: convert rockchip vop bindings to yaml

2020-03-25 Thread Johan Jonker
Current dts files with 'vop' nodes are manually verified.
In order to automate this process rockchip-vop.txt
has to be converted to yaml.

Signed-off-by: Johan Jonker 
---
Changes v4:
  Change description
  Replace compatible oneOf by enum
  Change interrupts description
  Remove resets minItems

Changes v3:
  Change description

Changes v2:
  No new properties
---
 .../bindings/display/rockchip/rockchip-vop.txt |  74 
 .../bindings/display/rockchip/rockchip-vop.yaml| 124 +
 2 files changed, 124 insertions(+), 74 deletions(-)
 delete mode 100644 
Documentation/devicetree/bindings/display/rockchip/rockchip-vop.txt
 create mode 100644 
Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml

diff --git 
a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.txt 
b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.txt
deleted file mode 100644
index 8b3a5f514..0
--- a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.txt
+++ /dev/null
@@ -1,74 +0,0 @@
-device-tree bindings for rockchip soc display controller (vop)
-
-VOP (Visual Output Processor) is the Display Controller for the Rockchip
-series of SoCs which transfers the image data from a video memory
-buffer to an external LCD interface.
-
-Required properties:
-- compatible: value should be one of the following
-   "rockchip,rk3036-vop";
-   "rockchip,rk3126-vop";
-   "rockchip,px30-vop-lit";
-   "rockchip,px30-vop-big";
-   "rockchip,rk3066-vop";
-   "rockchip,rk3188-vop";
-   "rockchip,rk3288-vop";
-   "rockchip,rk3368-vop";
-   "rockchip,rk3366-vop";
-   "rockchip,rk3399-vop-big";
-   "rockchip,rk3399-vop-lit";
-   "rockchip,rk3228-vop";
-   "rockchip,rk3328-vop";
-
-- reg: Must contain one entry corresponding to the base address and length
-   of the register space. Can optionally contain a second entry
-   corresponding to the CRTC gamma LUT address.
-
-- interrupts: should contain a list of all VOP IP block interrupts in the
-order: VSYNC, LCD_SYSTEM. The interrupt specifier
-format depends on the interrupt controller used.
-
-- clocks: must include clock specifiers corresponding to entries in the
-   clock-names property.
-
-- clock-names: Must contain
-   aclk_vop: for ddr buffer transfer.
-   hclk_vop: for ahb bus to R/W the phy regs.
-   dclk_vop: pixel clock.
-
-- resets: Must contain an entry for each entry in reset-names.
-  See ../reset/reset.txt for details.
-- reset-names: Must include the following entries:
-  - axi
-  - ahb
-  - dclk
-
-- iommus: required a iommu node
-
-- port: A port node with endpoint definitions as defined in
-  Documentation/devicetree/bindings/media/video-interfaces.txt.
-
-Example:
-SoC specific DT entry:
-   vopb: vopb@ff93 {
-   compatible = "rockchip,rk3288-vop";
-   reg = <0x0 0xff93 0x0 0x19c>, <0x0 0xff931000 0x0 0x1000>;
-   interrupts = ;
-   clocks = < ACLK_VOP0>, < DCLK_VOP0>, < HCLK_VOP0>;
-   clock-names = "aclk_vop", "dclk_vop", "hclk_vop";
-   resets = < SRST_LCDC1_AXI>, < SRST_LCDC1_AHB>, < 
SRST_LCDC1_DCLK>;
-   reset-names = "axi", "ahb", "dclk";
-   iommus = <_mmu>;
-   vopb_out: port {
-   #address-cells = <1>;
-   #size-cells = <0>;
-   vopb_out_edp: endpoint@0 {
-   reg = <0>;
-   remote-endpoint=<_in_vopb>;
-   };
-   vopb_out_hdmi: endpoint@1 {
-   reg = <1>;
-   remote-endpoint=<_in_vopb>;
-   };
-   };
-   };
diff --git 
a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml 
b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
new file mode 100644
index 0..bc58c5132
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
@@ -0,0 +1,124 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/rockchip/rockchip-vop.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Rockchip SoC display controller (VOP)
+
+description:
+  VOP (Video Output Processor) is the display controller for the Rockchip
+  series of SoCs which transfers the image data from a video memory
+  buffer to an external LCD interface.
+
+maintainers:
+  - Sandy Huang 
+  - Heiko Stuebner 
+
+properties:
+  compatible:
+enum:
+  - rockchip,px30-vop-big
+  - rockchip,px30-vop-lit
+  - rockchip,rk3036-vop
+  - rockchip,rk3066-vop
+  - rockchip,rk3126-vop
+  -