Thread (17 messages) 17 messages, 4 authors, 8d ago

Re: [PATCH 01/16] dt-bindings: display: bridge: renesas,dsi: Document RZ/G3L

From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2026-07-12 15:24:37
Also in: dri-devel, linux-renesas-soc, lkml

On Wed, Jul 08, 2026 at 09:39:01AM +0000, Biju Das wrote:
Hi Krzysztof Kozlowski,

Thanks for the feedback.
quoted
-----Original Message-----
From: Krzysztof Kozlowski <krzk@kernel.org>
Sent: 08 July 2026 09:30
Subject: Re: [PATCH 01/16] dt-bindings: display: bridge: renesas,dsi: Document RZ/G3L

On Sat, Jul 04, 2026 at 10:34:11AM +0100, Biju wrote:
quoted
From: Biju Das <biju.das.jz@bp.renesas.com>

The RZ/G3L DSI IP is similar to the RZ/G2L but has different global
PHY timings and also the PLLCLK is ungateble clock. Add the compatible
string "renesas,r9a08g046-mipi-dsi" to handle these difference for the
Renesas RZ/G3L SoC. The power to DSI region is controlled by SYSC block.
Document renesas,sysc-pwrrdy property to handle the power control.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
 .../bindings/display/bridge/renesas,dsi.yaml      | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git
a/Documentation/devicetree/bindings/display/bridge/renesas,dsi.yaml
b/Documentation/devicetree/bindings/display/bridge/renesas,dsi.yaml
index c20625b8425e..b114ac3b111a 100644
---
a/Documentation/devicetree/bindings/display/bridge/renesas,dsi.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/renesas,dsi.yam
+++ l
@@ -28,6 +28,7 @@ properties:
           - const: renesas,r9a09g057-mipi-dsi

       - enum:
+          - renesas,r9a08g046-mipi-dsi # RZ/G3L
           - renesas,r9a09g057-mipi-dsi # RZ/V2H(P)

   reg:
@@ -108,6 +109,20 @@ properties:
   power-domains:
     maxItems: 1

+  renesas,sysc-pwrrdy:
+    description:
+      The system controller PWRRDY indicates to the DSI region, if the power
+      supply is ready. PWRRDY needs to be set during power-on before applying
+      any other settings. It also needs to be set before powering off the DSI.
+    $ref: /schemas/types.yaml#/definitions/phandle-array
This feels a lot like a power domain. Please elaborate what is PWRRDY and why power-on/off and power
status within SoC (important!) is not encoded as power domain.
We already tried modelling signal as power domain in RZ/G3S and finally Ulf
agreed that it cannot be power-domain[1]

" SYSC signal seems best to be modelled as a reset.
Although, it looks like the USB PM domain provider should rather be
the consumer of that reset, instead of having the reset being consumed
by the consumers of the USB PM domain."

Then Phillip proposed power sequencing driver[2] and finally he and Rob ok for the 
solution [3] 
The problem is that you did not implement or reference here power
sequencing. You created phandle without corresponding any hardware
signal and used "driver" as an argument.

power sequencing does not mean you can stuff random phandles here and
there.

Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help