Hi Marek
On 5/4/22 14:50, 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 <ma...@denx.de>
> Cc: Patrice Chotard <patrice.chot...@foss.st.com>
> Cc: Patrick Delaunay <patrick.delau...@foss.st.com>
> ---
> arch/arm/dts/stm32mp15xx-dhcor-io1v8.dtsi | 5 +++++
> arch/arm/dts/stm32mp15xx-dhcor-som.dtsi | 4 ++--
> 2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/dts/stm32mp15xx-dhcor-io1v8.dtsi
> b/arch/arm/dts/stm32mp15xx-dhcor-io1v8.dtsi
> index 75172314d7a..9937b28548c 100644
> --- a/arch/arm/dts/stm32mp15xx-dhcor-io1v8.dtsi
> +++ b/arch/arm/dts/stm32mp15xx-dhcor-io1v8.dtsi
> @@ -18,6 +18,11 @@
> };
> };
>
> +&vdd {
> + regulator-min-microvolt = <2900000>;
> + regulator-max-microvolt = <2900000>;
> +};
> +
> &pwr_regulators {
> vdd-supply = <&vdd_io>;
> };
> diff --git a/arch/arm/dts/stm32mp15xx-dhcor-som.dtsi
> b/arch/arm/dts/stm32mp15xx-dhcor-som.dtsi
> index 44ecc470858..98033b51479 100644
> --- a/arch/arm/dts/stm32mp15xx-dhcor-som.dtsi
> +++ b/arch/arm/dts/stm32mp15xx-dhcor-som.dtsi
> @@ -77,8 +77,8 @@
>
> 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;
Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com>
Thanks
Patrice
