ARM: dts: stm32: Remove buck3 regulator-always-on on AV96

In case the regulator-always-on is present in regulator DT node,
the regulator is always reconfigured to the voltage set in DT on
probe, even if regulator_set_value() has been called before. Drop
the property from AV96 U-Boot DT and enable the regulator manually
in code, as the board already reconfigures the Buck3 regulator in
code per PMIC NVM content instead.

Fixes: 0adf10a87b1 ("ARM: dts: stm32: Configure Buck3 voltage per PMIC NVM on Avenger96")
Signed-off-by: Marek Vasut <marex@denx.de>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

diff --git a/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi b/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
index 5bed53e3fdb91756dafe9c04cfde87a14161ba94..b6fceb9ad5813002fb0b68407e4d2fc79d3f9e55 100644 (file)
--- a/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
@@ -102,6 +102,10 @@
        hnp-srp-disable;
 };
 
+&vdd {
+       /delete-property/ regulator-always-on;
+};
+
 &vdd_io {
        u-boot,dm-spl;
 };

diff --git a/board/dhelectronics/dh_stm32mp1/board.c b/board/dhelectronics/dh_stm32mp1/board.c
index 2bc0d7b943a1a817908fe877b53cc7a1321154ba..05f884cfbfd9784ea9bc8817b6cbe7c0870691b6 100644 (file)
--- a/board/dhelectronics/dh_stm32mp1/board.c
+++ b/board/dhelectronics/dh_stm32mp1/board.c
@@ -595,6 +595,7 @@ static void board_init_regulator_av96(void)
 
        /* Adjust Buck3 per preconfigured PMIC voltage from NVM. */
        regulator_set_value(rdev, uv);
+       regulator_set_enable(rdev, true);
 }
 
 static void board_init_regulator(void)