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

[Frank Li]

On Fri, Feb 14, 2025 at 10:26:34AM +0100, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
>
> 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@xxxxxxxxxxx>
> ---

Thank you very much.

Reviewed-by: Frank Li <Frank.Li@xxxxxxx>

>  .../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
>