On Tue, Nov 19, 2019 at 04:16:56PM +0800, Henry Lin wrote: > diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c > index 6c17e3fe181a..e59346488f64 100644 > --- a/drivers/usb/host/xhci.c > +++ b/drivers/usb/host/xhci.c > @@ -770,7 +770,7 @@ static void xhci_stop(struct usb_hcd *hcd) > * > * This will only ever be called with the main usb_hcd (the USB3 roothub). > */ > -static void xhci_shutdown(struct usb_hcd *hcd) > +void xhci_shutdown(struct usb_hcd *hcd) > { > struct xhci_hcd *xhci = hcd_to_xhci(hcd); > > @@ -789,10 +789,6 @@ static void xhci_shutdown(struct usb_hcd *hcd) > xhci_dbg_trace(xhci, trace_xhci_dbg_init, > "xhci_shutdown completed - status = %x", > readl(&xhci->op_regs->status)); > - > - /* Yet another workaround for spurious wakeups at shutdown with HSW */ > - if (xhci->quirks & XHCI_SPURIOUS_WAKEUP) > - pci_set_power_state(to_pci_dev(hcd->self.sysdev), PCI_D3hot); > } Shouldn't this function also now need to be EXPORTed? Jack -- The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project