2024-06-05 10:57:57

by Krzysztof Kozlowski

[permalink] [raw]

Subject: [PATCH] dt-bindings: display: panel: constrain 'reg' in DSI panels (part two)

---

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).

Few bindings missed previous fixup: LG SW43408 and Raydium RM69380.

Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

This should apply on any tree.
---
.../devicetree/bindings/display/panel/lg,sw43408.yaml | 4 +++-
.../devicetree/bindings/display/panel/raydium,rm69380.yaml | 5 +++--
2 files changed, 6 insertions(+), 3 deletions(-)

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/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
--
2.43.0

2024-06-05 16:56:13

by Conor Dooley

[permalink] [raw]

Subject: Re: [PATCH] dt-bindings: display: panel: constrain 'reg' in DSI panels (part two)

---

On Wed, Jun 05, 2024 at 12:56:59PM +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).
>
> Few bindings missed previous fixup: LG SW43408 and Raydium RM69380.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

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

Thanks,
Conor.

---

Attachments:

(No filename) (619.00 B)
signature.asc (235.00 B)
Download all attachments

2024-06-06 20:55:15

by Rob Herring (Arm)

[permalink] [raw]

Subject: Re: [PATCH] dt-bindings: display: panel: constrain 'reg' in DSI panels (part two)

---

On Wed, 05 Jun 2024 12:56:59 +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).
>
> Few bindings missed previous fixup: LG SW43408 and Raydium RM69380.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> This should apply on any tree.
> ---
> .../devicetree/bindings/display/panel/lg,sw43408.yaml | 4 +++-
> .../devicetree/bindings/display/panel/raydium,rm69380.yaml | 5 +++--
> 2 files changed, 6 insertions(+), 3 deletions(-)
>

Applied, thanks!