mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: beacon-renesom-baseboard: Remove LVDS Panel
@ 2026-03-19 13:29 Adam Ford
  2026-03-19 14:57 ` Marek Vasut
  2026-03-25 16:14 ` Geert Uytterhoeven
  0 siblings, 2 replies; 3+ messages in thread
From: Adam Ford @ 2026-03-19 13:29 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: marek.vasut, Adam Ford, Geert Uytterhoeven, Magnus Damm,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, devicetree,
	linux-kernel

The LVDS Panel was never shipped from Beacon, and there are device
tree errors, so rather than trying to fix them, remove it instead.

Signed-off-by: Adam Ford <aford173@gmail.com>

diff --git a/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi b/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
index d55f2d7066ad..dcc98e8424f7 100644
--- a/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
+++ b/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
@@ -101,38 +101,6 @@ led3 {
 		};
 	};
 
-	lvds {
-		compatible = "panel-lvds";
-		power-supply = <&reg_lcd_reset>;
-		width-mm = <223>;
-		height-mm = <125>;
-		backlight = <&backlight_lvds>;
-		data-mapping = "vesa-24";
-
-		panel-timing {
-			/* 800x480@60Hz */
-			clock-frequency = <30000000>;
-			hactive = <800>;
-			vactive = <480>;
-			hsync-len = <48>;
-			hfront-porch = <40>;
-			hback-porch = <40>;
-			vfront-porch = <13>;
-			vback-porch = <29>;
-			vsync-len = <1>;
-			hsync-active = <1>;
-			vsync-active = <3>;
-			de-active = <1>;
-			pixelclk-active = <0>;
-		};
-
-		port {
-			panel_in: endpoint {
-				remote-endpoint = <&lvds0_out>;
-			};
-		};
-	};
-
 	rgb {
 		/* Different LCD with compatible timings */
 		compatible = "rocktech,rk070er9427";
@@ -480,18 +448,6 @@ gpio_exp1: gpio@70 {
 	};
 };
 
-&lvds0 {
-	status = "okay";
-
-	ports {
-		port@1 {
-			lvds0_out: endpoint {
-				remote-endpoint = <&panel_in>;
-			};
-		};
-	};
-};
-
 &msiof1 {
 	pinctrl-0 = <&msiof1_pins>;
 	pinctrl-names = "default";
-- 
2.51.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: beacon-renesom-baseboard: Remove LVDS Panel
  2026-03-19 13:29 [PATCH] arm64: dts: beacon-renesom-baseboard: Remove LVDS Panel Adam Ford
@ 2026-03-19 14:57 ` Marek Vasut
  2026-03-25 16:14 ` Geert Uytterhoeven
  1 sibling, 0 replies; 3+ messages in thread
From: Marek Vasut @ 2026-03-19 14:57 UTC (permalink / raw)
  To: Adam Ford, linux-renesas-soc
  Cc: Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

On 3/19/26 2:29 PM, Adam Ford wrote:
> The LVDS Panel was never shipped from Beacon, and there are device
> tree errors, so rather than trying to fix them, remove it instead.
> 
> Signed-off-by: Adam Ford <aford173@gmail.com>
Reviewed-by: Marek Vasut <marek.vasut+renesas@mailbox.org>

Thank you ! (and good bye another DT checker warning)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: beacon-renesom-baseboard: Remove LVDS Panel
  2026-03-19 13:29 [PATCH] arm64: dts: beacon-renesom-baseboard: Remove LVDS Panel Adam Ford
  2026-03-19 14:57 ` Marek Vasut
@ 2026-03-25 16:14 ` Geert Uytterhoeven
  1 sibling, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2026-03-25 16:14 UTC (permalink / raw)
  To: Adam Ford
  Cc: linux-renesas-soc, marek.vasut, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

Hi Adam,

On Thu, 19 Mar 2026 at 14:29, Adam Ford <aford173@gmail.com> wrote:
> The LVDS Panel was never shipped from Beacon, and there are device
> tree errors, so rather than trying to fix them, remove it instead.
>
> Signed-off-by: Adam Ford <aford173@gmail.com>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
> +++ b/arch/arm64/boot/dts/renesas/beacon-renesom-baseboard.dtsi
> @@ -101,38 +101,6 @@ led3 {
>                 };
>         };
>
> -       lvds {
> -               compatible = "panel-lvds";
> -               power-supply = <&reg_lcd_reset>;

Shouldn't you remove the reg_lcd_reset...

> -               width-mm = <223>;
> -               height-mm = <125>;
> -               backlight = <&backlight_lvds>;

... and backlight_lvds nodes, too?

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-03-25 16:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-03-19 13:29 [PATCH] arm64: dts: beacon-renesom-baseboard: Remove LVDS Panel Adam Ford
2026-03-19 14:57 ` Marek Vasut
2026-03-25 16:14 ` Geert Uytterhoeven

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®