[PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML

[PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML

[Jagan Teki]

These panel bindings are owned by me, so updated all of them into
YAML DT schema.

Any inputs?
Jagan.

Jagan Teki (6):
  dt-bindings: display: panel: Convert feiyang,fy07024di26a30d to DT
    schema
  dt-bindings: display: panel: Convert sitronix,st7701 to DT schema
  MAINTAINERS: Update feiyang, st7701 panel bindings converted as YAML
  dt-bindings: display: panel: Convert friendlyarm,hd702e to DT schema
  dt-bindings: display: panel: Convert rocktech,rk070er9427 to DT schema
  dt-bindings: display: panel: Convert koe,tx31d200vm0baa to DT schema

 .../display/panel/feiyang,fy07024di26a30d.txt | 20 ------
 .../panel/feiyang,fy07024di26a30d.yaml        | 50 +++++++++++++++
 .../display/panel/friendlyarm,hd702e.txt      | 32 ----------
 .../display/panel/friendlyarm,hd702e.yaml     | 47 ++++++++++++++
 .../display/panel/koe,tx31d200vm0baa.txt      | 25 --------
 .../display/panel/koe,tx31d200vm0baa.yaml     | 37 +++++++++++
 .../display/panel/rocktech,rk070er9427.txt    | 25 --------
 .../display/panel/rocktech,rk070er9427.yaml   | 37 +++++++++++
 .../display/panel/sitronix,st7701.txt         | 30 ---------
 .../display/panel/sitronix,st7701.yaml        | 61 +++++++++++++++++++
 MAINTAINERS                                   |  4 +-
 11 files changed, 234 insertions(+), 134 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/sitronix,st7701.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml

-- 
2.18.0.321.gffc6fa0e3

[PATCH 1/6] dt-bindings: display: panel: Convert feiyang,fy07024di26a30d to DT schema

[Jagan Teki]

Convert the feiyang,fy07024di26a30d panel bindings to DT schema.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 .../display/panel/feiyang,fy07024di26a30d.txt | 20 --------
 .../panel/feiyang,fy07024di26a30d.yaml        | 50 +++++++++++++++++++
 2 files changed, 50 insertions(+), 20 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.txt b/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.txt
deleted file mode 100644
index 82caa7b65ae8..000000000000
--- a/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.txt
+++ /dev/null
@@ -1,20 +0,0 @@
-Feiyang FY07024DI26A30-D 7" MIPI-DSI LCD Panel
-
-Required properties:
-- compatible: must be "feiyang,fy07024di26a30d"
-- reg: DSI virtual channel used by that screen
-- avdd-supply: analog regulator dc1 switch
-- dvdd-supply: 3v3 digital regulator
-- reset-gpios: a GPIO phandle for the reset pin
-
-Optional properties:
-- backlight: phandle for the backlight control.
-
-panel@0 {
-	compatible = "feiyang,fy07024di26a30d";
-	reg = <0>;
-	avdd-supply = <&reg_dc1sw>;
-	dvdd-supply = <&reg_dldo2>;
-	reset-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* LCD-RST: PD24 */
-	backlight = <&backlight>;
-};
diff --git a/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml b/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
new file mode 100644
index 000000000000..16e2fedda49e
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
@@ -0,0 +1,50 @@
+# SPDX-License-Identifier: (GPL-2.0+ OR X11)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/panel/feiyang,fy07024di26a30d.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Feiyang FY07024DI26A30-D 7" MIPI-DSI LCD Panel
+
+maintainers:
+  - Jagan Teki <jagan@amarulasolutions.com>
+
+properties:
+  compatible:
+    const: feiyang,fy07024di26a30d
+
+  reg:
+    description: DSI virtual channel used by that screen
+
+  avdd-supply:
+    description: analog regulator dc1 switch
+
+  dvdd-supply:
+    description: 3v3 digital regulator
+
+  reset-gpios:
+    description: a GPIO phandle for the reset pin
+
+  backlight:
+    description: Backlight used by the panel
+    $ref: "/schemas/types.yaml#/definitions/phandle"
+
+required:
+  - compatible
+  - reg
+  - avdd-supply
+  - dvdd-supply
+  - reset-gpios
+
+additionalProperties: false
+
+examples:
+  - |
+    panel@0 {
+            compatible = "feiyang,fy07024di26a30d";
+            reg = <0>;
+            avdd-supply = <&reg_dc1sw>;
+            dvdd-supply = <&reg_dldo2>;
+            reset-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* LCD-RST: PD24 */
+            backlight = <&backlight>;
+    };
-- 
2.18.0.321.gffc6fa0e3

[PATCH 2/6] dt-bindings: display: panel: Convert sitronix,st7701 to DT schema

[Jagan Teki]

Convert the sitronix,st7701 panel bindings to DT schema.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 .../display/panel/sitronix,st7701.txt         | 30 ---------
 .../display/panel/sitronix,st7701.yaml        | 61 +++++++++++++++++++
 2 files changed, 61 insertions(+), 30 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/panel/sitronix,st7701.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/sitronix,st7701.txt b/Documentation/devicetree/bindings/display/panel/sitronix,st7701.txt
deleted file mode 100644
index ccd17597f1f6..000000000000
--- a/Documentation/devicetree/bindings/display/panel/sitronix,st7701.txt
+++ /dev/null
@@ -1,30 +0,0 @@
-Sitronix ST7701 based LCD panels
-
-ST7701 designed for small and medium sizes of TFT LCD display, is
-capable of supporting up to 480RGBX864 in resolution. It provides
-several system interfaces like MIPI/RGB/SPI.
-
-Techstar TS8550B is 480x854, 2-lane MIPI DSI LCD panel which has
-inbuilt ST7701 chip.
-
-Required properties:
-- compatible: must be "sitronix,st7701" and one of
-  * "techstar,ts8550b"
-- reset-gpios: a GPIO phandle for the reset pin
-
-Required properties for techstar,ts8550b:
-- reg: DSI virtual channel used by that screen
-- VCC-supply: analog regulator for MIPI circuit
-- IOVCC-supply: I/O system regulator
-
-Optional properties:
-- backlight: phandle for the backlight control.
-
-panel@0 {
-	compatible = "techstar,ts8550b", "sitronix,st7701";
-	reg = <0>;
-	VCC-supply = <&reg_dldo2>;
-	IOVCC-supply = <&reg_dldo2>;
-	reset-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* LCD-RST: PD24 */
-	backlight = <&backlight>;
-};
diff --git a/Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml b/Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml
new file mode 100644
index 000000000000..49e4b54ecf7c
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml
@@ -0,0 +1,61 @@
+# SPDX-License-Identifier: (GPL-2.0+ OR X11)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/panel/sitronix,st7701.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Sitronix ST7701 based LCD panels
+
+maintainers:
+  - Jagan Teki <jagan@amarulasolutions.com>
+
+description: |
+  ST7701 designed for small and medium sizes of TFT LCD display, is
+  capable of supporting up to 480RGBX864 in resolution. It provides
+  several system interfaces like MIPI/RGB/SPI.
+
+  Techstar TS8550B is 480x854, 2-lane MIPI DSI LCD panel which has
+  inbuilt ST7701 chip.
+
+properties:
+  compatible:
+    oneOf:
+      - const: sitronix,st7701
+      - items:
+          - const: techstar,ts8550b
+
+  reg:
+    description: DSI virtual channel used by that screen
+
+  VCC-supply:
+    description: analog regulator for MIPI circuit
+
+  IOVCC-supply:
+    description: I/O system regulator
+
+  reset-gpios:
+    description: a GPIO phandle for the reset pin
+
+  backlight:
+    description: Backlight used by the panel
+    $ref: "/schemas/types.yaml#/definitions/phandle"
+
+required:
+  - compatible
+  - reg
+  - VCC-supply
+  - IOVCC-supply
+  - reset-gpios
+
+additionalProperties: false
+
+examples:
+  - |
+    panel@0 {
+            compatible = "techstar,ts8550b", "sitronix,st7701";
+            reg = <0>;
+            VCC-supply = <&reg_dldo2>;
+            IOVCC-supply = <&reg_dldo2>;
+            reset-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* LCD-RST: PD24 */
+            backlight = <&backlight>;
+    };
-- 
2.18.0.321.gffc6fa0e3

[PATCH 3/6] MAINTAINERS: Update feiyang, st7701 panel bindings converted as YAML

[Jagan Teki]

The feiyang,fy07024di26a30d.txt and sitronix,st7701.txt has been
converted to YAML schemas, update MAINTAINERS to match them again.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 MAINTAINERS | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8b2e6c68832d..4d6e4febd170 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5169,7 +5169,7 @@ DRM DRIVER FOR FEIYANG FY07024DI26A30-D MIPI-DSI LCD PANELS
 M:	Jagan Teki <jagan@amarulasolutions.com>
 S:	Maintained
 F:	drivers/gpu/drm/panel/panel-feiyang-fy07024di26a30d.c
-F:	Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.txt
+F:	Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
 
 DRM DRIVER FOR GRAIN MEDIA GM12U320 PROJECTORS
 M:	Hans de Goede <hdegoede@redhat.com>
@@ -5298,7 +5298,7 @@ DRM DRIVER FOR SITRONIX ST7701 PANELS
 M:	Jagan Teki <jagan@amarulasolutions.com>
 S:	Maintained
 F:	drivers/gpu/drm/panel/panel-sitronix-st7701.c
-F:	Documentation/devicetree/bindings/display/panel/sitronix,st7701.txt
+F:	Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml
 
 DRM DRIVER FOR SITRONIX ST7586 PANELS
 M:	David Lechner <david@lechnology.com>
-- 
2.18.0.321.gffc6fa0e3

[PATCH 4/6] dt-bindings: display: panel: Convert friendlyarm,hd702e to DT schema

[Jagan Teki]

Convert the friendlyarm,hd702e panel bindings to DT schema.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 .../display/panel/friendlyarm,hd702e.txt      | 32 -------------
 .../display/panel/friendlyarm,hd702e.yaml     | 47 +++++++++++++++++++
 2 files changed, 47 insertions(+), 32 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.txt b/Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.txt
deleted file mode 100644
index 6c9156fc3478..000000000000
--- a/Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.txt
+++ /dev/null
@@ -1,32 +0,0 @@
-FriendlyELEC HD702E 800x1280 LCD panel
-
-HD702E lcd is FriendlyELEC developed eDP LCD panel with 800x1280
-resolution. It has built in Goodix, GT9271 captive touchscreen
-with backlight adjustable via PWM.
-
-Required properties:
-- compatible: should be "friendlyarm,hd702e"
-- power-supply: regulator to provide the supply voltage
-
-Optional properties:
-- backlight: phandle of the backlight device attached to the panel
-
-Optional nodes:
-- Video port for LCD panel input.
-
-This binding is compatible with the simple-panel binding, which is specified
-in simple-panel.txt in this directory.
-
-Example:
-
-	panel {
-		compatible ="friendlyarm,hd702e", "simple-panel";
-		backlight = <&backlight>;
-		power-supply = <&vcc3v3_sys>;
-
-		port {
-			panel_in_edp: endpoint {
-				remote-endpoint = <&edp_out_panel>;
-			};
-		};
-	};
diff --git a/Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.yaml b/Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.yaml
new file mode 100644
index 000000000000..0509c34d98b2
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.yaml
@@ -0,0 +1,47 @@
+# SPDX-License-Identifier: (GPL-2.0+ OR X11)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/panel/friendlyarm,hd702e.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: FriendlyELEC HD702E 800x1280 LCD panel
+
+maintainers:
+  - Jagan Teki <jagan@amarulasolutions.com>
+  - Thierry Reding <thierry.reding@gmail.com>
+  - Sam Ravnborg <sam@ravnborg.org>
+
+description: |
+  HD702E lcd is FriendlyELEC developed eDP LCD panel with 800x1280
+  resolution. It has built in Goodix, GT9271 captive touchscreen
+  with backlight adjustable via PWM.
+
+allOf:
+  - $ref: panel-common.yaml#
+
+properties:
+  compatible:
+    const: friendlyarm,hd702e
+
+  power-supply:
+    description: regulator to provide the supply voltage
+
+required:
+  - compatible
+  - power-supply
+
+additionalProperties: false
+
+examples:
+  - |
+    panel {
+            compatible ="friendlyarm,hd702e", "simple-panel";
+            backlight = <&backlight>;
+            power-supply = <&vcc3v3_sys>;
+
+            port {
+                    panel_in_edp: endpoint {
+                            remote-endpoint = <&edp_out_panel>;
+                    };
+            };
+    };
-- 
2.18.0.321.gffc6fa0e3

[PATCH 5/6] dt-bindings: display: panel: Convert rocktech,rk070er9427 to DT schema

[Jagan Teki]

Convert the rocktech,rk070er9427 panel bindings to DT schema.

Also, drop the description from legacy .txt since the yaml DT schema
of panel-common.yaml already have that information.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 .../display/panel/rocktech,rk070er9427.txt    | 25 -------------
 .../display/panel/rocktech,rk070er9427.yaml   | 37 +++++++++++++++++++
 2 files changed, 37 insertions(+), 25 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.txt b/Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.txt
deleted file mode 100644
index eb1fb9f8d1f4..000000000000
--- a/Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.txt
+++ /dev/null
@@ -1,25 +0,0 @@
-Rocktech Display Ltd. RK070ER9427 800(RGB)x480 TFT LCD panel
-
-This binding is compatible with the simple-panel binding, which is specified
-in simple-panel.txt in this directory.
-
-Required properties:
-- compatible: should be "rocktech,rk070er9427"
-
-Optional properties:
-- backlight: phandle of the backlight device attached to the panel
-
-Optional nodes:
-- Video port for LCD panel input.
-
-Example:
-	panel {
-		compatible = "rocktech,rk070er9427";
-		backlight = <&backlight_lcd>;
-
-		port {
-			lcd_panel_in: endpoint {
-				remote-endpoint = <&lcd_display_out>;
-			};
-		};
-	};
diff --git a/Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.yaml b/Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.yaml
new file mode 100644
index 000000000000..ac5aebfa4adc
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.yaml
@@ -0,0 +1,37 @@
+# SPDX-License-Identifier: (GPL-2.0+ OR X11)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/panel/rocktech,rk070er9427.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Rocktech Display Ltd. RK070ER9427 800(RGB)x480 TFT LCD panel
+
+maintainers:
+  - Jagan Teki <jagan@amarulasolutions.com>
+  - Thierry Reding <thierry.reding@gmail.com>
+  - Sam Ravnborg <sam@ravnborg.org>
+
+allOf:
+  - $ref: panel-common.yaml#
+
+properties:
+  compatible:
+    const: rocktech,rk070er9427
+
+required:
+  - compatible
+
+additionalProperties: false
+
+examples:
+  - |
+    panel {
+            compatible = "rocktech,rk070er9427";
+            backlight = <&backlight_lcd>;
+
+            port {
+                    lcd_panel_in: endpoint {
+                            remote-endpoint = <&lcd_display_out>;
+                    };
+            };
+    };
-- 
2.18.0.321.gffc6fa0e3

[PATCH 6/6] dt-bindings: display: panel: Convert koe,tx31d200vm0baa to DT schema

[Jagan Teki]

Convert the koe,tx31d200vm0baa panel bindings to DT schema.

Also, drop the description from legacy .txt since the yaml DT schema
of panel-common.yaml already have that information.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 .../display/panel/koe,tx31d200vm0baa.txt      | 25 -------------
 .../display/panel/koe,tx31d200vm0baa.yaml     | 37 +++++++++++++++++++
 2 files changed, 37 insertions(+), 25 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.txt b/Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.txt
deleted file mode 100644
index 6a036ede3e28..000000000000
--- a/Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.txt
+++ /dev/null
@@ -1,25 +0,0 @@
-Kaohsiung Opto-Electronics. TX31D200VM0BAA 12.3" HSXGA LVDS panel
-
-This binding is compatible with the simple-panel binding, which is specified
-in simple-panel.txt in this directory.
-
-Required properties:
-- compatible: should be "koe,tx31d200vm0baa"
-
-Optional properties:
-- backlight: phandle of the backlight device attached to the panel
-
-Optional nodes:
-- Video port for LVDS panel input.
-
-Example:
-	panel {
-		compatible = "koe,tx31d200vm0baa";
-		backlight = <&backlight_lvds>;
-
-		port {
-			panel_in: endpoint {
-				remote-endpoint = <&lvds0_out>;
-			};
-		};
-	};
diff --git a/Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.yaml b/Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.yaml
new file mode 100644
index 000000000000..4b64e8ad8ec5
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.yaml
@@ -0,0 +1,37 @@
+# SPDX-License-Identifier: (GPL-2.0+ OR X11)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/panel/koe,tx31d200vm0baa.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Kaohsiung Opto-Electronics. TX31D200VM0BAA 12.3" HSXGA LVDS panel
+
+maintainers:
+  - Jagan Teki <jagan@amarulasolutions.com>
+  - Thierry Reding <thierry.reding@gmail.com>
+  - Sam Ravnborg <sam@ravnborg.org>
+
+allOf:
+  - $ref: panel-common.yaml#
+
+properties:
+  compatible:
+    const: koe,tx31d200vm0baa
+
+required:
+  - compatible
+
+additionalProperties: false
+
+examples:
+  - |
+    panel {
+            compatible = "koe,tx31d200vm0baa";
+            backlight = <&backlight_lcd>;
+
+            port {
+                    panel_in: endpoint {
+                            remote-endpoint = <&lvds0_out>;
+                    };
+            };
+    };
-- 
2.18.0.321.gffc6fa0e3

Re: [PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML

[Sam Ravnborg]

Hi Jagan.
On Wed, Jan 01, 2020 at 04:54:38PM +0530, Jagan Teki wrote:
> These panel bindings are owned by me, so updated all of them into
> YAML DT schema.
> 
> Any inputs?
Thanks for doing the conversion.

dt_binding_check was not happy:
Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
  DTC     Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
  CHECK   Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: Additional items are not allowed ('simple-panel' was unexpected)
Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: ['friendlyarm,hd702e', 'simple-panel'] is too long
  DTC     Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dt.yaml
Error: Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dts:22.42-43 syntax error
FATAL ERROR: Unable to parse input tree

Please fix and check the bindings using dt_binding_check before
resubmit.

I had to install libyaml-dev (as least I recall this was the name)
before dt_binding_check worked OK for me.

	Sam

Re: [PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML

[Jagan Teki]

Hi Sam,

On Sat, Jan 4, 2020 at 8:47 PM Sam Ravnborg <sam@ravnborg.org> wrote:
>
> Hi Jagan.
> On Wed, Jan 01, 2020 at 04:54:38PM +0530, Jagan Teki wrote:
> > These panel bindings are owned by me, so updated all of them into
> > YAML DT schema.
> >
> > Any inputs?
> Thanks for doing the conversion.
>
> dt_binding_check was not happy:
> Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
>   DTC     Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
>   CHECK   Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
> Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
> Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: Additional items are not allowed ('simple-panel' was unexpected)
> Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: ['friendlyarm,hd702e', 'simple-panel'] is too long
>   DTC     Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dt.yaml
> Error: Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dts:22.42-43 syntax error
> FATAL ERROR: Unable to parse input tree
>
> Please fix and check the bindings using dt_binding_check before
> resubmit.
>
> I had to install libyaml-dev (as least I recall this was the name)
> before dt_binding_check worked OK for me.

I did check dt_binfing_check with this series. Here is the complete
build log and you can see the panels related to this series are
checked fine. Let me know if I miss anything here?

₹ make dt_binding_check
arch/x86/Makefile:147: CONFIG_X86_X32 enabled but no binutils support
  SCHEMA  Documentation/devicetree/bindings/processed-schema.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml:
ignoring, error in schema: properties: reg: minItems
warning: no schema found in file:
Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml:
ignoring, error in schema: properties: ppi-partitions:
patternProperties: ^interrupt-partition-[0-9]+$
warning: no schema found in file:
Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/net/snps,dwmac.yaml:
ignoring, error in schema: allOf: 2: then
warning: no schema found in file:
Documentation/devicetree/bindings/net/snps,dwmac.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml:
ignoring, error in schema: properties: compatible: enum: 0
warning: no schema found in file:
Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/riscv/cpus.yaml:
ignoring, error in schema: properties: timebase-frequency: type
warning: no schema found in file:
Documentation/devicetree/bindings/riscv/cpus.yaml
  CHKDT   Documentation/devicetree/bindings/arm/pmu.yaml
  CHKDT   Documentation/devicetree/bindings/arm/rockchip.yaml
  CHKDT   Documentation/devicetree/bindings/arm/sirf.yaml
  CHKDT   Documentation/devicetree/bindings/arm/vt8500.yaml
  CHKDT   Documentation/devicetree/bindings/arm/calxeda.yaml
  CHKDT   Documentation/devicetree/bindings/arm/atmel-at91.yaml
  CHKDT   Documentation/devicetree/bindings/arm/sunxi.yaml
  CHKDT   Documentation/devicetree/bindings/arm/xilinx.yaml
  CHKDT   Documentation/devicetree/bindings/arm/moxart.yaml
  CHKDT   Documentation/devicetree/bindings/arm/stm32/stm32.yaml
  CHKDT   Documentation/devicetree/bindings/arm/spear.yaml
  CHKDT   Documentation/devicetree/bindings/arm/al,alpine.yaml
  CHKDT   Documentation/devicetree/bindings/arm/digicolor.yaml
  CHKDT   Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
  CHKDT   Documentation/devicetree/bindings/arm/altera.yaml
  CHKDT   Documentation/devicetree/bindings/arm/bitmain.yaml
  CHKDT   Documentation/devicetree/bindings/arm/nxp/lpc32xx.yaml
  CHKDT   Documentation/devicetree/bindings/arm/fsl.yaml
  CHKDT   Documentation/devicetree/bindings/arm/axxia.yaml
  CHKDT   Documentation/devicetree/bindings/arm/psci.yaml
  CHKDT   Documentation/devicetree/bindings/arm/qcom.yaml
  CHKDT   Documentation/devicetree/bindings/arm/zte.yaml
  CHKDT   Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
  CHKDT   Documentation/devicetree/bindings/arm/mediatek.yaml
  CHKDT   Documentation/devicetree/bindings/arm/socionext/milbeaut.yaml
  CHKDT   Documentation/devicetree/bindings/arm/ti/nspire.yaml
  CHKDT   Documentation/devicetree/bindings/arm/ti/ti,davinci.yaml
  CHKDT   Documentation/devicetree/bindings/arm/realtek.yaml
  CHKDT   Documentation/devicetree/bindings/arm/rda.yaml
  CHKDT   Documentation/devicetree/bindings/arm/actions.yaml
  CHKDT   Documentation/devicetree/bindings/arm/l2c2x0.yaml
  CHKDT   Documentation/devicetree/bindings/arm/renesas.yaml
  CHKDT   Documentation/devicetree/bindings/arm/tegra.yaml
  CHKDT   Documentation/devicetree/bindings/arm/amlogic.yaml
  CHKDT   Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml
  CHKDT   Documentation/devicetree/bindings/arm/sti.yaml
  CHKDT   Documentation/devicetree/bindings/arm/primecell.yaml
  CHKDT   Documentation/devicetree/bindings/arm/cpus.yaml
  CHKDT   Documentation/devicetree/bindings/bus/allwinner,sun50i-a64-de2.yaml
  CHKDT   Documentation/devicetree/bindings/bus/allwinner,sun8i-a23-rsb.yaml
  CHKDT   Documentation/devicetree/bindings/clock/fixed-factor-clock.yaml
  CHKDT   Documentation/devicetree/bindings/clock/milbeaut-clock.yaml
  CHKDT   Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml
  CHKDT   Documentation/devicetree/bindings/clock/imx8mn-clock.yaml
  CHKDT   Documentation/devicetree/bindings/clock/fixed-clock.yaml
  CHKDT   Documentation/devicetree/bindings/crypto/allwinner,sun4i-a10-crypto.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/ampire,am-480272h3tmqw-t01h.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/pda,91-00156-a0.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/bananapi,s070wv20-ct16.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/dlc,dlc0700yzg-1.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/mitsubishi,aa121td01.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/logicpd,type28.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/tfc,s9700rtwv43tr-01b.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/ti,nspire.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/armadeus,st0700-adapt.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/raspberrypi,7inch-touchscreen.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/lvds.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/panel-common.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/sgd,gktw70sdae4se.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/mitsubishi,aa104xd12.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
  CHKDT   Documentation/devicetree/bindings/display/amlogic,meson-vpu.yaml
  CHKDT   Documentation/devicetree/bindings/display/dsi-controller.yaml
  CHKDT   Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
  CHKDT   Documentation/devicetree/bindings/display/amlogic,meson-dw-hdmi.yaml
  CHKDT   Documentation/devicetree/bindings/display/simple-framebuffer.yaml
  CHKDT   Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml
  CHKDT   Documentation/devicetree/bindings/dma/dma-router.yaml
  CHKDT   Documentation/devicetree/bindings/dma/dma-controller.yaml
  CHKDT   Documentation/devicetree/bindings/dma/allwinner,sun50i-a64-dma.yaml
  CHKDT   Documentation/devicetree/bindings/dma/allwinner,sun4i-a10-dma.yaml
  CHKDT   Documentation/devicetree/bindings/dma/dma-common.yaml
  CHKDT   Documentation/devicetree/bindings/dma/allwinner,sun6i-a31-dma.yaml
  CHKDT   Documentation/devicetree/bindings/dsp/fsl,dsp.yaml
  CHKDT   Documentation/devicetree/bindings/example-schema.yaml
  CHKDT   Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml:
properties:reg:minItems: False schema does not allow 3
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml:
properties:reg:maxItems: False schema does not allow 3
Documentation/devicetree/bindings/Makefile:12: recipe for target
'Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.example.dts'
failed
make[1]: *** [Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.example.dts]
Error 1
Makefile:1263: recipe for target 'dt_binding_check' failed
make: *** [dt_binding_check] Error 2

Re: [PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML

[Sam Ravnborg]

Hi Jagan.

On Mon, Jan 06, 2020 at 03:41:40PM +0530, Jagan Teki wrote:
> Hi Sam,
> 
> On Sat, Jan 4, 2020 at 8:47 PM Sam Ravnborg <sam@ravnborg.org> wrote:
> >
> > Hi Jagan.
> > On Wed, Jan 01, 2020 at 04:54:38PM +0530, Jagan Teki wrote:
> > > These panel bindings are owned by me, so updated all of them into
> > > YAML DT schema.
> > >
> > > Any inputs?
> > Thanks for doing the conversion.
> >
> > dt_binding_check was not happy:
> > Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
> >   DTC     Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
> >   CHECK   Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
> > Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
> > Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: Additional items are not allowed ('simple-panel' was unexpected)
> > Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: ['friendlyarm,hd702e', 'simple-panel'] is too long
> >   DTC     Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dt.yaml
> > Error: Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dts:22.42-43 syntax error
> > FATAL ERROR: Unable to parse input tree
> >
> > Please fix and check the bindings using dt_binding_check before
> > resubmit.
> >
> > I had to install libyaml-dev (as least I recall this was the name)
> > before dt_binding_check worked OK for me.
> 
> I did check dt_binfing_check with this series. Here is the complete
> build log and you can see the panels related to this series are
> checked fine. Let me know if I miss anything here?

Due to other errors you never reached the poart that checks the example.
I use the following to focus on a single binding:

(from my script, but you get the idea)

domake ${arch} ${config} dt_binding_check DT_SCHEMA_FILES=${bindings}/display/panel/feixin,k101-im2ba02.yaml

In this way I can ignore all the bindings that otherwise fails.

Note: I sometimes have to clean my ouput directory to force that the
examples are tested.

It is also a good idea to introduce errors on purpose in the example, in
this way you can check that they are tested.

	Sam