From: Wesley Cheng quic_wcheng@quicinc.com
commit 040f2dbd2010c43f33ad27249e6dac48456f4d99 upstream.
Relocate the pullups_connected check until after it is ensured that there are no runtime PM transitions. If another context triggered the DWC3 core's runtime resume, it may have already enabled the Run/Stop. Do not re-run the entire pullup sequence again, as it may issue a core soft reset while Run/Stop is already set.
This patch depends on commit 69e131d1ac4e ("usb: dwc3: gadget: Prevent repeat pullup()")
Fixes: 77adb8bdf422 ("usb: dwc3: gadget: Allow runtime suspend if UDC unbinded") Cc: stable stable@kernel.org Signed-off-by: Wesley Cheng quic_wcheng@quicinc.com Link: https://lore.kernel.org/r/20220728020647.9377-1-quic_wcheng@quicinc.com Signed-off-by: Greg Kroah-Hartman gregkh@linuxfoundation.org --- drivers/usb/dwc3/gadget.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)
--- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -2538,9 +2538,6 @@ static int dwc3_gadget_pullup(struct usb
is_on = !!is_on;
- if (dwc->pullups_connected == is_on) - return 0; - dwc->softconnect = is_on;
/* @@ -2564,6 +2561,11 @@ static int dwc3_gadget_pullup(struct usb pm_runtime_put(dwc->dev); return 0; } + + if (dwc->pullups_connected == is_on) { + pm_runtime_put(dwc->dev); + return 0; + }
if (!is_on) { ret = dwc3_gadget_soft_disconnect(dwc);