Skip to content

USB: dwc3: qcom: fix wakeup after probe deferral

Johan Hovold linux-arm-msm linux-devicetree linux-usb lkml stable 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 9 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3dt-bindings: usb: qcom,dwc3: fix example wakeup interrupt typesDocumentation/devicetree/bindings/usb/qcom,dwc3.yaml+2 -21 reply
2/3USB: dwc3: qcom: fix wakeup after probe deferraldrivers/usb/dwc3/dwc3-qcom.c+4 -47 replies
3/3USB: dwc3: qcom: simplify wakeup interrupt setupdrivers/usb/dwc3/dwc3-qcom.c+4 -80 replies

Reviews trailers per patch

1 of 3 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchKrzysztof KozlowskiAndrew Halaney
SeriesR-b
1/3A-b
2/3
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/3] USB: dwc3: qcom: fix wakeup after probe deferral

When testing a recent series that addresses resource leaks on probe deferral [1] I realised that probe deferral can break wakeup from suspend due to how the wakeup interrupts are currently requested.

I'll send a separate series for the Qualcomm devicetrees that used incorrect trigger types for the wakeup interrupts. Included here is just a patch fixing the binding example which hopefully will make it less likely that more of these gets introduced. Fortunately, there should be no dependency between this series and the devicetree one.

Note also that I decided to include a related trivial cleanup patch.

Johan

[1] https://lore.kernel.org/lkml/[email protected]/

Johan Hovold (3):
dt-bindings: usb: qcom,dwc3: fix example wakeup interrupt types
USB: dwc3: qcom: fix wakeup after probe deferral
USB: dwc3: qcom: simplify wakeup interrupt setup

 Documentation/devicetree/bindings/usb/qcom,dwc3.yaml |  4 ++--
 drivers/usb/dwc3/dwc3-qcom.c                         | 12 ++++--------
 2 files changed, 6 insertions(+), 10 deletions(-)
Signature · 1 line
2.41.0