diff mbox series

[v8,3/4] arm64: dts: renesas: rzg2l-smarc: Enable DU and link with DSI

Message ID 20230424163908.137535-4-biju.das.jz@bp.renesas.com
State New
Headers show
Series Enable RZ/{G2L,G2LC} and RZ/V2L DU | expand

Commit Message

Biju Das April 24, 2023, 4:39 p.m. UTC 
Enable DU and link with DSI on RZ/{G2L,V2L} SMARC EVK.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
v7->v8:
 * No Change.
v7:
 * New patch.
---
 arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi | 21 ++++++++++++++++++++
 1 file changed, 21 insertions(+)

Comments

Geert Uytterhoeven May 3, 2023, 10:35 a.m. UTC | #1 
Hi Biju,

On Mon, Apr 24, 2023 at 6:39 PM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> Enable DU and link with DSI on RZ/{G2L,V2L} SMARC EVK.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
> @@ -49,6 +49,7 @@ ports {
>                 port@0 {
>                         reg = <0>;
>                         dsi0_in: endpoint {
> +                               remote-endpoint = <&du_out_dsi>;
>                         };
>                 };
>
> @@ -62,6 +63,26 @@ dsi0_out: endpoint {
>         };
>  };
>
> +&du {
> +       status = "okay";
> +
> +       ports {
> +               #address-cells = <1>;
> +               #size-cells = <0>;
> +
> +               port@0 {
> +                       reg = <0>;
> +                       du_out_dsi: endpoint {
> +                               remote-endpoint = <&dsi0_in>;
> +                       };
> +               };
> +
> +               port@1 {
> +                       reg = <1>;
> +               };

Do you need port@1 if it is not connected to anything?
If yes, don't the port@ subnodes and reg properties belong in the
SoC-specific .dtsi?

> +       };
> +};
> +
>  &i2c1 {
>         adv7535: hdmi@3d {
>                 compatible = "adi,adv7535";

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
index 2a158a954b2f..cd4f569df5cd 100644
--- a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
+++ b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
@@ -49,6 +49,7 @@  ports {
 		port@0 {
 			reg = <0>;
 			dsi0_in: endpoint {
+				remote-endpoint = <&du_out_dsi>;
 			};
 		};
 
@@ -62,6 +63,26 @@  dsi0_out: endpoint {
 	};
 };
 
+&du {
+	status = "okay";
+
+	ports {
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		port@0 {
+			reg = <0>;
+			du_out_dsi: endpoint {
+				remote-endpoint = <&dsi0_in>;
+			};
+		};
+
+		port@1 {
+			reg = <1>;
+		};
+	};
+};
+
 &i2c1 {
 	adv7535: hdmi@3d {
 		compatible = "adi,adv7535";