On 4/1/20 9:34 AM, Patrick DELAUNAY wrote:
> Hi Marek and Pascal,
Hi,
[...]
>> @@ -167,7 +178,7 @@
>>
>> vddcore: buck1 {
>> regulator-name = "vddcore";
>> - regulator-min-microvolt = <1200000>;
>> + regulator-min-microvolt = <800000>;
>
> I don't understood this modification, If I correctly remember, on STM32MP15x
> On ST boards, we set this value to 1200000 to protect VDDCORE on
> SOC (it is not allowed to be reduce lower than 1,2V.
>
> 0.8V is the minimal supported voltage by STPMIC but not the minal expected
> VDDCORE.
Then we should likely drop this part.
That said, is the core voltage supposed to be 1V2 even in sleep states ?
>> @@ -185,8 +196,8 @@
>>
>> vdd: buck3 {
>> regulator-name = "vdd";
>> - regulator-min-microvolt = <3300000>;
>> - regulator-max-microvolt = <3300000>;
>> + regulator-min-microvolt = <2900000>;
>> + regulator-max-microvolt = <2900000>;
>> regulator-always-on;
>> st,mask_reset;
>> regulator-initial-mode = <0>;
>> @@ -268,6 +279,7 @@
>> regulator-name = "vbus_otg";
>> interrupts = <IT_OCP_OTG 0>;
>> interrupt-parent = <&pmic>;
>> + regulator-active-discharge = <1>;
>
> It seems this setting introduced other-consumption, it should not be used.
Other-consumption ? What does that mean ?
