diff mbox series

ARM: dts: stm32: Move DHCOR BUCK3 VDD 2V9 adjustment to 1V8 DTSI

Message ID 20220504124945.77331-1-marex@denx.de (mailing list archive)
State New, archived
Headers show
Series ARM: dts: stm32: Move DHCOR BUCK3 VDD 2V9 adjustment to 1V8 DTSI | expand

Commit Message

Marek Vasut May 4, 2022, 12:49 p.m. UTC
The Buck3 on DHCOR is used to supply IO voltage. It can output either 3V3
in the default DHCOR configuration, or 2V9 in case of AV96 DHCOR variant
which has extra Empirion DCDC converter in front of the 1V8 IO supply, or
outright 1V8 in case of 1V8 IO DHCOR without the Empirion DCDC converter.

The 2V9 mode in case of AV96 DHCOR variant is used to reduce unnecessarily
high input voltage to the Empirion DCDC converter, so move it into matching
DTSI to stop confusing users.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
Cc: linux-stm32@st-md-mailman.stormreply.com
To: linux-arm-kernel@lists.infradead.org
---
 arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi | 5 +++++
 arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi   | 4 ++--
 2 files changed, 7 insertions(+), 2 deletions(-)

Comments

Alexandre TORGUE June 13, 2022, 7:42 a.m. UTC | #1
Hi Marek

On 5/4/22 14:49, Marek Vasut wrote:
> The Buck3 on DHCOR is used to supply IO voltage. It can output either 3V3
> in the default DHCOR configuration, or 2V9 in case of AV96 DHCOR variant
> which has extra Empirion DCDC converter in front of the 1V8 IO supply, or
> outright 1V8 in case of 1V8 IO DHCOR without the Empirion DCDC converter.
> 
> The 2V9 mode in case of AV96 DHCOR variant is used to reduce unnecessarily
> high input voltage to the Empirion DCDC converter, so move it into matching
> DTSI to stop confusing users.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> To: linux-arm-kernel@lists.infradead.org
> ---
>   arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi | 5 +++++
>   arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi   | 4 ++--
>   2 files changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi
> index 75172314d7afd..9937b28548c23 100644
> --- a/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi
> @@ -18,6 +18,11 @@ vdd_io: regulator-buck-io {
>   	};
>   };
>   
> +&vdd {
> +	regulator-min-microvolt = <2900000>;
> +	regulator-max-microvolt = <2900000>;
> +};
> +
>   &pwr_regulators {
>   	vdd-supply = <&vdd_io>;
>   };
> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
> index 6336c3ca0f0e2..134a798ad3f23 100644
> --- a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
> @@ -119,8 +119,8 @@ vdd_ddr: buck2 {
>   
>   			vdd: buck3 {
>   				regulator-name = "vdd";
> -				regulator-min-microvolt = <2900000>;
> -				regulator-max-microvolt = <2900000>;
> +				regulator-min-microvolt = <3300000>;
> +				regulator-max-microvolt = <3300000>;
>   				regulator-always-on;
>   				regulator-initial-mode = <0>;
>   				regulator-over-current-protection;

Applied on stm32-next.

Thanks.
Alex
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi
index 75172314d7afd..9937b28548c23 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-io1v8.dtsi
@@ -18,6 +18,11 @@  vdd_io: regulator-buck-io {
 	};
 };
 
+&vdd {
+	regulator-min-microvolt = <2900000>;
+	regulator-max-microvolt = <2900000>;
+};
+
 &pwr_regulators {
 	vdd-supply = <&vdd_io>;
 };
diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
index 6336c3ca0f0e2..134a798ad3f23 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
@@ -119,8 +119,8 @@  vdd_ddr: buck2 {
 
 			vdd: buck3 {
 				regulator-name = "vdd";
-				regulator-min-microvolt = <2900000>;
-				regulator-max-microvolt = <2900000>;
+				regulator-min-microvolt = <3300000>;
+				regulator-max-microvolt = <3300000>;
 				regulator-always-on;
 				regulator-initial-mode = <0>;
 				regulator-over-current-protection;