RE: [PATCH V3 12/14] ARM: dts: stm32: Repair PMIC configuration on AV96

2 Apr 2020

      Hi Marek,
...
-----Original Message-----
From: Patrick DELAUNAY patrick.delaunay@st.com
Sent: mercredi 1 avril 2020 15:52
To: Marek Vasut marex@denx.de; Patrice CHOTARD
patrice.chotard@st.com; u-boot@lists.denx.de; Pascal PAILLET-LME
p.paillet@st.com
Cc: Manivannan Sadhasivam manivannan.sadhasivam@linaro.org;
Alexandre TORGUE alexandre.torgue@st.com
Subject: RE: [PATCH V3 12/14] ARM: dts: stm32: Repair PMIC configuration on
AV96
Hi,
...
From: Marek Vasut marex@denx.de
Sent: mercredi 1 avril 2020 14:53
To: Patrice CHOTARD patrice.chotard@st.com; Patrick DELAUNAY
patrick.delaunay@st.com; u-boot@lists.denx.de; Pascal PAILLET-LME
p.paillet@st.com
Cc: Manivannan Sadhasivam manivannan.sadhasivam@linaro.org;
Alexandre TORGUE alexandre.torgue@st.com
Subject: Re: [PATCH V3 12/14] ARM: dts: stm32: Repair PMIC
configuration on
AV96
Importance: High
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,
[...]
[...]
...
...
...
...
...
	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 ?
Sorry for typo.
No details on my side, it was a Pascal Paillet's direct feedback (Maintainer of
STPMIC1 driver in Linux).
You can enable the active discharge on the STPMIC1's VBUSOTG. But, as soon as a USB host peripheral (such as laptop) will attach and provide VBUS, there will be a ~8mA continuous power consumption on VBUS through this OTG active discharge. There is no functional issue behind this 8mA; but you need to know that 8mA is not compliant with the USB suspend constraint (2.5mA max).
...
Anyway he will review your patch for Linux, with more details, and I merge
this patch without modification for U-Boot.
Patrick
Pascal