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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <Z6+fjygpOeCUM9nd@lizhi-Precision-Tower-5810>
Date: Fri, 14 Feb 2025 14:54:55 -0500
From: Frank Li <Frank.li@....com>
To: Francesco Dolcini <francesco@...cini.it>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
	Sascha Hauer <s.hauer@...gutronix.de>,
	Pengutronix Kernel Team <kernel@...gutronix.de>,
	Fabio Estevam <festevam@...il.com>,
	Francesco Dolcini <francesco.dolcini@...adex.com>,
	devicetree@...r.kernel.org, imx@...ts.linux.dev,
	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1] arm64: dts: freescale: imx8mm-verdin: Remove LVDS
 panel and backlight

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

Thank you very much.

Reviewed-by: Frank Li <Frank.Li@....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
>