2024-05-09 09:43:27

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

Hi,

Cleanups for display panel bindings.

Rob, maybe you could take entire set if it applies? I based it on
linux-next, so letl me know if I need to rebase on your for-next.

I actually do not know if devices affected here take more than one chip
select or DSI virtual channel number. I assume DTS and examples are
correct here. Worst case this can be fixed later - maxItems:2.

Best regards,
Krzysztof

---
Krzysztof Kozlowski (3):
dt-bindings: display: samsung,ams495qa01: add missing SPI properties ref
dt-bindings: display: panel: constrain 'reg' in SPI panels
dt-bindings: display: panel: constrain 'reg' in DSI panels

Documentation/devicetree/bindings/display/panel/abt,y030xx067a.yaml | 4 +++-
.../devicetree/bindings/display/panel/asus,z00t-tm5p5-nt35596.yaml | 5 ++++-
.../devicetree/bindings/display/panel/boe,bf060y8m-aj0.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/boe,himax8279d.yaml | 4 +++-
.../devicetree/bindings/display/panel/boe,th101mb31ig002-28a.yaml | 4 +++-
.../devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml | 2 +-
Documentation/devicetree/bindings/display/panel/elida,kd35t133.yaml | 5 ++++-
.../devicetree/bindings/display/panel/fascontek,fs035vg158.yaml | 3 +++
.../devicetree/bindings/display/panel/feixin,k101-im2ba02.yaml | 5 ++++-
Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml | 3 ++-
Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/ilitek,ili9322.yaml | 3 +++
Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml | 3 ++-
Documentation/devicetree/bindings/display/panel/ilitek,ili9805.yaml | 4 +++-
.../devicetree/bindings/display/panel/ilitek,ili9881c.yaml | 4 +++-
.../devicetree/bindings/display/panel/innolux,ej030na.yaml | 4 +++-
.../devicetree/bindings/display/panel/innolux,p097pfg.yaml | 4 +++-
.../devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml | 3 ++-
.../devicetree/bindings/display/panel/jdi,lpm102a188a.yaml | 4 +++-
.../devicetree/bindings/display/panel/jdi,lt070me05000.yaml | 4 +++-
.../devicetree/bindings/display/panel/kingdisplay,kd035g6-54nt.yaml | 4 +++-
.../devicetree/bindings/display/panel/leadtek,ltk035c5444t.yaml | 3 +++
.../devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml | 5 ++++-
.../devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml | 5 ++++-
Documentation/devicetree/bindings/display/panel/lg,lg4573.yaml | 3 ++-
Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml | 4 +++-
.../devicetree/bindings/display/panel/lgphilips,lb035q02.yaml | 3 +++
Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml | 4 +++-
.../devicetree/bindings/display/panel/newvision,nv3051d.yaml | 4 +++-
.../devicetree/bindings/display/panel/novatek,nt35510.yaml | 5 ++++-
.../devicetree/bindings/display/panel/novatek,nt35950.yaml | 4 +++-
.../devicetree/bindings/display/panel/novatek,nt36523.yaml | 4 +++-
.../devicetree/bindings/display/panel/novatek,nt36672a.yaml | 4 +++-
.../devicetree/bindings/display/panel/olimex,lcd-olinuxino.yaml | 4 +++-
.../devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml | 3 +++
.../devicetree/bindings/display/panel/raydium,rm67191.yaml | 4 +++-
.../devicetree/bindings/display/panel/raydium,rm692e5.yaml | 4 +++-
.../devicetree/bindings/display/panel/raydium,rm69380.yaml | 5 +++--
Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml | 2 +-
.../devicetree/bindings/display/panel/samsung,amoled-mipi-dsi.yaml | 4 +++-
.../devicetree/bindings/display/panel/samsung,ams495qa01.yaml | 5 ++++-
Documentation/devicetree/bindings/display/panel/samsung,ld9040.yaml | 4 +++-
.../devicetree/bindings/display/panel/samsung,lms380kf01.yaml | 3 ++-
.../devicetree/bindings/display/panel/samsung,lms397kf04.yaml | 3 ++-
.../devicetree/bindings/display/panel/samsung,s6d16d0.yaml | 4 +++-
.../devicetree/bindings/display/panel/samsung,s6d27a1.yaml | 3 ++-
.../devicetree/bindings/display/panel/samsung,s6d7aa0.yaml | 3 ++-
.../devicetree/bindings/display/panel/samsung,s6e63m0.yaml | 4 +++-
.../bindings/display/panel/samsung,s6e88a0-ams452ef01.yaml | 5 ++++-
.../devicetree/bindings/display/panel/samsung,s6e8aa0.yaml | 4 +++-
.../devicetree/bindings/display/panel/sharp,lq101r1sx01.yaml | 4 +++-
.../devicetree/bindings/display/panel/sharp,ls043t1le01.yaml | 4 +++-
.../devicetree/bindings/display/panel/sharp,ls060t1sx01.yaml | 4 +++-
.../devicetree/bindings/display/panel/sitronix,st7789v.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml | 5 ++++-
Documentation/devicetree/bindings/display/panel/sony,acx565akm.yaml | 3 +++
.../devicetree/bindings/display/panel/sony,td4353-jdi.yaml | 3 ++-
.../devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml | 3 ++-
.../devicetree/bindings/display/panel/synaptics,r63353.yaml | 6 ++++--
Documentation/devicetree/bindings/display/panel/tpo,td.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml | 3 ++-
.../devicetree/bindings/display/panel/visionox,rm69299.yaml | 3 ++-
.../devicetree/bindings/display/panel/xinpeng,xpp055c272.yaml | 5 ++++-
64 files changed, 187 insertions(+), 60 deletions(-)
---
base-commit: b010817abf0197596fbfddf1957bac968facb991
change-id: 20240509-dt-bindings-dsi-panel-reg-a6e7d930b46f

Best regards,
--
Krzysztof Kozlowski <[email protected]>



2024-05-09 09:43:38

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 1/3] dt-bindings: display: samsung,ams495qa01: add missing SPI properties ref

Samsung AMS495QA01 panel is a SPI device, so it should reference
spi-peripheral-props.yaml schema to allow and validate the SPI device
properties.

Fixes: 92be07c65b22 ("dt-bindings: display: panel: Add Samsung AMS495QA01")
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml b/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml
index 58fa073ce258..a5ff4de74a15 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml
@@ -11,6 +11,7 @@ maintainers:

allOf:
- $ref: panel-common.yaml#
+ - $ref: /schemas/spi/spi-peripheral-props.yaml#

