diff options
author | Peter Korsgaard <jacmet@sunsite.dk> | 2007-07-02 18:46:42 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-07-10 12:41:18 -0400 |
commit | 9514bfe5d92000a9fd6f3c44aa775819b5d78aaf (patch) | |
tree | 5fed048ae23f5f5501230637130271b954c61e01 /drivers | |
parent | 4638aef1e2d8d938e52cd49f6492e9d414b38380 (diff) |
Cleanup usbnet_probe() return value handling
usbnet_probe() handles a positive return value from the driver bind()
function as success, but will later only setup the status handler if the
return value was zero, leading to confusion. Patch adjusts this to accept
positive values as success in both checks.
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/usb/usbnet.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/usb/usbnet.c b/drivers/net/usb/usbnet.c index 86b690843362..37bf4f2c0a44 100644 --- a/drivers/net/usb/usbnet.c +++ b/drivers/net/usb/usbnet.c | |||
@@ -1213,7 +1213,7 @@ usbnet_probe (struct usb_interface *udev, const struct usb_device_id *prod) | |||
1213 | status = 0; | 1213 | status = 0; |
1214 | 1214 | ||
1215 | } | 1215 | } |
1216 | if (status == 0 && dev->status) | 1216 | if (status >= 0 && dev->status) |
1217 | status = init_status (dev, udev); | 1217 | status = init_status (dev, udev); |
1218 | if (status < 0) | 1218 | if (status < 0) |
1219 | goto out3; | 1219 | goto out3; |