dt-bindings: display: convert via,vt8500-fb binding to YAML

[PATCH] dt-bindings: display: convert via,vt8500-fb binding to YAML

Posted by Kalyani Patra 2 weeks, 6 days ago

Signed-off-by: Kalyani Patra <kalyanipatra2003@gmail.com>
---
Note: this patch is part of the GSoC 2026 Device Tree bindings conversion project.
---
 .../bindings/display/via,vt8500-fb.txt        | 36 -----------
 .../bindings/display/via,vt8500-fb.yaml       | 63 +++++++++++++++++++
 2 files changed, 63 insertions(+), 36 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/via,vt8500-fb.txt
 create mode 100644 Documentation/devicetree/bindings/display/via,vt8500-fb.yaml

diff --git a/Documentation/devicetree/bindings/display/via,vt8500-fb.txt b/Documentation/devicetree/bindings/display/via,vt8500-fb.txt
deleted file mode 100644
index 2871e218a0fb..000000000000
--- a/Documentation/devicetree/bindings/display/via,vt8500-fb.txt
+++ /dev/null
@@ -1,36 +0,0 @@
-VIA VT8500 Framebuffer
------------------------------------------------------
-
-Required properties:
-- compatible : "via,vt8500-fb"
-- reg : Should contain 1 register ranges(address and length)
-- interrupts : framebuffer controller interrupt
-- bits-per-pixel : bit depth of framebuffer (16 or 32)
-
-Required subnodes:
-- display-timings: see display-timing.txt for information
-
-Example:
-
-	fb@d8050800 {
-		compatible = "via,vt8500-fb";
-		reg = <0xd800e400 0x400>;
-		interrupts = <12>;
-		bits-per-pixel = <16>;
-
-		display-timings {
-			native-mode = <&timing0>;
-			timing0: 800x480 {
-				clock-frequency = <0>; /* unused but required */
-				hactive = <800>;
-				vactive = <480>;
-				hfront-porch = <40>;
-				hback-porch = <88>;
-				hsync-len = <0>;
-				vback-porch = <32>;
-				vfront-porch = <11>;
-				vsync-len = <1>;
-			};
-		};
-	};
-
diff --git a/Documentation/devicetree/bindings/display/via,vt8500-fb.yaml b/Documentation/devicetree/bindings/display/via,vt8500-fb.yaml
new file mode 100644
index 000000000000..1ebea3b82f30
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/via,vt8500-fb.yaml
@@ -0,0 +1,63 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/via,vt8500-fb.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: VIA VT8500 Framebuffer Controller
+
+maintainers:
+  - Kalyani Patra <kalyanipatra2003@gmail.com>
+
+description:
+  Framebuffer controller present on VIA VT8500 SoCs.
+
+properties:
+  compatible:
+    const: via,vt8500-fb
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  bits-per-pixel:
+    description: Framebuffer bit depth
+    enum: [16, 32]
+
+  display-timings:
+    $ref: /schemas/display/panel/display-timings.yaml#
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - bits-per-pixel
+  - display-timings
+
+additionalProperties: false
+
+examples:
+  - |
+    fb@d8050800 {
+        compatible = "via,vt8500-fb";
+        reg = <0xd800e400 0x400>;
+        interrupts = <12>;
+        bits-per-pixel = <16>;
+
+        display-timings {
+            native-mode = <&timing0>;
+            timing0: timing0 {
+                clock-frequency = <0>;
+                hactive = <800>;
+                vactive = <480>;
+                hfront-porch = <40>;
+                hback-porch = <88>;
+                hsync-len = <0>;
+                vback-porch = <32>;
+                vfront-porch = <11>;
+                vsync-len = <1>;
+            };
+        };
+    };
-- 
2.43.0

Re: [PATCH] dt-bindings: display: convert via,vt8500-fb binding to YAML

Posted by Krzysztof Kozlowski 2 weeks, 6 days ago

On 17/03/2026 18:29, Kalyani Patra wrote:
> Signed-off-by: Kalyani Patra <kalyanipatra2003@gmail.com>
> ---
> Note: this patch is part of the GSoC 2026 Device Tree bindings conversion project.

Please carefully follow GSoC requirements. Really, really carefully.

For example one of the required steps is to run checkpatch. You did not
fulfill that requirement.

...

> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/display/via,vt8500-fb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: VIA VT8500 Framebuffer Controller
> +
> +maintainers:
> +  - Kalyani Patra <kalyanipatra2003@gmail.com>

I have doubts you have hardware or manual, so this should be the driver
or platform maintainer.

> +
> +description:
> +  Framebuffer controller present on VIA VT8500 SoCs.
> +

Best regards,
Krzysztof

Re: [PATCH] dt-bindings: display: convert via,vt8500-fb binding to YAML

Posted by Kalyani P 2 weeks, 6 days ago

On Tue, Mar 17, 2026 at 06:38:21PM +0100, Krzysztof Kozlowski wrote:
>Please carefully follow GSoC requirements. Really, really carefully.
>
>For example one of the required steps is to run checkpatch. You did not
>fulfill that requirement.
Understood. I'll re-run the checkpatch to confirm no errors and warnings are present.

>
>I have doubts you have hardware or manual, so this should be the driver
>or platform maintainer.
I'll update the maintainers field appropriately and send a revised version. 
Thank you for looking into it. 

Warm Regards, 
Kalyani Patra