properties:
compatible:

--
2.43.0


2024-05-09 09:43:57

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 2/3] dt-bindings: display: panel: constrain 'reg' in SPI panels

SPI-attached devices could have more than one chip-select, thus their
bindings are supposed to constrain the 'reg' property to match hardware.
Add missing 'reg' constrain for SPI-attached display panels.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
Documentation/devicetree/bindings/display/panel/abt,y030xx067a.yaml | 4 +++-
.../devicetree/bindings/display/panel/fascontek,fs035vg158.yaml | 3 +++
Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/ilitek,ili9322.yaml | 3 +++
Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml | 3 ++-
Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml | 4 +++-
.../devicetree/bindings/display/panel/kingdisplay,kd035g6-54nt.yaml | 4 +++-
.../devicetree/bindings/display/panel/leadtek,ltk035c5444t.yaml | 3 +++
Documentation/devicetree/bindings/display/panel/lg,lg4573.yaml | 3 ++-
.../devicetree/bindings/display/panel/lgphilips,lb035q02.yaml | 3 +++
Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml | 4 +++-
.../devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml | 3 +++
.../devicetree/bindings/display/panel/samsung,ams495qa01.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/samsung,ld9040.yaml | 4 +++-
.../devicetree/bindings/display/panel/samsung,lms380kf01.yaml | 3 ++-
.../devicetree/bindings/display/panel/samsung,lms397kf04.yaml | 3 ++-
Documentation/devicetree/bindings/display/panel/samsung,s6d27a1.yaml | 3 ++-
Documentation/devicetree/bindings/display/panel/samsung,s6e63m0.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/sitronix,st7789v.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/sony,acx565akm.yaml | 3 +++
Documentation/devicetree/bindings/display/panel/tpo,td.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml | 3 ++-
22 files changed, 60 insertions(+), 16 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/panel/abt,y030xx067a.yaml b/Documentation/devicetree/bindings/display/panel/abt,y030xx067a.yaml
index acd2f3faa6b9..0aa2d3fbadaa 100644
--- a/Documentation/devicetree/bindings/display/panel/abt,y030xx067a.yaml
+++ b/Documentation/devicetree/bindings/display/panel/abt,y030xx067a.yaml
@@ -17,10 +17,12 @@ properties:
compatible:
const: abt,y030xx067a

+ reg:
+ maxItems: 1
+
backlight: true
port: true
power-supply: true
- reg: true
reset-gpios: true

required:
diff --git a/Documentation/devicetree/bindings/display/panel/fascontek,fs035vg158.yaml b/Documentation/devicetree/bindings/display/panel/fascontek,fs035vg158.yaml
index d13c4bd26de4..9847da784cc8 100644
--- a/Documentation/devicetree/bindings/display/panel/fascontek,fs035vg158.yaml
+++ b/Documentation/devicetree/bindings/display/panel/fascontek,fs035vg158.yaml
@@ -17,6 +17,9 @@ properties:
compatible:
const: fascontek,fs035vg158

+ reg:
+ maxItems: 1
+
spi-3wire: true

required:
diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml
index 3cabbba86581..ef5a2240b684 100644
--- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml
@@ -24,6 +24,9 @@ properties:
- newhaven,1.8-128160EF
- const: ilitek,ili9163

+ reg:
+ maxItems: 1
+
spi-max-frequency:
maximum: 32000000

@@ -32,7 +35,6 @@ properties:
description: Display data/command selection (D/CX)

backlight: true
- reg: true
reset-gpios: true
rotation: true

diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9322.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9322.yaml
index 7d221ef35443..44423465f6e3 100644
--- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9322.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9322.yaml
@@ -26,6 +26,9 @@ properties:
- dlink,dir-685-panel
- const: ilitek,ili9322

+ reg:
+ maxItems: 1
+
reset-gpios: true
port: true

diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml
index 94f169ea065a..5f41758c96d5 100644
--- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml
@@ -28,7 +28,8 @@ properties:
- canaan,kd233-tft
- const: ilitek,ili9341

- reg: true
+ reg:
+ maxItems: 1

dc-gpios:
maxItems: 1
diff --git a/Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml b/Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml
index 72788e3e6c59..c7df9a7f6589 100644
--- a/Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml
+++ b/Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml
@@ -17,10 +17,12 @@ properties:
compatible:
const: innolux,ej030na

+ reg:
+ maxItems: 1
+
backlight: true
port: true
power-supply: true
- reg: true
reset-gpios: true

required:
diff --git a/Documentation/devicetree/bindings/display/panel/kingdisplay,kd035g6-54nt.yaml b/Documentation/devicetree/bindings/display/panel/kingdisplay,kd035g6-54nt.yaml
index b4be9bd8ddde..d86c916f7b55 100644
--- a/Documentation/devicetree/bindings/display/panel/kingdisplay,kd035g6-54nt.yaml
+++ b/Documentation/devicetree/bindings/display/panel/kingdisplay,kd035g6-54nt.yaml
@@ -17,10 +17,12 @@ properties:
compatible:
const: kingdisplay,kd035g6-54nt

+ reg:
+ maxItems: 1
+
backlight: true
port: true
power-supply: true
- reg: true
reset-gpios: true

spi-3wire: true
diff --git a/Documentation/devicetree/bindings/display/panel/leadtek,ltk035c5444t.yaml b/Documentation/devicetree/bindings/display/panel/leadtek,ltk035c5444t.yaml
index 7a55961e1a3d..b5dc02b27200 100644
--- a/Documentation/devicetree/bindings/display/panel/leadtek,ltk035c5444t.yaml
+++ b/Documentation/devicetree/bindings/display/panel/leadtek,ltk035c5444t.yaml
@@ -18,6 +18,9 @@ properties:
compatible:
const: leadtek,ltk035c5444t

+ reg:
+ maxItems: 1
+
spi-3wire: true

required:
diff --git a/Documentation/devicetree/bindings/display/panel/lg,lg4573.yaml b/Documentation/devicetree/bindings/display/panel/lg,lg4573.yaml
index ee357e139ac0..590ccc27d104 100644
--- a/Documentation/devicetree/bindings/display/panel/lg,lg4573.yaml
+++ b/Documentation/devicetree/bindings/display/panel/lg,lg4573.yaml
@@ -21,7 +21,8 @@ properties:
compatible:
const: lg,lg4573

- reg: true
+ reg:
+ maxItems: 1

