On 4/1/20 2:45 PM, Patrice CHOTARD wrote:
>
> On 4/1/20 1:06 PM, Marek Vasut wrote:
>> 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 ?
>
> It's a typo, Patrick means over-consumption.
Ah. Do you have any details on that ?
