[PATCH v1] arm64: dts: freescale: imx8mm-verdin: Remove LVDS panel and backlight

Francesco Dolcini posted 1 patch 3 weeks, 5 days ago
.../boot/dts/freescale/imx8mm-verdin.dtsi     | 21 -------------------
1 file changed, 21 deletions(-)
[PATCH v1] arm64: dts: freescale: imx8mm-verdin: Remove LVDS panel and backlight
Posted by Francesco Dolcini 3 weeks, 5 days ago
From: Francesco Dolcini <francesco.dolcini@toradex.com>

Remove LVDS panel and backlight nodes from the Verdin iMX8M Mini SoM
dtsi file, those two hardware components are not part of the SoM,
therefore they should not be present in this file.

This is solving a dtb checker warning about panel-lvds compatible.

Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
---
 .../boot/dts/freescale/imx8mm-verdin.dtsi     | 21 -------------------
 1 file changed, 21 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi
index c528594ac442..7251ad3a0017 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi
@@ -18,20 +18,6 @@ aliases {
 		rtc1 = &snvs_rtc;
 	};
 
-	backlight: backlight {
-		compatible = "pwm-backlight";
-		brightness-levels = <0 45 63 88 119 158 203 255>;
-		default-brightness-level = <4>;
-		/* Verdin I2S_2_D_OUT (DSI_1_BKL_EN/DSI_1_BKL_EN_LVDS, SODIMM 46) */
-		enable-gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>;
-		pinctrl-names = "default";
-		pinctrl-0 = <&pinctrl_i2s_2_d_out_dsi_1_bkl_en>;
-		power-supply = <&reg_3p3v>;
-		/* Verdin PWM_3_DSI/PWM_3_DSI_LVDS (SODIMM 19) */
-		pwms = <&pwm1 0 6666667 PWM_POLARITY_INVERTED>;
-		status = "disabled";
-	};
-
 	/* Fixed clock dedicated to SPI CAN controller */
 	clk40m: oscillator {
 		compatible = "fixed-clock";
@@ -66,13 +52,6 @@ hdmi_connector: hdmi-connector {
 		status = "disabled";
 	};
 
-	panel_lvds: panel-lvds {
-		compatible = "panel-lvds";
-		backlight = <&backlight>;
-		data-mapping = "vesa-24";
-		status = "disabled";
-	};
-
 	/* Carrier Board Supplies */
 	reg_1p8v: regulator-1p8v {
 		compatible = "regulator-fixed";
-- 
2.39.5
Re: [PATCH v1] arm64: dts: freescale: imx8mm-verdin: Remove LVDS panel and backlight
Posted by Shawn Guo 2 weeks, 3 days ago
On Fri, Feb 14, 2025 at 10:26:34AM +0100, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
> 
> Remove LVDS panel and backlight nodes from the Verdin iMX8M Mini SoM
> dtsi file, those two hardware components are not part of the SoM,
> therefore they should not be present in this file.
> 
> This is solving a dtb checker warning about panel-lvds compatible.
> 
> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>

Applied, thanks!
Re: [PATCH v1] arm64: dts: freescale: imx8mm-verdin: Remove LVDS panel and backlight
Posted by Frank Li 3 weeks, 5 days ago
On Fri, Feb 14, 2025 at 10:26:34AM +0100, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
>
> Remove LVDS panel and backlight nodes from the Verdin iMX8M Mini SoM
> dtsi file, those two hardware components are not part of the SoM,
> therefore they should not be present in this file.
>
> This is solving a dtb checker warning about panel-lvds compatible.
>
> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
> ---

Thank you very much.

Reviewed-by: Frank Li <Frank.Li@nxp.com>

>  .../boot/dts/freescale/imx8mm-verdin.dtsi     | 21 -------------------
>  1 file changed, 21 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi
> index c528594ac442..7251ad3a0017 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-verdin.dtsi
> @@ -18,20 +18,6 @@ aliases {
>  		rtc1 = &snvs_rtc;
>  	};
>
> -	backlight: backlight {
> -		compatible = "pwm-backlight";
> -		brightness-levels = <0 45 63 88 119 158 203 255>;
> -		default-brightness-level = <4>;
> -		/* Verdin I2S_2_D_OUT (DSI_1_BKL_EN/DSI_1_BKL_EN_LVDS, SODIMM 46) */
> -		enable-gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>;
> -		pinctrl-names = "default";
> -		pinctrl-0 = <&pinctrl_i2s_2_d_out_dsi_1_bkl_en>;
> -		power-supply = <&reg_3p3v>;
> -		/* Verdin PWM_3_DSI/PWM_3_DSI_LVDS (SODIMM 19) */
> -		pwms = <&pwm1 0 6666667 PWM_POLARITY_INVERTED>;
> -		status = "disabled";
> -	};
> -
>  	/* Fixed clock dedicated to SPI CAN controller */
>  	clk40m: oscillator {
>  		compatible = "fixed-clock";
> @@ -66,13 +52,6 @@ hdmi_connector: hdmi-connector {
>  		status = "disabled";
>  	};
>
> -	panel_lvds: panel-lvds {
> -		compatible = "panel-lvds";
> -		backlight = <&backlight>;
> -		data-mapping = "vesa-24";
> -		status = "disabled";
> -	};
> -
>  	/* Carrier Board Supplies */
>  	reg_1p8v: regulator-1p8v {
>  		compatible = "regulator-fixed";
> --
> 2.39.5
>