required:
- compatible
diff --git a/Documentation/devicetree/bindings/display/panel/lgphilips,lb035q02.yaml b/Documentation/devicetree/bindings/display/panel/lgphilips,lb035q02.yaml
index 628c4b898111..3de17fd8513b 100644
--- a/Documentation/devicetree/bindings/display/panel/lgphilips,lb035q02.yaml
+++ b/Documentation/devicetree/bindings/display/panel/lgphilips,lb035q02.yaml
@@ -17,6 +17,9 @@ properties:
compatible:
const: lgphilips,lb035q02

+ reg:
+ maxItems: 1
+
label: true
enable-gpios: true
port: true
diff --git a/Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml b/Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml
index accf933d6e46..1cffe4d6d498 100644
--- a/Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml
+++ b/Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml
@@ -21,9 +21,11 @@ properties:
compatible:
const: nec,nl8048hl11

+ reg:
+ maxItems: 1
+
label: true
port: true
- reg: true
reset-gpios: true

spi-max-frequency:
diff --git a/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml b/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml
index e808215cb39e..d0ac31ab60cf 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml
+++ b/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml
@@ -71,6 +71,9 @@ properties:
- shineworld,lh133k
- const: panel-mipi-dbi-spi

+ reg:
+ maxItems: 1
+
write-only:
type: boolean
description:
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml b/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml
index a5ff4de74a15..e081c84a932b 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,ams495qa01.yaml
@@ -17,7 +17,9 @@ properties:
compatible:
const: samsung,ams495qa01

- reg: true
+ reg:
+ maxItems: 1
+
reset-gpios:
description: reset gpio, must be GPIO_ACTIVE_LOW
elvdd-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,ld9040.yaml b/Documentation/devicetree/bindings/display/panel/samsung,ld9040.yaml
index c0fabeb38628..bc92b16c95b9 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,ld9040.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,ld9040.yaml
@@ -17,9 +17,11 @@ properties:
compatible:
const: samsung,ld9040

+ reg:
+ maxItems: 1
+
display-timings: true
port: true
- reg: true
reset-gpios: true

vdd3-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,lms380kf01.yaml b/Documentation/devicetree/bindings/display/panel/samsung,lms380kf01.yaml
index 70ffc88d2a08..7ce8540551f9 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,lms380kf01.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,lms380kf01.yaml
@@ -21,7 +21,8 @@ properties:
compatible:
const: samsung,lms380kf01

- reg: true
+ reg:
+ maxItems: 1

interrupts:
description: provides an optional ESD (electrostatic discharge)
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,lms397kf04.yaml b/Documentation/devicetree/bindings/display/panel/samsung,lms397kf04.yaml
index 5e77cee93f83..9363032883de 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,lms397kf04.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,lms397kf04.yaml
@@ -20,7 +20,8 @@ properties:
compatible:
const: samsung,lms397kf04

- reg: true
+ reg:
+ maxItems: 1

reset-gpios: true

diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6d27a1.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6d27a1.yaml
index d273faf4442a..d74904164719 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,s6d27a1.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6d27a1.yaml
@@ -20,7 +20,8 @@ properties:
compatible:
const: samsung,s6d27a1

- reg: true
+ reg:
+ maxItems: 1

interrupts:
description: provides an optional ESD (electrostatic discharge)
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e63m0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e63m0.yaml
index 6f1fc7469f07..c47e2a1a30e5 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,s6e63m0.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e63m0.yaml
@@ -18,7 +18,9 @@ properties:
compatible:
const: samsung,s6e63m0

- reg: true
+ reg:
+ maxItems: 1
+
reset-gpios: true
port: true
default-brightness: true
diff --git a/Documentation/devicetree/bindings/display/panel/sitronix,st7789v.yaml b/Documentation/devicetree/bindings/display/panel/sitronix,st7789v.yaml
index ef162b51d010..0ce2ea13583d 100644
--- a/Documentation/devicetree/bindings/display/panel/sitronix,st7789v.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sitronix,st7789v.yaml
@@ -21,7 +21,9 @@ properties:
- jasonic,jt240mhqs-hwt-ek-e3
- sitronix,st7789v

- reg: true
+ reg:
+ maxItems: 1
+
reset-gpios: true
power-supply: true
backlight: true
diff --git a/Documentation/devicetree/bindings/display/panel/sony,acx565akm.yaml b/Documentation/devicetree/bindings/display/panel/sony,acx565akm.yaml
index 98abdf4ddeac..5a8260224b74 100644
--- a/Documentation/devicetree/bindings/display/panel/sony,acx565akm.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sony,acx565akm.yaml
@@ -17,6 +17,9 @@ properties:
compatible:
const: sony,acx565akm

+ reg:
+ maxItems: 1
+
label: true
reset-gpios: true
port: true
diff --git a/Documentation/devicetree/bindings/display/panel/tpo,td.yaml b/Documentation/devicetree/bindings/display/panel/tpo,td.yaml
index e8c8ee8d7c88..7edd29df4bbb 100644
--- a/Documentation/devicetree/bindings/display/panel/tpo,td.yaml
+++ b/Documentation/devicetree/bindings/display/panel/tpo,td.yaml
@@ -22,7 +22,9 @@ properties:
# Toppoly TD043MTEA1 Panel
- tpo,td043mtea1

- reg: true
+ reg:
+ maxItems: 1
+
label: true
reset-gpios: true
backlight: true
diff --git a/Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml b/Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml
index f0243d196191..59a373728e62 100644
--- a/Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml
+++ b/Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml
@@ -52,7 +52,8 @@ properties:
- const: tpo,tpg110
- const: tpo,tpg110

- reg: true
+ reg:
+ maxItems: 1

grestb-gpios:
maxItems: 1

--
2.43.0


2024-05-09 09:44:09

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 3/3] dt-bindings: display: panel: constrain 'reg' in DSI panels

