diff options
author | Steve Glendinning <steve.glendinning@shawell.net> | 2012-11-29 23:52:43 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-11-30 12:27:20 -0500 |
commit | 2305c54ffc389f1e73f2b1122a4c75bb0e8e6601 (patch) | |
tree | 414cfe6e0d31e634ae47d27c7d2ad9445d7bbeaf | |
parent | 06a31e2b918dd992dd104e082ec8a33d6235c7b3 (diff) |
smsc75xx: don't call usbnet_resume if usbnet_suspend fails
If usbnet_suspend returns an error we don't want to call
usbnet_resume to clean up, but instead just return the error.
If usbnet_suspend *does* succeed, and we have a problem further
on, the desired behaviour is still to call usbnet_resume
to clean up before returning.
Signed-off-by: Steve Glendinning <steve.glendinning@shawell.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/usb/smsc75xx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/usb/smsc75xx.c b/drivers/net/usb/smsc75xx.c index 18238060f1c0..86d924951d2e 100644 --- a/drivers/net/usb/smsc75xx.c +++ b/drivers/net/usb/smsc75xx.c | |||
@@ -1411,7 +1411,7 @@ static int smsc75xx_suspend(struct usb_interface *intf, pm_message_t message) | |||
1411 | int ret; | 1411 | int ret; |
1412 | 1412 | ||
1413 | ret = usbnet_suspend(intf, message); | 1413 | ret = usbnet_suspend(intf, message); |
1414 | check_warn_goto_done(ret, "usbnet_suspend error\n"); | 1414 | check_warn_return(ret, "usbnet_suspend error\n"); |
1415 | 1415 | ||
1416 | if (pdata->suspend_flags) { | 1416 | if (pdata->suspend_flags) { |
1417 | netdev_warn(dev->net, "error during last resume\n"); | 1417 | netdev_warn(dev->net, "error during last resume\n"); |