diff options
author | David Brownell <david-b@pacbell.net> | 2005-09-13 22:57:27 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-10-28 19:47:38 -0400 |
commit | 390a8c345e6415cbf811232feedac70b56c9fc8d (patch) | |
tree | 5a27e9421575c080d31884c4eedeed710d7134f1 /drivers/usb/host | |
parent | c9f89fa40cf274130b56475175a35af961d4b165 (diff) |
[PATCH] remove usb_suspend_device() parameter
This patch removes the extra usb_suspend_device() parameter. The original
reason to pass that parameter was so that this routine could suspend any
active children. A previous patch removed that functionality ... leaving
no reason to pass the parameter. A close analogy is pci_set_power_state,
which doesn't need a pm_message_t either.
On the internal code path that comes through the driver model, the parameter
is now used to distinguish cases where USB devices need to "freeze" but not
suspend. It also checks for an error case that's accessible through sysfs:
attempting to suspend a device before its interfaces (or for hubs, ports).
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/core/hub.c | 34 +++++++++++++++++++++-------------
drivers/usb/core/usb.c | 23 +++++++++++++++++++++--
drivers/usb/host/ehci-hcd.c | 2 +-
drivers/usb/host/isp116x-hcd.c | 2 +-
drivers/usb/host/ohci-pci.c | 2 +-
include/linux/usb.h | 2 +-
6 files changed, 46 insertions(+), 19 deletions(-)
Diffstat (limited to 'drivers/usb/host')
-rw-r--r-- | drivers/usb/host/ehci-hcd.c | 2 | ||||
-rw-r--r-- | drivers/usb/host/isp116x-hcd.c | 2 | ||||
-rw-r--r-- | drivers/usb/host/ohci-pci.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c index b3eb02613bff..513fccbb8e43 100644 --- a/drivers/usb/host/ehci-hcd.c +++ b/drivers/usb/host/ehci-hcd.c | |||
@@ -759,7 +759,7 @@ static int ehci_suspend (struct usb_hcd *hcd, pm_message_t message) | |||
759 | msleep (100); | 759 | msleep (100); |
760 | 760 | ||
761 | #ifdef CONFIG_USB_SUSPEND | 761 | #ifdef CONFIG_USB_SUSPEND |
762 | (void) usb_suspend_device (hcd->self.root_hub, message); | 762 | (void) usb_suspend_device (hcd->self.root_hub); |
763 | #else | 763 | #else |
764 | usb_lock_device (hcd->self.root_hub); | 764 | usb_lock_device (hcd->self.root_hub); |
765 | (void) ehci_hub_suspend (hcd); | 765 | (void) ehci_hub_suspend (hcd); |
diff --git a/drivers/usb/host/isp116x-hcd.c b/drivers/usb/host/isp116x-hcd.c index 642f35068ce2..554d60282a9d 100644 --- a/drivers/usb/host/isp116x-hcd.c +++ b/drivers/usb/host/isp116x-hcd.c | |||
@@ -1781,7 +1781,7 @@ static int isp116x_suspend(struct device *dev, pm_message_t state) | |||
1781 | 1781 | ||
1782 | VDBG("%s: state %x\n", __func__, state); | 1782 | VDBG("%s: state %x\n", __func__, state); |
1783 | 1783 | ||
1784 | ret = usb_suspend_device(hcd->self.root_hub, state); | 1784 | ret = usb_suspend_device(hcd->self.root_hub); |
1785 | if (!ret) { | 1785 | if (!ret) { |
1786 | dev->power.power_state = state; | 1786 | dev->power.power_state = state; |
1787 | INFO("%s suspended\n", hcd_name); | 1787 | INFO("%s suspended\n", hcd_name); |
diff --git a/drivers/usb/host/ohci-pci.c b/drivers/usb/host/ohci-pci.c index eede6be098d2..41e85980fa7a 100644 --- a/drivers/usb/host/ohci-pci.c +++ b/drivers/usb/host/ohci-pci.c | |||
@@ -119,7 +119,7 @@ static int ohci_pci_suspend (struct usb_hcd *hcd, pm_message_t message) | |||
119 | msleep (100); | 119 | msleep (100); |
120 | 120 | ||
121 | #ifdef CONFIG_USB_SUSPEND | 121 | #ifdef CONFIG_USB_SUSPEND |
122 | (void) usb_suspend_device (hcd->self.root_hub, message); | 122 | (void) usb_suspend_device (hcd->self.root_hub); |
123 | #else | 123 | #else |
124 | usb_lock_device (hcd->self.root_hub); | 124 | usb_lock_device (hcd->self.root_hub); |
125 | (void) ohci_hub_suspend (hcd); | 125 | (void) ohci_hub_suspend (hcd); |