Hi Jagan,
On 2020/5/1 14:53, Jagan Teki wrote:
On Thu, Apr 30, 2020 at 7:47 AM Frank Wang <frank.w...@rock-chips.com> wrote:
Add a quirk to clear the GUSB2PHYCFG.ENBLSLPM bit, which controls
whether the PHY receives the suspend signal from the controller.
Refer to commit ec791d149bca("usb: dwc3: Add dis_enblslpm_quirk")
in Linux Kernel.
Signed-off-by: Frank Wang <frank.w...@rock-chips.com>
Reviewed-by: Kever Yang <kever.y...@rock-chips.com>
---
drivers/usb/dwc3/core.c | 6 ++++++
drivers/usb/dwc3/core.h | 1 +
include/dwc3-uboot.h | 1 +
3 files changed, 8 insertions(+)
diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
index 4ec3f6df6a..a80e7d54aa 100644
--- a/drivers/usb/dwc3/core.c
+++ b/drivers/usb/dwc3/core.c
@@ -395,6 +395,9 @@ static void dwc3_phy_setup(struct dwc3 *dwc)
if (dwc->dis_u2_susphy_quirk)
reg &= ~DWC3_GUSB2PHYCFG_SUSPHY;
+ if (dwc->dis_enblslpm_quirk)
+ reg &= ~DWC3_GUSB2PHYCFG_ENBLSLPM;
+
This patch alone seems not building with DWC3
enabled,drivers/usb/dwc3/core.c: In function ‘dwc3_phy_setup’:
drivers/usb/dwc3/core.c:399:11: error: ‘DWC3_GUSB2PHYCFG_ENBLSLPM’
undeclared (first use in this function)
reg &= ~DWC3_GUSB2PHYCFG_ENBLSLPM;
Add this macros to core.h
ACK
Jagan.