diff options
author | Alan Stern <stern@rowland.harvard.edu> | 2007-09-10 11:34:26 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-10-12 17:55:26 -0400 |
commit | 6840d2555afd66290be7a39b400b5e66a840b82d (patch) | |
tree | cfc969126e5915db6536382cddbfdd9a63f5b3c8 /drivers/usb/core/hub.c | |
parent | 95cf82f99cfbd697c15572c444bd4f54f19745b0 (diff) |
USB: flush outstanding URBs when suspending
This patch (as989) makes usbcore flush all outstanding URBs for each
device as the device is suspended. This will be true even when
CONFIG_USB_SUSPEND is not enabled.
In addition, an extra can_submit flag is added to the usb_device
structure. That flag will be turned off whenever a suspend request
has been received for the device, even if the device isn't actually
suspended because CONFIG_USB_SUSPEND isn't set.
It's no longer necessary to check for the device state being equal to
USB_STATE_SUSPENDED during URB submission; that check can be replaced
by a check of the can_submit flag. This also permits us to remove
some questionable references to the deprecated power.power_state field.
Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/core/hub.c')
-rw-r--r-- | drivers/usb/core/hub.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index bd08d51d7f4..691acf2223c 100644 --- a/drivers/usb/core/hub.c +++ b/drivers/usb/core/hub.c | |||
@@ -1955,14 +1955,7 @@ static int hub_suspend(struct usb_interface *intf, pm_message_t msg) | |||
1955 | struct usb_device *udev; | 1955 | struct usb_device *udev; |
1956 | 1956 | ||
1957 | udev = hdev->children [port1-1]; | 1957 | udev = hdev->children [port1-1]; |
1958 | if (udev && msg.event == PM_EVENT_SUSPEND && | 1958 | if (udev && udev->can_submit) { |
1959 | #ifdef CONFIG_USB_SUSPEND | ||
1960 | udev->state != USB_STATE_SUSPENDED | ||
1961 | #else | ||
1962 | udev->dev.power.power_state.event | ||
1963 | == PM_EVENT_ON | ||
1964 | #endif | ||
1965 | ) { | ||
1966 | if (!hdev->auto_pm) | 1959 | if (!hdev->auto_pm) |
1967 | dev_dbg(&intf->dev, "port %d nyet suspended\n", | 1960 | dev_dbg(&intf->dev, "port %d nyet suspended\n", |
1968 | port1); | 1961 | port1); |