DSI-attached devices could respond to more than one virtual channel
number, thus their bindings are supposed to constrain the 'reg' property
to match hardware. Add missing 'reg' constrain for DSI-attached display
panels, based on DTS sources in Linux kernel (assume all devices take
only one channel number).

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../devicetree/bindings/display/panel/asus,z00t-tm5p5-nt35596.yaml | 5 ++++-
.../devicetree/bindings/display/panel/boe,bf060y8m-aj0.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/boe,himax8279d.yaml | 4 +++-
.../devicetree/bindings/display/panel/boe,th101mb31ig002-28a.yaml | 4 +++-
.../devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml | 2 +-
Documentation/devicetree/bindings/display/panel/elida,kd35t133.yaml | 5 ++++-
.../devicetree/bindings/display/panel/feixin,k101-im2ba02.yaml | 5 ++++-
Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml | 3 ++-
Documentation/devicetree/bindings/display/panel/ilitek,ili9805.yaml | 4 +++-
.../devicetree/bindings/display/panel/ilitek,ili9881c.yaml | 4 +++-
.../devicetree/bindings/display/panel/innolux,p097pfg.yaml | 4 +++-
.../devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml | 3 ++-
.../devicetree/bindings/display/panel/jdi,lpm102a188a.yaml | 4 +++-
.../devicetree/bindings/display/panel/jdi,lt070me05000.yaml | 4 +++-
.../devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml | 5 ++++-
.../devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml | 5 ++++-
Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml | 4 +++-
.../devicetree/bindings/display/panel/newvision,nv3051d.yaml | 4 +++-
.../devicetree/bindings/display/panel/novatek,nt35510.yaml | 5 ++++-
.../devicetree/bindings/display/panel/novatek,nt35950.yaml | 4 +++-
.../devicetree/bindings/display/panel/novatek,nt36523.yaml | 4 +++-
.../devicetree/bindings/display/panel/novatek,nt36672a.yaml | 4 +++-
.../devicetree/bindings/display/panel/olimex,lcd-olinuxino.yaml | 4 +++-
.../devicetree/bindings/display/panel/raydium,rm67191.yaml | 4 +++-
.../devicetree/bindings/display/panel/raydium,rm692e5.yaml | 4 +++-
.../devicetree/bindings/display/panel/raydium,rm69380.yaml | 5 +++--
Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml | 2 +-
.../devicetree/bindings/display/panel/samsung,amoled-mipi-dsi.yaml | 4 +++-
.../devicetree/bindings/display/panel/samsung,s6d16d0.yaml | 4 +++-
.../devicetree/bindings/display/panel/samsung,s6d7aa0.yaml | 3 ++-
.../bindings/display/panel/samsung,s6e88a0-ams452ef01.yaml | 5 ++++-
.../devicetree/bindings/display/panel/samsung,s6e8aa0.yaml | 4 +++-
.../devicetree/bindings/display/panel/sharp,lq101r1sx01.yaml | 4 +++-
.../devicetree/bindings/display/panel/sharp,ls043t1le01.yaml | 4 +++-
.../devicetree/bindings/display/panel/sharp,ls060t1sx01.yaml | 4 +++-
Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml | 5 ++++-
.../devicetree/bindings/display/panel/sony,td4353-jdi.yaml | 3 ++-
.../devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml | 3 ++-
.../devicetree/bindings/display/panel/synaptics,r63353.yaml | 6 ++++--
.../devicetree/bindings/display/panel/visionox,rm69299.yaml | 3 ++-
.../devicetree/bindings/display/panel/xinpeng,xpp055c272.yaml | 5 ++++-
42 files changed, 126 insertions(+), 44 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/panel/asus,z00t-tm5p5-nt35596.yaml b/Documentation/devicetree/bindings/display/panel/asus,z00t-tm5p5-nt35596.yaml
index 75a09df68ba0..2399cabf044c 100644
--- a/Documentation/devicetree/bindings/display/panel/asus,z00t-tm5p5-nt35596.yaml
+++ b/Documentation/devicetree/bindings/display/panel/asus,z00t-tm5p5-nt35596.yaml
@@ -21,7 +21,10 @@ allOf:
properties:
compatible:
const: asus,z00t-tm5p5-n35596
- reg: true
+
+ reg:
+ maxItems: 1
+
reset-gpios: true
vdd-supply:
description: core voltage supply
diff --git a/Documentation/devicetree/bindings/display/panel/boe,bf060y8m-aj0.yaml b/Documentation/devicetree/bindings/display/panel/boe,bf060y8m-aj0.yaml
index a8f3afa922c8..8b7448ad9138 100644
--- a/Documentation/devicetree/bindings/display/panel/boe,bf060y8m-aj0.yaml
+++ b/Documentation/devicetree/bindings/display/panel/boe,bf060y8m-aj0.yaml
@@ -26,6 +26,9 @@ properties:
compatible:
const: boe,bf060y8m-aj0

+ reg:
+ maxItems: 1
+
elvdd-supply:
description: EL Driving positive (VDD) supply (4.40-4.80V)
elvss-supply:
@@ -38,7 +41,6 @@ properties:
description: I/O voltage supply (1.62-1.98V)

port: true
- reg: true
reset-gpios: true

required:
diff --git a/Documentation/devicetree/bindings/display/panel/boe,himax8279d.yaml b/Documentation/devicetree/bindings/display/panel/boe,himax8279d.yaml
index 272a3a018a33..f2496cdd9260 100644
--- a/Documentation/devicetree/bindings/display/panel/boe,himax8279d.yaml
+++ b/Documentation/devicetree/bindings/display/panel/boe,himax8279d.yaml
@@ -18,9 +18,11 @@ properties:
- const: boe,himax8279d8p
- const: boe,himax8279d10p

+ reg:
+ maxItems: 1
+
backlight: true
enable-gpios: true
- reg: true

pp33-gpios:
maxItems: 1
diff --git a/Documentation/devicetree/bindings/display/panel/boe,th101mb31ig002-28a.yaml b/Documentation/devicetree/bindings/display/panel/boe,th101mb31ig002-28a.yaml
index 32df26cbfeed..5eaccce13c21 100644
--- a/Documentation/devicetree/bindings/display/panel/boe,th101mb31ig002-28a.yaml
+++ b/Documentation/devicetree/bindings/display/panel/boe,th101mb31ig002-28a.yaml
@@ -18,7 +18,9 @@ properties:
# BOE TH101MB31IG002-28A 10.1" WXGA TFT LCD panel
- boe,th101mb31ig002-28a

- reg: true
+ reg:
+ maxItems: 1
+
backlight: true
enable-gpios: true
power-supply: true
diff --git a/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml b/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml
index 906ef62709b8..9e603cad1348 100644
--- a/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml
+++ b/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml
@@ -38,7 +38,7 @@ properties:
- starry,ili9882t

reg:
- description: the virtual channel number of a DSI peripheral
+ maxItems: 1

