linux-kernel - RE: [PATCH] ARM: dts: imx6qdl-dhcom-pdk2: Fill in missing panel power-supply

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <ba06badd3e524f7f8951802d8ebb999a@dh-electronics.com>
Date: Wed, 16 Oct 2024 08:32:17 +0000
From: Christoph Niedermaier <cniedermaier@...electronics.com>
To: Marek Vasut <marex@...x.de>, "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>
CC: Conor Dooley <conor+dt@...nel.org>, Fabio Estevam <festevam@...il.com>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Pengutronix Kernel Team
	<kernel@...gutronix.de>, Rob Herring <robh@...nel.org>, Sascha Hauer
	<s.hauer@...gutronix.de>, Shawn Guo <shawnguo@...nel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"imx@...ts.linux.dev" <imx@...ts.linux.dev>, kernel
	<kernel@...electronics.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] ARM: dts: imx6qdl-dhcom-pdk2: Fill in missing panel
 power-supply

From: Marek Vasut <marex@...x.de>
Sent: Wednesday, October 16, 2024 1:25 AM
> Add missing panel power-supply property to DT to fix the following warning.
> The power supply on this device is very simple chain of regulators from the
> main 24V input, describe those.
> 
> "
> arch/arm/boot/dts/nxp/imx/imx6q-dhcom-pdk2.dtb: panel: 'power-supply' is a required
> property
> "
> 
> Signed-off-by: Marek Vasut <marex@...x.de>
> ---
> Cc: Christoph Niedermaier <cniedermaier@...electronics.com>
> Cc: Conor Dooley <conor+dt@...nel.org>
> Cc: Fabio Estevam <festevam@...il.com>
> Cc: Krzysztof Kozlowski <krzk+dt@...nel.org>
> Cc: Pengutronix Kernel Team <kernel@...gutronix.de>
> Cc: Rob Herring <robh@...nel.org>
> Cc: Sascha Hauer <s.hauer@...gutronix.de>
> Cc: Shawn Guo <shawnguo@...nel.org>
> Cc: devicetree@...r.kernel.org
> Cc: imx@...ts.linux.dev
> Cc: kernel@...electronics.com
> Cc: linux-arm-kernel@...ts.infradead.org
> Cc: linux-kernel@...r.kernel.org
> ---
>  .../boot/dts/nxp/imx/imx6qdl-dhcom-pdk2.dtsi  | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-dhcom-pdk2.dtsi
> b/arch/arm/boot/dts/nxp/imx/imx6qdl-dhcom-pdk2.dtsi
> index 773fdcfcd0015..d7c2b30aecfd1 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx6qdl-dhcom-pdk2.dtsi
> +++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-dhcom-pdk2.dtsi
> @@ -143,6 +143,7 @@ led-8 {
>         panel {
>                 backlight = <&display_bl>;
>                 compatible = "edt,etm0700g0edh6";
> +               power-supply = <&reg_panel_3v3>;
> 
>                 port {
>                         lcd_panel_in: endpoint {
> @@ -151,6 +152,25 @@ lcd_panel_in: endpoint {
>                 };
>         };
> 
> +       /* Filtered supply voltage */
> +       reg_pdk2_24v: regulator-pdk2-24v {
> +               compatible = "regulator-fixed";
> +               regulator-always-on;
> +               regulator-max-microvolt = <24000000>;
> +               regulator-min-microvolt = <24000000>;
> +               regulator-name = "24V_PDK2";
> +       };
> +
> +       /* 560-200 U1 */
> +       reg_panel_3v3: regulator-panel-3v3 {
> +               compatible = "regulator-fixed";
> +               regulator-always-on;
> +               regulator-min-microvolt = <3300000>;
> +               regulator-max-microvolt = <3300000>;
> +               regulator-name = "3V3_PANEL";
> +               vin-supply = <&reg_pdk2_24v>;
> +       };
> +
>         sound {
>                 audio-codec = <&sgtl5000>;
>                 audio-routing =

Reviewed-by: Christoph Niedermaier <cniedermaier@...electronics.com>