diff options
author | Dan Williams <dan.j.williams@intel.com> | 2014-06-17 19:16:22 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-06-17 19:44:57 -0400 |
commit | b658b8f5c09d3d1db9641c9d520b36b3e4c0d744 (patch) | |
tree | e228b01f7b2a22e7523ef31145303116cb6e4383 | |
parent | be3de32107091c266b00a48265fe7e06233af4f0 (diff) |
usb: improve "not suspended yet" message in hub_suspend()
Reading through a recent bug report [1], Alan notes:
"Dan, the warning message in hub_suspend() should mention that the
child device isn't suspended yet."
...update the warning from:
"usb usb3-port4: not suspended yet"
...to:
"usb usb3-port4: device 3-4: not suspended yet"
[1]: http://marc.info/?l=linux-usb&m=140290586301336&w=2
Reported-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/usb/core/hub.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index ec6f05cbbae3..78c3cd20d7ae 100644 --- a/drivers/usb/core/hub.c +++ b/drivers/usb/core/hub.c | |||
@@ -3458,7 +3458,8 @@ static int hub_suspend(struct usb_interface *intf, pm_message_t msg) | |||
3458 | struct usb_device *udev = port_dev->child; | 3458 | struct usb_device *udev = port_dev->child; |
3459 | 3459 | ||
3460 | if (udev && udev->can_submit) { | 3460 | if (udev && udev->can_submit) { |
3461 | dev_warn(&port_dev->dev, "not suspended yet\n"); | 3461 | dev_warn(&port_dev->dev, "device %s not suspended yet\n", |
3462 | dev_name(&udev->dev)); | ||
3462 | if (PMSG_IS_AUTO(msg)) | 3463 | if (PMSG_IS_AUTO(msg)) |
3463 | return -EBUSY; | 3464 | return -EBUSY; |
3464 | } | 3465 | } |