enable-gpios:
description: a GPIO spec for the enable pin
diff --git a/Documentation/devicetree/bindings/display/panel/elida,kd35t133.yaml b/Documentation/devicetree/bindings/display/panel/elida,kd35t133.yaml
index 265ab6d30572..f4cb825d1e96 100644
--- a/Documentation/devicetree/bindings/display/panel/elida,kd35t133.yaml
+++ b/Documentation/devicetree/bindings/display/panel/elida,kd35t133.yaml
@@ -15,7 +15,10 @@ allOf:
properties:
compatible:
const: elida,kd35t133
- reg: true
+
+ reg:
+ maxItems: 1
+
backlight: true
port: true
reset-gpios: true
diff --git a/Documentation/devicetree/bindings/display/panel/feixin,k101-im2ba02.yaml b/Documentation/devicetree/bindings/display/panel/feixin,k101-im2ba02.yaml
index 81adb82f061d..0d8707a5844d 100644
--- a/Documentation/devicetree/bindings/display/panel/feixin,k101-im2ba02.yaml
+++ b/Documentation/devicetree/bindings/display/panel/feixin,k101-im2ba02.yaml
@@ -15,7 +15,10 @@ allOf:
properties:
compatible:
const: feixin,k101-im2ba02
- reg: true
+
+ reg:
+ maxItems: 1
+
backlight: true
reset-gpios: true
avdd-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
index 174661d13811..56bcd152f43c 100644
--- a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
+++ b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
@@ -21,6 +21,9 @@ properties:
contains:
const: djn,9a-3r063-1102b

+ reg:
+ maxItems: 1
+
vdd1-supply:
description: Digital voltage rail

@@ -30,7 +33,6 @@ properties:
vsp-supply:
description: Negative source voltage rail

- reg: true
port: true

required:
diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
index 916bb7f94206..644387e4fb6f 100644
--- a/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
+++ b/Documentation/devicetree/bindings/display/panel/himax,hx8394.yaml
@@ -26,7 +26,8 @@ properties:
- powkiddy,x55-panel
- const: himax,hx8394

- reg: true
+ reg:
+ maxItems: 1

reset-gpios: true

diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9805.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9805.yaml
index f4f91f93f490..ff67129c9466 100644
--- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9805.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9805.yaml
@@ -20,9 +20,11 @@ properties:
- tianma,tm041xdhg01
- const: ilitek,ili9805

+ reg:
+ maxItems: 1
+
avdd-supply: true
dvdd-supply: true
- reg: true

required:
- compatible
diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml
index a015dce72f60..baf5dfe5f5eb 100644
--- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml
@@ -24,9 +24,11 @@ properties:
- wanchanglong,w552946aba
- const: ilitek,ili9881c

+ reg:
+ maxItems: 1
+
backlight: true
power-supply: true
- reg: true
reset-gpios: true
rotation: true

diff --git a/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.yaml b/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.yaml
index 5a5f071627fb..4164e3f7061d 100644
--- a/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.yaml
+++ b/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.yaml
@@ -16,9 +16,11 @@ properties:
compatible:
const: innolux,p097pfg

+ reg:
+ maxItems: 1
+
backlight: true
enable-gpios: true
- reg: true

avdd-supply:
description: The regulator that provides positive voltage
diff --git a/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml b/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
index 41eb7fbf7715..20afdb4568a2 100644
--- a/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
+++ b/Documentation/devicetree/bindings/display/panel/jadard,jd9365da-h3.yaml
@@ -21,7 +21,8 @@ properties:
- radxa,display-8hd-ad002
- const: jadard,jd9365da-h3

- reg: true
+ reg:
+ maxItems: 1

vdd-supply:
description: supply regulator for VDD, usually 3.3V
diff --git a/Documentation/devicetree/bindings/display/panel/jdi,lpm102a188a.yaml b/Documentation/devicetree/bindings/display/panel/jdi,lpm102a188a.yaml
index 2f4d27a309a7..a8621459005b 100644
--- a/Documentation/devicetree/bindings/display/panel/jdi,lpm102a188a.yaml
+++ b/Documentation/devicetree/bindings/display/panel/jdi,lpm102a188a.yaml
@@ -26,7 +26,9 @@ properties:
compatible:
const: jdi,lpm102a188a

- reg: true
+ reg:
+ maxItems: 1
+
enable-gpios: true
reset-gpios: true
power-supply: true
diff --git a/Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.yaml b/Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.yaml
index 63c82a4378ff..0c8b5cb78bfe 100644
--- a/Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.yaml
+++ b/Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.yaml
@@ -16,8 +16,10 @@ properties:
compatible:
const: jdi,lt070me05000

+ reg:
+ maxItems: 1
+
enable-gpios: true
- reg: true
reset-gpios: true

vddp-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml b/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
index a40ab887ada7..e2a2dd4ef5fa 100644
--- a/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
+++ b/Documentation/devicetree/bindings/display/panel/leadtek,ltk050h3146w.yaml
@@ -18,7 +18,10 @@ properties:
- leadtek,ltk050h3146w
- leadtek,ltk050h3146w-a2
- leadtek,ltk050h3148w
- reg: true
+
+ reg:
+ maxItems: 1
+
backlight: true
reset-gpios: true
iovcc-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml b/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
index d589f1677214..af9e0ea0e72f 100644
--- a/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
+++ b/Documentation/devicetree/bindings/display/panel/leadtek,ltk500hd1829.yaml
@@ -17,7 +17,10 @@ properties:
enum:
- leadtek,ltk101b4029w
- leadtek,ltk500hd1829
- reg: true
+
+ reg:
+ maxItems: 1
+
backlight: true
reset-gpios: true
iovcc-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml b/Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
index 1e08648f5bc7..bbaaa783d184 100644
--- a/Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
+++ b/Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
@@ -21,7 +21,9 @@ properties:
items:
- const: lg,sw43408

- reg: true
+ reg:
+ maxItems: 1
+
port: true
vddi-supply: true
vpnl-supply: true
diff --git a/Documentation/devicetree/bindings/display/panel/newvision,nv3051d.yaml b/Documentation/devicetree/bindings/display/panel/newvision,nv3051d.yaml
index 7a634fbc465e..d3a25a8fd7e3 100644
--- a/Documentation/devicetree/bindings/display/panel/newvision,nv3051d.yaml
+++ b/Documentation/devicetree/bindings/display/panel/newvision,nv3051d.yaml
@@ -24,7 +24,9 @@ properties:
- powkiddy,rk2023-panel
- const: newvision,nv3051d

