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: <4043f877-fc29-4d81-b96a-83f6ef3fa926@pengutronix.de>
Date: Thu, 31 Jul 2025 11:31:38 +0200
From: Ahmad Fatoum <a.fatoum@...gutronix.de>
To: Alexander Stein <alexander.stein@...tq-group.com>,
 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>
Cc: devicetree@...r.kernel.org, Markus Niebel
 <Markus.Niebel@...tq-group.com>, imx@...ts.linux.dev,
 linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux@...tq-group.com
Subject: Re: [PATCH 2/2] arm64: dts: imx8mp-tqma8mpql: remove virtual 3.3V
 regulator

Hi Alexander,

On 7/31/25 11:16, Alexander Stein wrote:
> From: Markus Niebel <Markus.Niebel@...tq-group.com>
> 
> BUCK4 rail supplies the 3.3V rail. Use the actual regulator
> instead of a virtual fixed regulator.
> 
> Signed-off-by: Markus Niebel <Markus.Niebel@...tq-group.com>
> Signed-off-by: Alexander Stein <alexander.stein@...tq-group.com>

jfyi, this may impact boot time as it delays eMMC probe until after the
PMIC on the slower I2C bus is registered. It would be cool if we could
annotate regulators as left-on by bootloader somehow to weaken
dependencies, but that's unrelated to your patch.

Cheers,
Ahmad

> ---
>  .../boot/dts/freescale/imx8mp-tqma8mpql.dtsi      | 15 +++------------
>  1 file changed, 3 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
> index b48d5da147273..9716f24f7c6ed 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
> @@ -16,15 +16,6 @@ memory@...00000 {
>  		reg = <0x0 0x40000000 0 0x80000000>;
>  	};
>  
> -	/* identical to buck4_reg, but should never change */
> -	reg_vcc3v3: regulator-vcc3v3 {
> -		compatible = "regulator-fixed";
> -		regulator-name = "VCC3V3";
> -		regulator-min-microvolt = <3300000>;
> -		regulator-max-microvolt = <3300000>;
> -		regulator-always-on;
> -	};
> -
>  	reg_usdhc2_vqmmc: regulator-usdhc2-vqmmc {
>  		compatible = "regulator-gpio";
>  		pinctrl-names = "default";
> @@ -187,14 +178,14 @@ at24c02: eeprom@53 {
>  		read-only;
>  		reg = <0x53>;
>  		pagesize = <16>;
> -		vcc-supply = <&reg_vcc3v3>;
> +		vcc-supply = <&buck4_reg>;
>  	};
>  
>  	m24c64: eeprom@57 {
>  		compatible = "atmel,24c64";
>  		reg = <0x57>;
>  		pagesize = <32>;
> -		vcc-supply = <&reg_vcc3v3>;
> +		vcc-supply = <&buck4_reg>;
>  	};
>  };
>  
> @@ -211,7 +202,7 @@ &usdhc3 {
>  	non-removable;
>  	no-sd;
>  	no-sdio;
> -	vmmc-supply = <&reg_vcc3v3>;
> +	vmmc-supply = <&buck4_reg>;
>  	vqmmc-supply = <&buck5_reg>;
>  	status = "okay";
>  };

-- 
Pengutronix e.K.                  |                             |
Steuerwalder Str. 21              | http://www.pengutronix.de/  |
31137 Hildesheim, Germany         | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