[<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 = <®_vcc3v3>;
> + vcc-supply = <&buck4_reg>;
> };
>
> m24c64: eeprom@57 {
> compatible = "atmel,24c64";
> reg = <0x57>;
> pagesize = <32>;
> - vcc-supply = <®_vcc3v3>;
> + vcc-supply = <&buck4_reg>;
> };
> };
>
> @@ -211,7 +202,7 @@ &usdhc3 {
> non-removable;
> no-sd;
> no-sdio;
> - vmmc-supply = <®_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