linux-kernel - Re: [PATCH 1/2] arm64: dts: imx93-11x11-evk: Add PWM backlight for "LVDS" connector

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 for Android: free password hash cracker in your pocket

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

Message-ID: <Zs3tJzt7sdvK8yeu@lizhi-Precision-Tower-5810>
Date: Tue, 27 Aug 2024 11:13:43 -0400
From: Frank Li <Frank.li@....com>
To: Liu Ying <victor.liu@....com>
Cc: devicetree@...r.kernel.org, imx@...ts.linux.dev,
	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
	robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
	shawnguo@...nel.org, s.hauer@...gutronix.de, kernel@...gutronix.de,
	festevam@...il.com, catalin.marinas@....com, will@...nel.org,
	quic_bjorande@...cinc.com, geert+renesas@...der.be,
	dmitry.baryshkov@...aro.org, neil.armstrong@...aro.org,
	arnd@...db.de, nfraprado@...labora.com
Subject: Re: [PATCH 1/2] arm64: dts: imx93-11x11-evk: Add PWM backlight for
 "LVDS" connector

On Tue, Aug 27, 2024 at 02:47:16PM +0800, Liu Ying wrote:
> J702 on i.MX93 11x11 EVK base board is a 40-pin connector labelled
> "LVDS".  This connector supports PWM backlight with default 12V power
> supply and ADP5585 PWM controller.  Add support for the backlight.
>
> Signed-off-by: Liu Ying <victor.liu@....com>
> ---
> This patch is based on next-20240826.
>
>  .../boot/dts/freescale/imx93-11x11-evk.dts    | 30 +++++++++++++++++++
>  1 file changed, 30 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts b/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts
> index 60eb64761392..8d036b3962e9 100644
> --- a/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts
> +++ b/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts
> @@ -62,6 +62,15 @@ vdevbuffer: vdevbuffer@...20000 {
>
>  	};
>
> +	reg_vdd_12v: regulator-vdd-12v {
> +		compatible = "regulator-fixed";
> +		regulator-name = "VDD_12V";
> +		regulator-min-microvolt = <12000000>;
> +		regulator-max-microvolt = <12000000>;
> +		gpio = <&pcal6524 14 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +	};
> +
>  	reg_vref_1v8: regulator-adc-vref {
>  		compatible = "regulator-fixed";
>  		regulator-name = "vref_1v8";
> @@ -81,6 +90,17 @@ reg_usdhc2_vmmc: regulator-usdhc2 {
>  		enable-active-high;
>  	};
>
> +	backlight_lvds: backlight-lvds {
> +		compatible = "pwm-backlight";
> +		pwms = <&adp5585 0 100000 0>;
> +		brightness-levels = <0 100>;
> +		num-interpolated-steps = <100>;
> +		default-brightness-level = <100>;
> +		power-supply = <&reg_vdd_12v>;
> +		enable-gpios = <&adp5585 9 GPIO_ACTIVE_HIGH>;
> +		status = "disabled";
> +	};
> +
>  	bt_sco_codec: bt-sco-codec {
>  		compatible = "linux,bt-sco";
>  		#sound-dai-cells = <1>;
> @@ -302,6 +322,16 @@ ldo5: LDO5 {
>  			};
>  		};
>  	};
> +
> +	adp5585: io-expander@34 {
> +		compatible = "adi,adp5585-00", "adi,adp5585";
> +		reg = <0x34>;
> +		vdd-supply = <&buck4>;
> +		gpio-controller;
> +		#gpio-cells = <2>;
> +		gpio-reserved-ranges = <5 1>;
> +		#pwm-cells = <3>;
> +	};

This part will be conflict with
https://lore.kernel.org/imx/172476183829.3553281.6314012832354346473.robh@kernel.org/T/#m5e2189badf00653200421880c209312fc0aba7f3

Can I merge this to one set?

Frank

>  };
>
>  &lpi2c3 {
> --
> 2.34.1
>