1. Patchew
  2. linux
  3. [PATCH 0/5] dt-bindings: display: panel: Few cleanups and fixes
  4. View patch

[PATCH 5/5] dt-bindings: display: panel: Align style of "true" properties

Krzysztof Kozlowski posted 5 patches 1 month ago
[PATCH 5/5] dt-bindings: display: panel: Align style of "true" properties
Posted by Krzysztof Kozlowski 1 month ago 
For code readability, several bindings which list allowed properties
with ": true" syntax group them in one place, without line breaks
between each.  Align a few bindings to match this style.  No functional
impact.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
---
 .../devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml | 1 -
 Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml  | 3 ---
 .../devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml      | 2 --
 .../devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml    | 1 +
 .../devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml    | 1 +
 .../devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml    | 5 ++---
 .../devicetree/bindings/display/panel/novatek,nt35510.yaml         | 3 ++-
 .../devicetree/bindings/display/panel/renesas,r61307.yaml          | 3 +--
 .../devicetree/bindings/display/panel/renesas,r69328.yaml          | 1 -
 .../devicetree/bindings/display/panel/rocktech,jh057n00900.yaml    | 5 ++---
 .../bindings/display/panel/sony,tulip-truly-nt35521.yaml           | 2 --
 .../devicetree/bindings/display/panel/startek,kd070fhfid015.yaml   | 7 ++-----
 12 files changed, 11 insertions(+), 23 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml b/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
index 92df69e80a82..f288fa2390c9 100644
--- a/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
+++ b/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
@@ -28,7 +28,6 @@ properties:
 
   port: true
   reset-gpios: true
-
   backlight: true
 
 required:
diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
index 182a2b825e1c..84e840e0224f 100644
--- a/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
+++ b/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
@@ -33,11 +33,8 @@ properties:
     maxItems: 1
 
   reset-gpios: true
-
   backlight: true
-
   rotation: true
-
   port: true
 
   vcc-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml b/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
index 5802fb3c9ffe..2fa07ec55b08 100644
--- a/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
+++ b/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
@@ -35,9 +35,7 @@ properties:
     description: supply regulator for VCCIO, usually 1.8V
 
   reset-gpios: true
-
   backlight: true
-
   port: true
 
 required:
diff --git a/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml b/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
index 5fcea62fd58f..2f49a6bbf3d7 100644
--- a/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
+++ b/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
@@ -25,6 +25,7 @@ properties:
   backlight: true
   port: true
   reset-gpios: true
+
   iovcc-supply:
     description: regulator that supplies the iovcc voltage
   vci-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml b/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
index b0e2c82232d3..3f56047f4469 100644
--- a/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
+++ b/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
@@ -24,6 +24,7 @@ properties:
   backlight: true
   port: true
   reset-gpios: true
+
   iovcc-supply:
     description: regulator that supplies the iovcc voltage
   vcc-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml b/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml
index 74ff772973d6..b8b153a6e6cc 100644
--- a/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml
+++ b/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml
@@ -22,7 +22,6 @@ properties:
       - mantix,mlaf057we51-x
       - ys,ys57pss36bh5gq
 
-  port: true
   reg:
     maxItems: 1
     description: DSI virtual channel
@@ -36,13 +35,13 @@ properties:
   vddi-supply:
     description: 1.8V I/O voltage supply
 
-  reset-gpios: true
-
   mantix,tp-rstn-gpios:
     maxItems: 1
     description: second reset line that triggers DSI config load
 
   backlight: true
+  port: true
+  reset-gpios: true
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
index b39fd0c5a48a..43d134daf0ac 100644
--- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
+++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
@@ -28,13 +28,14 @@ properties:
   reg:
     maxItems: 1
 
-  reset-gpios: true
   vdd-supply:
     description: regulator that supplies the vdd voltage
   vddi-supply:
     description: regulator that supplies the vddi voltage
+
   backlight: true
   port: true
+  reset-gpios: true
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml b/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml
index 90cce221c0d1..3d7761717b74 100644
--- a/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml
+++ b/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml
@@ -33,8 +33,6 @@ properties:
   iovcc-supply:
     description: Regulator for 1.8V IO power supply.
 
-  backlight: true
-
   renesas,gamma:
     $ref: /schemas/types.yaml#/definitions/uint32
     description:
@@ -51,6 +49,7 @@ properties:
     type: boolean
     description: digital contrast adjustment
 
+  backlight: true
   reset-gpios: true
   port: true
 
diff --git a/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml b/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml
index 1cd219b510ee..740185f778a1 100644
--- a/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml
+++ b/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml
@@ -33,7 +33,6 @@ properties:
     description: Regulator for 1.8V IO power supply.
 
   backlight: true
-
   reset-gpios: true
   port: true
 
diff --git a/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml b/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml
index 4ae152cc55e0..ebfc825b8346 100644
--- a/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml
+++ b/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml
@@ -33,7 +33,6 @@ properties:
       # Xingbangda XBD599 5.99" 720x1440 TFT LCD panel
       - xingbangda,xbd599
 
-  port: true
   reg:
     maxItems: 1
     description: DSI virtual channel
@@ -44,9 +43,9 @@ properties:
   iovcc-supply:
     description: I/O voltage supply
 
-  reset-gpios: true
-
   backlight: true
+  port: true
+  reset-gpios: true
   rotation: true
 
 required:
diff --git a/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml b/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
index a58a31349757..85c5dee65383 100644
--- a/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
@@ -31,9 +31,7 @@ properties:
     description: Negative 5V supply
 
   reset-gpios: true
-
   enable-gpios: true
-
   port: true
 
 required:
diff --git a/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml b/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml
index 0819f38a9d2c..7fd9364fa385 100644
--- a/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml
+++ b/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml
@@ -16,8 +16,6 @@ properties:
   compatible:
     const: startek,kd070fhfid015
 
-  enable-gpios: true
-
   iovcc-supply:
     description: Reference to the regulator powering the panel IO pins.
 
@@ -25,11 +23,10 @@ properties:
     maxItems: 1
     description: DSI virtual channel
 
-  reset-gpios: true
-
+  enable-gpios: true
   port: true
-
   power-supply: true
+  reset-gpios: true
 
 required:
   - compatible