- reg: true
+ reg:
+ maxItems: 1
+
backlight: true
port: true
reset-gpios:
diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
index 91921f4b0e5f..bb50fd5506c3 100644
--- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
+++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
@@ -24,7 +24,10 @@ properties:
string determines how the NT35510 panel driver shall be configured
to work with the indicated panel. The novatek,nt35510 compatible shall
always be provided as a fallback.
- reg: true
+
+ reg:
+ maxItems: 1
+
reset-gpios: true
vdd-supply:
description: regulator that supplies the vdd voltage
diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35950.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35950.yaml
index 7cac93b20944..a9e40493986b 100644
--- a/Documentation/devicetree/bindings/display/panel/novatek,nt35950.yaml
+++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35950.yaml
@@ -33,6 +33,9 @@ properties:
to work with the indicated panel. The novatek,nt35950 compatible shall
always be provided as a fallback.

+ reg:
+ maxItems: 1
+
reset-gpios:
maxItems: 1
description: phandle of gpio for reset line - This should be 8mA, gpio
@@ -49,7 +52,6 @@ properties:

backlight: true
ports: true
- reg: true

required:
- compatible
diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml
index bbeea8cfa5fb..c4bae4f77085 100644
--- a/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml
+++ b/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml
@@ -27,6 +27,9 @@ properties:
- lenovo,j606f-boe-nt36523w
- const: novatek,nt36523w

+ reg:
+ maxItems: 1
+
reset-gpios:
maxItems: 1
description: phandle of gpio for reset line - This should be 8mA
@@ -34,7 +37,6 @@ properties:
vddio-supply:
description: regulator that supplies the I/O voltage

- reg: true
rotation: true
backlight: true

diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt36672a.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt36672a.yaml
index ae821f465e1c..800a2f0a4dad 100644
--- a/Documentation/devicetree/bindings/display/panel/novatek,nt36672a.yaml
+++ b/Documentation/devicetree/bindings/display/panel/novatek,nt36672a.yaml
@@ -29,6 +29,9 @@ properties:
determines how the NT36672A panel driver is configured for the indicated
panel. The novatek,nt36672a compatible shall always be provided as a fallback.

+ reg:
+ maxItems: 1
+
reset-gpios:
maxItems: 1
description: phandle of gpio for reset line - This should be 8mA, gpio
@@ -44,7 +47,6 @@ properties:
vddneg-supply:
description: phandle of the negative boost supply regulator

- reg: true
port: true
backlight: true

diff --git a/Documentation/devicetree/bindings/display/panel/olimex,lcd-olinuxino.yaml b/Documentation/devicetree/bindings/display/panel/olimex,lcd-olinuxino.yaml
index 72463795e4c6..e5d8785fdf90 100644
--- a/Documentation/devicetree/bindings/display/panel/olimex,lcd-olinuxino.yaml
+++ b/Documentation/devicetree/bindings/display/panel/olimex,lcd-olinuxino.yaml
@@ -38,10 +38,12 @@ properties:
compatible:
const: olimex,lcd-olinuxino

+ reg:
+ maxItems: 1
+
backlight: true
enable-gpios: true
power-supply: true
- reg: true

required:
- compatible
diff --git a/Documentation/devicetree/bindings/display/panel/raydium,rm67191.yaml b/Documentation/devicetree/bindings/display/panel/raydium,rm67191.yaml
index d62fd692bf10..4825792bf712 100644
--- a/Documentation/devicetree/bindings/display/panel/raydium,rm67191.yaml
+++ b/Documentation/devicetree/bindings/display/panel/raydium,rm67191.yaml
@@ -16,7 +16,9 @@ properties:
compatible:
const: raydium,rm67191

- reg: true
+ reg:
+ maxItems: 1
+
port: true
reset-gpios: true
width-mm: true
diff --git a/Documentation/devicetree/bindings/display/panel/raydium,rm692e5.yaml b/Documentation/devicetree/bindings/display/panel/raydium,rm692e5.yaml
index f436ba6738ca..7ad223f98253 100644
--- a/Documentation/devicetree/bindings/display/panel/raydium,rm692e5.yaml
+++ b/Documentation/devicetree/bindings/display/panel/raydium,rm692e5.yaml
@@ -22,6 +22,9 @@ properties:
- const: fairphone,fp5-rm692e5-boe
- const: raydium,rm692e5

+ reg:
+ maxItems: 1
+
dvdd-supply:
description: Digital voltage rail

@@ -31,7 +34,6 @@ properties:
vddio-supply:
description: I/O voltage rail

- reg: true
port: true

required:
diff --git a/Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml b/Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml
index b17765b2b351..ec445ff5631c 100644
--- a/Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml
+++ b/Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml
@@ -28,6 +28,9 @@ properties:
to work with the indicated panel. The raydium,rm69380 compatible shall
always be provided as a fallback.

+ reg:
+ maxItems: 1
+
avdd-supply:
description: Analog voltage rail

@@ -38,8 +41,6 @@ properties:
maxItems: 1
description: phandle of gpio for reset line - This should be active low

- reg: true
-
required:
- compatible
- reg
diff --git a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
index 95ce22c6787a..04f86e0cbac9 100644
--- a/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
@@ -14,7 +14,7 @@ properties:
const: ronbo,rb070d30

reg:
- description: MIPI-DSI virtual channel
+ maxItems: 1

power-gpios:
description: GPIO used for the power pin
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,amoled-mipi-dsi.yaml b/Documentation/devicetree/bindings/display/panel/samsung,amoled-mipi-dsi.yaml
index ccc482570d6a..e8f9e9d06a29 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,amoled-mipi-dsi.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,amoled-mipi-dsi.yaml
@@ -33,7 +33,9 @@ properties:
# Samsung S6E3HF2 5.65" 1600x2560 AMOLED panel
- samsung,s6e3hf2

- reg: true
+ reg:
+ maxItems: 1
+
reset-gpios: true
enable-gpios: true
te-gpios: true
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.yaml
index 66d147496bc3..2af5bc47323f 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.yaml
@@ -16,8 +16,10 @@ properties:
compatible:
const: samsung,s6d16d0

+ reg:
+ maxItems: 1
+
port: true
- reg: true
reset-gpios: true

vdd1-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6d7aa0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6d7aa0.yaml
index 45a236d2cc70..939da65114bf 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,s6d7aa0.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6d7aa0.yaml
@@ -24,7 +24,8 @@ properties:
- samsung,ltl101at01
- const: samsung,s6d7aa0

- reg: true
+ reg:
+ maxItems: 1

backlight:
description:
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e88a0-ams452ef01.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e88a0-ams452ef01.yaml
index b749e9e906b7..42634fc3b5b2 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,s6e88a0-ams452ef01.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e88a0-ams452ef01.yaml
@@ -15,7 +15,10 @@ allOf:
properties:
compatible:
const: samsung,s6e88a0-ams452ef01
- reg: true
+
+ reg:
+ maxItems: 1
+
port: true
reset-gpios: true
vdd3-supply:
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
index 200fbf1c74a0..4601fa460680 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
@@ -16,7 +16,9 @@ properties:
compatible:
const: samsung,s6e8aa0

