Commit 3494252d authored by Rafael J. Wysocki's avatar Rafael J. Wysocki Committed by Linus Torvalds

USB/PCI: Fix resume breakage of controllers behind cardbus bridges

If a USB PCI controller is behind a cardbus bridge, we are trying to
restore its configuration registers too early, before the cardbus
bridge is operational.  To fix this, call pci_restore_state() from
usb_hcd_pci_resume() and remove usb_hcd_pci_resume_early() which is
no longer necessary (the configuration spaces of USB controllers that
are not behind cardbus bridges will be restored by the PCI PM core
with interrupts disabled anyway).

This patch fixes the regression from 2.6.28 tracked as
http://bugzilla.kernel.org/show_bug.cgi?id=12659

[ Side note: the proper long-term fix is probably to just force the
  unplug event at suspend time instead of doing a plug/unplug at resume
  time, but this patch is fine regardless  - Linus ]
Signed-off-by: default avatarRafael J. Wysocki <rjw@sisk.pl>
Reported-by: default avatarMiles Lane <miles.lane@gmail.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent e78ac4b9
...@@ -297,19 +297,6 @@ int usb_hcd_pci_suspend(struct pci_dev *dev, pm_message_t message) ...@@ -297,19 +297,6 @@ int usb_hcd_pci_suspend(struct pci_dev *dev, pm_message_t message)
} }
EXPORT_SYMBOL_GPL(usb_hcd_pci_suspend); EXPORT_SYMBOL_GPL(usb_hcd_pci_suspend);
/**
* usb_hcd_pci_resume_early - resume a PCI-based HCD before IRQs are enabled
* @dev: USB Host Controller being resumed
*
* Store this function in the HCD's struct pci_driver as .resume_early.
*/
int usb_hcd_pci_resume_early(struct pci_dev *dev)
{
pci_restore_state(dev);
return 0;
}
EXPORT_SYMBOL_GPL(usb_hcd_pci_resume_early);
/** /**
* usb_hcd_pci_resume - power management resume of a PCI-based HCD * usb_hcd_pci_resume - power management resume of a PCI-based HCD
* @dev: USB Host Controller being resumed * @dev: USB Host Controller being resumed
...@@ -333,6 +320,8 @@ int usb_hcd_pci_resume(struct pci_dev *dev) ...@@ -333,6 +320,8 @@ int usb_hcd_pci_resume(struct pci_dev *dev)
} }
#endif #endif
pci_restore_state(dev);
hcd = pci_get_drvdata(dev); hcd = pci_get_drvdata(dev);
if (hcd->state != HC_STATE_SUSPENDED) { if (hcd->state != HC_STATE_SUSPENDED) {
dev_dbg(hcd->self.controller, dev_dbg(hcd->self.controller,
......
...@@ -257,7 +257,6 @@ extern void usb_hcd_pci_remove(struct pci_dev *dev); ...@@ -257,7 +257,6 @@ extern void usb_hcd_pci_remove(struct pci_dev *dev);
#ifdef CONFIG_PM #ifdef CONFIG_PM
extern int usb_hcd_pci_suspend(struct pci_dev *dev, pm_message_t msg); extern int usb_hcd_pci_suspend(struct pci_dev *dev, pm_message_t msg);
extern int usb_hcd_pci_resume_early(struct pci_dev *dev);
extern int usb_hcd_pci_resume(struct pci_dev *dev); extern int usb_hcd_pci_resume(struct pci_dev *dev);
#endif /* CONFIG_PM */ #endif /* CONFIG_PM */
......
...@@ -432,7 +432,6 @@ static struct pci_driver ehci_pci_driver = { ...@@ -432,7 +432,6 @@ static struct pci_driver ehci_pci_driver = {
#ifdef CONFIG_PM #ifdef CONFIG_PM
.suspend = usb_hcd_pci_suspend, .suspend = usb_hcd_pci_suspend,
.resume_early = usb_hcd_pci_resume_early,
.resume = usb_hcd_pci_resume, .resume = usb_hcd_pci_resume,
#endif #endif
.shutdown = usb_hcd_pci_shutdown, .shutdown = usb_hcd_pci_shutdown,
......
...@@ -487,7 +487,6 @@ static struct pci_driver ohci_pci_driver = { ...@@ -487,7 +487,6 @@ static struct pci_driver ohci_pci_driver = {
#ifdef CONFIG_PM #ifdef CONFIG_PM
.suspend = usb_hcd_pci_suspend, .suspend = usb_hcd_pci_suspend,
.resume_early = usb_hcd_pci_resume_early,
.resume = usb_hcd_pci_resume, .resume = usb_hcd_pci_resume,
#endif #endif
......
...@@ -942,7 +942,6 @@ static struct pci_driver uhci_pci_driver = { ...@@ -942,7 +942,6 @@ static struct pci_driver uhci_pci_driver = {
#ifdef CONFIG_PM #ifdef CONFIG_PM
.suspend = usb_hcd_pci_suspend, .suspend = usb_hcd_pci_suspend,
.resume_early = usb_hcd_pci_resume_early,
.resume = usb_hcd_pci_resume, .resume = usb_hcd_pci_resume,
#endif /* PM */ #endif /* PM */
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment