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. Patrice