- reg: true
+ reg:
+ maxItems: 1
+
reset-gpios: true
display-timings: true

diff --git a/Documentation/devicetree/bindings/display/panel/sharp,lq101r1sx01.yaml b/Documentation/devicetree/bindings/display/panel/sharp,lq101r1sx01.yaml
index 57b44a0e763d..ce820b96a7e2 100644
--- a/Documentation/devicetree/bindings/display/panel/sharp,lq101r1sx01.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sharp,lq101r1sx01.yaml
@@ -37,7 +37,9 @@ properties:
- enum:
- sharp,lq101r1sx01

- reg: true
+ reg:
+ maxItems: 1
+
power-supply: true
backlight: true

diff --git a/Documentation/devicetree/bindings/display/panel/sharp,ls043t1le01.yaml b/Documentation/devicetree/bindings/display/panel/sharp,ls043t1le01.yaml
index a90d0d8bf7c9..b6ea246430ce 100644
--- a/Documentation/devicetree/bindings/display/panel/sharp,ls043t1le01.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sharp,ls043t1le01.yaml
@@ -16,7 +16,9 @@ properties:
compatible:
const: sharp,ls043t1le01-qhd

- reg: true
+ reg:
+ maxItems: 1
+
backlight: true
reset-gpios: true
port: true
diff --git a/Documentation/devicetree/bindings/display/panel/sharp,ls060t1sx01.yaml b/Documentation/devicetree/bindings/display/panel/sharp,ls060t1sx01.yaml
index 271c097cc9a4..77a4fce129e7 100644
--- a/Documentation/devicetree/bindings/display/panel/sharp,ls060t1sx01.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sharp,ls060t1sx01.yaml
@@ -16,7 +16,9 @@ properties:
compatible:
const: sharp,ls060t1sx01

- reg: true
+ reg:
+ maxItems: 1
+
backlight: true
reset-gpios: true
port: true
diff --git a/Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml b/Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml
index 059cc6dbcfca..fd778a20f760 100644
--- a/Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml
@@ -22,7 +22,10 @@ properties:
enum:
- sony,acx424akp
- sony,acx424akm
- reg: true
+
+ reg:
+ maxItems: 1
+
reset-gpios: true
vddi-supply:
description: regulator that supplies the vddi voltage
diff --git a/Documentation/devicetree/bindings/display/panel/sony,td4353-jdi.yaml b/Documentation/devicetree/bindings/display/panel/sony,td4353-jdi.yaml
index 07bce556ad40..191b692125e1 100644
--- a/Documentation/devicetree/bindings/display/panel/sony,td4353-jdi.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sony,td4353-jdi.yaml
@@ -20,7 +20,8 @@ properties:
compatible:
const: sony,td4353-jdi-tama

- reg: true
+ reg:
+ maxItems: 1

backlight: true
width-mm: true
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 967972939598..a58a31349757 100644
--- a/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
+++ b/Documentation/devicetree/bindings/display/panel/sony,tulip-truly-nt35521.yaml
@@ -21,7 +21,8 @@ properties:
compatible:
const: sony,tulip-truly-nt35521

- reg: true
+ reg:
+ maxItems: 1

positive5-supply:
description: Positive 5V supply
diff --git a/Documentation/devicetree/bindings/display/panel/synaptics,r63353.yaml b/Documentation/devicetree/bindings/display/panel/synaptics,r63353.yaml
index e5617d125567..2fd6e0ec3682 100644
--- a/Documentation/devicetree/bindings/display/panel/synaptics,r63353.yaml
+++ b/Documentation/devicetree/bindings/display/panel/synaptics,r63353.yaml
@@ -19,15 +19,17 @@ properties:
- sharp,ls068b3sx02
- const: syna,r63353

+ reg:
+ maxItems: 1
+
avdd-supply: true
dvdd-supply: true
- reg: true

required:
- compatible
+ - reg
- avdd-supply
- dvdd-supply
- - reg
- reset-gpios
- port
- backlight
diff --git a/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml
index 772399067515..30047a62fc11 100644
--- a/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml
+++ b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml
@@ -20,7 +20,8 @@ properties:
compatible:
const: visionox,rm69299-1080p-display

- reg: true
+ reg:
+ maxItems: 1

vdda-supply:
description: |
diff --git a/Documentation/devicetree/bindings/display/panel/xinpeng,xpp055c272.yaml b/Documentation/devicetree/bindings/display/panel/xinpeng,xpp055c272.yaml
index c407deb6afb1..9c9743a23500 100644
--- a/Documentation/devicetree/bindings/display/panel/xinpeng,xpp055c272.yaml
+++ b/Documentation/devicetree/bindings/display/panel/xinpeng,xpp055c272.yaml
@@ -15,7 +15,10 @@ allOf:
properties:
compatible:
const: xinpeng,xpp055c272
- reg: true
+
+ reg:
+ maxItems: 1
+
backlight: true
port: true
reset-gpios: true

--
2.43.0


2024-05-11 12:57:37

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 3/3] dt-bindings: display: panel: constrain 'reg' in DSI panels

On Thu, May 09, 2024 at 11:42:53AM +0200, Krzysztof Kozlowski wrote:
> DSI-attached devices could respond to more than one virtual channel
> number, thus their bindings are supposed to constrain the 'reg' property
> to match hardware. Add missing 'reg' constrain for DSI-attached display
> panels, based on DTS sources in Linux kernel (assume all devices take
> only one channel number).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Acked-by: Conor Dooley <[email protected]>

Thanks,
Conor.


Attachments:
(No filename) (544.00 B)
signature.asc (235.00 B)
Download all attachments

2024-05-11 12:58:50

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 2/3] dt-bindings: display: panel: constrain 'reg' in SPI panels

On Thu, May 09, 2024 at 11:42:52AM +0200, Krzysztof Kozlowski wrote:
> SPI-attached devices could have more than one chip-select, thus their
> bindings are supposed to constrain the 'reg' property to match hardware.
> Add missing 'reg' constrain for SPI-attached display panels.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Acked-by: Conor Dooley <[email protected]>

Cheers,
Conor.


Attachments:
(No filename) (432.00 B)
signature.asc (235.00 B)
Download all attachments

2024-05-11 12:59:17

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: display: samsung,ams495qa01: add missing SPI properties ref

