Re: [PATCH v2 1/3] dt-bindings: display: convert faraday,tve200

2021-05-21 Thread Linus Walleij
On Wed, May 19, 2021 at 10:35 PM Corentin Labbe  wrote:

> Converts display/faraday,tve200.txt to yaml.
>
> Signed-off-by: Corentin Labbe 

Patch applied to the DRM misc tree.

Yours,
Linus Walleij


Re: [PATCH v2 1/3] dt-bindings: display: convert faraday,tve200

2021-05-20 Thread Rob Herring
On Wed, 19 May 2021 20:35:45 +, Corentin Labbe wrote:
> Converts display/faraday,tve200.txt to yaml.
> 
> Signed-off-by: Corentin Labbe 
> ---
> Changes since v1:
> - added two subsequent patchs fixing issue found when converting
> - fixed all issues reported by Rob Herring
>  .../bindings/display/faraday,tve200.txt   | 54 ---
>  .../bindings/display/faraday,tve200.yaml  | 68 +++
>  2 files changed, 68 insertions(+), 54 deletions(-)
>  delete mode 100644 
> Documentation/devicetree/bindings/display/faraday,tve200.txt
>  create mode 100644 
> Documentation/devicetree/bindings/display/faraday,tve200.yaml
> 

Reviewed-by: Rob Herring 


[PATCH v2 1/3] dt-bindings: display: convert faraday,tve200

2021-05-19 Thread Corentin Labbe
Converts display/faraday,tve200.txt to yaml.

Signed-off-by: Corentin Labbe 
---
Changes since v1:
- added two subsequent patchs fixing issue found when converting
- fixed all issues reported by Rob Herring
 .../bindings/display/faraday,tve200.txt   | 54 ---
 .../bindings/display/faraday,tve200.yaml  | 68 +++
 2 files changed, 68 insertions(+), 54 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/faraday,tve200.txt
 create mode 100644 
Documentation/devicetree/bindings/display/faraday,tve200.yaml

diff --git a/Documentation/devicetree/bindings/display/faraday,tve200.txt 
b/Documentation/devicetree/bindings/display/faraday,tve200.txt
deleted file mode 100644
index 82e3bc0b7485..
--- a/Documentation/devicetree/bindings/display/faraday,tve200.txt
+++ /dev/null
@@ -1,54 +0,0 @@
-* Faraday TV Encoder TVE200
-
-Required properties:
-
-- compatible: must be one of:
-   "faraday,tve200"
-   "cortina,gemini-tvc", "faraday,tve200"
-
-- reg: base address and size of the control registers block
-
-- interrupts: contains an interrupt specifier for the interrupt
-   line from the TVE200
-
-- clock-names: should contain "PCLK" for the clock line clocking the
-   silicon and "TVE" for the 27MHz clock to the video driver
-
-- clocks: contains phandle and clock specifier pairs for the entries
-   in the clock-names property. See
-   Documentation/devicetree/bindings/clock/clock-bindings.txt
-
-Optional properties:
-
-- resets: contains the reset line phandle for the block
-
-Required sub-nodes:
-
-- port: describes LCD panel signals, following the common binding
-   for video transmitter interfaces; see
-   Documentation/devicetree/bindings/media/video-interfaces.txt
-   This port should have the properties:
-   reg = <0>;
-   It should have one endpoint connected to a remote endpoint where
-   the display is connected.
-
-Example:
-
-display-controller@6a00 {
-   #address-cells = <1>;
-   #size-cells = <0>;
-   compatible = "faraday,tve200";
-   reg = <0x6a00 0x1000>;
-   interrupts = <13 IRQ_TYPE_EDGE_RISING>;
-   resets = < GEMINI_RESET_TVC>;
-   clocks = < GEMINI_CLK_GATE_TVC>,
-< GEMINI_CLK_TVC>;
-   clock-names = "PCLK", "TVE";
-
-   port@0 {
-   reg = <0>;
-   display_out: endpoint {
-   remote-endpoint = <_in>;
-   };
-   };
-};
diff --git a/Documentation/devicetree/bindings/display/faraday,tve200.yaml 
b/Documentation/devicetree/bindings/display/faraday,tve200.yaml
new file mode 100644
index ..e2ee77767321
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/faraday,tve200.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/faraday,tve200.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Faraday TV Encoder TVE200
+
+maintainers:
+  - Linus Walleij 
+
+properties:
+  compatible:
+oneOf:
+  - const: faraday,tve200
+  - items:
+  - const: cortina,gemini-tvc
+  - const: faraday,tve200
+
+  reg:
+maxItems: 1
+
+  interrupts:
+minItems: 1
+
+  clock-names:
+items:
+  - const: PCLK
+  - const: TVE
+
+  clocks:
+minItems: 2
+
+  resets:
+minItems: 1
+
+  port:
+$ref: /schemas/graph.yaml#/properties/port
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - clock-names
+  - clocks
+
+additionalProperties: false
+
+examples:
+  - |
+#include 
+#include 
+#include 
+display-controller@6a00 {
+  compatible = "faraday,tve200";
+  reg = <0x6a00 0x1000>;
+  interrupts = <13 IRQ_TYPE_EDGE_RISING>;
+  resets = < GEMINI_RESET_TVC>;
+  clocks = < GEMINI_CLK_GATE_TVC>,
+   < GEMINI_CLK_TVC>;
+  clock-names = "PCLK", "TVE";
+
+  port {
+display_out: endpoint {
+  remote-endpoint = <_in>;
+};
+  };
+};
-- 
2.26.3