diff mbox series

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

Message ID 20250214092634.12414-1-francesco@dolcini.it (mailing list archive)
State New
Headers show
Series [v1] arm64: dts: freescale: imx8mm-verdin: Remove LVDS panel and backlight | expand

Commit Message

Francesco Dolcini Feb. 14, 2025, 9:26 a.m. UTC
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(-)

Comments

Frank Li Feb. 14, 2025, 7:54 p.m. UTC | #1
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
>
diff mbox series

Patch

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";