-- 
2.51.0
Re: [PATCH 5/5] dt-bindings: display: panel: Align style of "true" properties
Posted by Neil Armstrong 1 month ago 
On 3/6/26 13:02, Krzysztof Kozlowski wrote:
> For code readability, several bindings which list allowed properties
> with ": true" syntax group them in one place, without line breaks
> between each.  Align a few bindings to match this style.  No functional
> impact.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> ---
>   .../devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml | 1 -
>   Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml  | 3 ---
>   .../devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml      | 2 --
>   .../devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml    | 1 +
>   .../devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml    | 1 +
>   .../devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml    | 5 ++---
>   .../devicetree/bindings/display/panel/novatek,nt35510.yaml         | 3 ++-
>   .../devicetree/bindings/display/panel/renesas,r61307.yaml          | 3 +--
>   .../devicetree/bindings/display/panel/renesas,r69328.yaml          | 1 -
>   .../devicetree/bindings/display/panel/rocktech,jh057n00900.yaml    | 5 ++---
>   .../bindings/display/panel/sony,tulip-truly-nt35521.yaml           | 2 --
>   .../devicetree/bindings/display/panel/startek,kd070fhfid015.yaml   | 7 ++-----
>   12 files changed, 11 insertions(+), 23 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml b/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
> index 92df69e80a82..f288fa2390c9 100644
> --- a/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
> @@ -28,7 +28,6 @@ properties:
>   
>     port: true
>     reset-gpios: true
> -
>     backlight: true
>   
>   required:
> diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
> index 182a2b825e1c..84e840e0224f 100644
> --- a/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
> @@ -33,11 +33,8 @@ properties:
>       maxItems: 1
>   
>     reset-gpios: true
> -
>     backlight: true
> -
>     rotation: true
> -
>     port: true
>   
>     vcc-supply:
> diff --git a/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml b/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
> index 5802fb3c9ffe..2fa07ec55b08 100644
> --- a/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
> @@ -35,9 +35,7 @@ properties:
>       description: supply regulator for VCCIO, usually 1.8V
>   
>     reset-gpios: true
> -
>     backlight: true
> -
>     port: true
>   
>   required:
> diff --git a/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml b/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
> index 5fcea62fd58f..2f49a6bbf3d7 100644
> --- a/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
> @@ -25,6 +25,7 @@ properties:
>     backlight: true
>     port: true
>     reset-gpios: true
> +
>     iovcc-supply:
>       description: regulator that supplies the iovcc voltage
>     vci-supply:
> diff --git a/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml b/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
> index b0e2c82232d3..3f56047f4469 100644
> --- a/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
> @@ -24,6 +24,7 @@ properties:
>     backlight: true
>     port: true
>     reset-gpios: true
> +
>     iovcc-supply:
>       description: regulator that supplies the iovcc voltage
>     vcc-supply:
> diff --git a/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml b/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml
> index 74ff772973d6..b8b153a6e6cc 100644
> --- a/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml
> @@ -22,7 +22,6 @@ properties:
>         - mantix,mlaf057we51-x
>         - ys,ys57pss36bh5gq
>   
> -  port: true
>     reg:
>       maxItems: 1
>       description: DSI virtual channel
> @@ -36,13 +35,13 @@ properties:
>     vddi-supply:
>       description: 1.8V I/O voltage supply
>   
> -  reset-gpios: true
> -
>     mantix,tp-rstn-gpios:
>       maxItems: 1
>       description: second reset line that triggers DSI config load
>   
>     backlight: true
> +  port: true
> +  reset-gpios: true
>   
>   required:
>     - compatible
> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> index b39fd0c5a48a..43d134daf0ac 100644
> --- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> @@ -28,13 +28,14 @@ properties:
>     reg:
>       maxItems: 1
>   
> -  reset-gpios: true
>     vdd-supply:
>       description: regulator that supplies the vdd voltage
>     vddi-supply:
>       description: regulator that supplies the vddi voltage
> +
>     backlight: true
>     port: true
> +  reset-gpios: true
>   
>   required:
>     - compatible
> diff --git a/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml b/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml
> index 90cce221c0d1..3d7761717b74 100644
> --- a/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/renesas,r61307.yaml
> @@ -33,8 +33,6 @@ properties:
>     iovcc-supply:
>       description: Regulator for 1.8V IO power supply.
>   
> -  backlight: true
> -
>     renesas,gamma:
>       $ref: /schemas/types.yaml#/definitions/uint32
>       description:
> @@ -51,6 +49,7 @@ properties:
>       type: boolean
>       description: digital contrast adjustment
>   
> +  backlight: true
>     reset-gpios: true
>     port: true
>   
> diff --git a/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml b/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml
> index 1cd219b510ee..740185f778a1 100644
> --- a/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/renesas,r69328.yaml
> @@ -33,7 +33,6 @@ properties:
>       description: Regulator for 1.8V IO power supply.
>   
>     backlight: true
> -
>     reset-gpios: true
>     port: true
>   
> diff --git a/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml b/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml
> index 4ae152cc55e0..ebfc825b8346 100644
> --- a/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/rocktech,jh057n00900.yaml
> @@ -33,7 +33,6 @@ properties:
>         # Xingbangda XBD599 5.99" 720x1440 TFT LCD panel
>         - xingbangda,xbd599
>   
> -  port: true
>     reg:
>       maxItems: 1
>       description: DSI virtual channel
> @@ -44,9 +43,9 @@ properties:
>     iovcc-supply:
>       description: I/O voltage supply
>   
> -  reset-gpios: true
> -
>     backlight: true
> +  port: true
> +  reset-gpios: true
>     rotation: true
>   
>   required:
> diff --git a/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml b/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
> index a58a31349757..85c5dee65383 100644
> --- a/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
> @@ -31,9 +31,7 @@ properties:
>       description: Negative 5V supply
>   
>     reset-gpios: true
> -
>     enable-gpios: true
> -
>     port: true
>   
>   required:
> diff --git a/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml b/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml
> index 0819f38a9d2c..7fd9364fa385 100644
> --- a/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/startek,kd070fhfid015.yaml
> @@ -16,8 +16,6 @@ properties:
>     compatible:
>       const: startek,kd070fhfid015
>   
> -  enable-gpios: true
> -
>     iovcc-supply:
>       description: Reference to the regulator powering the panel IO pins.
>   
> @@ -25,11 +23,10 @@ properties:
>       maxItems: 1
>       description: DSI virtual channel
>   
> -  reset-gpios: true
> -
> +  enable-gpios: true
>     port: true
> -
>     power-supply: true
> +  reset-gpios: true
>   
>   required:
>     - compatible
> 

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>

Thanks,
Neil

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.