On Thu, May 09, 2024 at 11:42:51AM +0200, Krzysztof Kozlowski wrote:
> Samsung AMS495QA01 panel is a SPI device, so it should reference
> spi-peripheral-props.yaml schema to allow and validate the SPI device
> properties.
>
> Fixes: 92be07c65b22 ("dt-bindings: display: panel: Add Samsung AMS495QA01")
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Acked-by: Conor Dooley <[email protected]>

Cheers,
Conor.


Attachments:
(No filename) (454.00 B)
signature.asc (235.00 B)
Download all attachments

2024-05-13 13:18:20

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

On Thu, May 09, 2024 at 11:42:50AM +0200, Krzysztof Kozlowski wrote:
> Hi,
>
> Cleanups for display panel bindings.
>
> Rob, maybe you could take entire set if it applies? I based it on
> linux-next, so letl me know if I need to rebase on your for-next.

Applied. These 2 don't exist in my tree:

Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml

Rob

2024-05-13 16:42:58

by Dmitry Baryshkov

[permalink] [raw]
Subject: Re: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

On Mon, 13 May 2024 at 16:17, Rob Herring <[email protected]> wrote:
>
> On Thu, May 09, 2024 at 11:42:50AM +0200, Krzysztof Kozlowski wrote:
> > Hi,
> >
> > Cleanups for display panel bindings.
> >
> > Rob, maybe you could take entire set if it applies? I based it on
> > linux-next, so letl me know if I need to rebase on your for-next.
>
> Applied. These 2 don't exist in my tree:

It's most likely fine, but was there an ack from drm-misc maintainers?

> Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
> Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml

Because those were added to drm-misc during the last cycle. So ideally
the patch should have gone through drm-misc.

--
With best wishes
Dmitry

2024-05-13 19:48:16

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 3/3] dt-bindings: display: panel: constrain 'reg' in DSI panels

On Thu, May 9, 2024 at 11:43 AM Krzysztof Kozlowski
<[email protected]> wrote:

> DSI-attached devices could respond to more than one virtual channel
> number, thus their bindings are supposed to constrain the 'reg' property
> to match hardware. Add missing 'reg' constrain for DSI-attached display
> panels, based on DTS sources in Linux kernel (assume all devices take
> only one channel number).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Looks right to me.
Reviewed-by: Linus Walleij <[email protected]>

Yours,
Linus Walleij

2024-05-13 20:36:52

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 2/3] dt-bindings: display: panel: constrain 'reg' in SPI panels

On Thu, May 9, 2024 at 11:43 AM Krzysztof Kozlowski
<[email protected]> wrote:

> SPI-attached devices could have more than one chip-select, thus their
> bindings are supposed to constrain the 'reg' property to match hardware.
> Add missing 'reg' constrain for SPI-attached display panels.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Linus Walleij <[email protected]>

Yours,
Linus Walleij

2024-05-14 08:44:38

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

On 13/05/2024 18:41, Dmitry Baryshkov wrote:
> On Mon, 13 May 2024 at 16:17, Rob Herring <[email protected]> wrote:
>>
>> On Thu, May 09, 2024 at 11:42:50AM +0200, Krzysztof Kozlowski wrote:
>>> Hi,
>>>
>>> Cleanups for display panel bindings.
>>>
>>> Rob, maybe you could take entire set if it applies? I based it on
>>> linux-next, so letl me know if I need to rebase on your for-next.
>>
>> Applied. These 2 don't exist in my tree:
>
> It's most likely fine, but was there an ack from drm-misc maintainers?
>
>> Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
>> Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml
>
> Because those were added to drm-misc during the last cycle. So ideally
> the patch should have gone through drm-misc.
>

Exact there's a conflict on today's next, Rob can you drop them so I can apply them via drm-misc ?

Thanks,
Neil

2024-05-14 09:07:28

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

On 14/05/2024 10:44, Neil Armstrong wrote:
> On 13/05/2024 18:41, Dmitry Baryshkov wrote:
>> On Mon, 13 May 2024 at 16:17, Rob Herring <[email protected]> wrote:
>>>
>>> On Thu, May 09, 2024 at 11:42:50AM +0200, Krzysztof Kozlowski wrote:
>>>> Hi,
>>>>
>>>> Cleanups for display panel bindings.
>>>>
>>>> Rob, maybe you could take entire set if it applies? I based it on
>>>> linux-next, so letl me know if I need to rebase on your for-next.
>>>
>>> Applied. These 2 don't exist in my tree:
>>
>> It's most likely fine, but was there an ack from drm-misc maintainers?
>>
>>> Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
>>> Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml
>>
>> Because those were added to drm-misc during the last cycle. So ideally
>> the patch should have gone through drm-misc.
>>
>
> Exact there's a conflict on today's next, Rob can you drop them so I can apply them via drm-misc ?

It's almost the first time I see bindings picked up via drm-misc. Is
this an exception or rather new trend (which would be awesome as this is
what we prefer usually)?

Best regards,
Krzysztof


2024-05-15 21:10:11

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

On 14/05/2024 11:07, Krzysztof Kozlowski wrote:
> On 14/05/2024 10:44, Neil Armstrong wrote:
>> On 13/05/2024 18:41, Dmitry Baryshkov wrote:
>>> On Mon, 13 May 2024 at 16:17, Rob Herring <[email protected]> wrote:
>>>>
>>>> On Thu, May 09, 2024 at 11:42:50AM +0200, Krzysztof Kozlowski wrote:
>>>>> Hi,
>>>>>
>>>>> Cleanups for display panel bindings.
>>>>>
>>>>> Rob, maybe you could take entire set if it applies? I based it on
>>>>> linux-next, so letl me know if I need to rebase on your for-next.
>>>>
>>>> Applied. These 2 don't exist in my tree:
>>>
>>> It's most likely fine, but was there an ack from drm-misc maintainers?
>>>
>>>> Documentation/devicetree/bindings/display/panel/lg,sw43408.yaml
>>>> Documentation/devicetree/bindings/display/panel/raydium,rm69380.yaml
>>>
>>> Because those were added to drm-misc during the last cycle. So ideally
>>> the patch should have gone through drm-misc.
>>>
>>
>> Exact there's a conflict on today's next, Rob can you drop them so I can apply them via drm-misc ?
>
> It's almost the first time I see bindings picked up via drm-misc. Is
> this an exception or rather new trend (which would be awesome as this is
> what we prefer usually)?

I usually pick up bindings along drivers like other subsystems, and when reviewed I
take independent bindings fixes aswell when rob doesn't pick them before.

Neil

>
> Best regards,
> Krzysztof
>