summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Korsgaard <jacmet@sunsite.dk>2007-07-03 00:46:42 +0200
committerJeff Garzik <jeff@garzik.org>2007-07-10 12:41:18 -0400
commit9514bfe5d92000a9fd6f3c44aa775819b5d78aaf (patch)
tree5fed048ae23f5f5501230637130271b954c61e01
parent4638aef1e2d8d938e52cd49f6492e9d414b38380 (diff)
downloadop-kernel-dev-9514bfe5d92000a9fd6f3c44aa775819b5d78aaf.zip
op-kernel-dev-9514bfe5d92000a9fd6f3c44aa775819b5d78aaf.tar.gz
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>
-rw-r--r--drivers/net/usb/usbnet.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/usb/usbnet.c b/drivers/net/usb/usbnet.c
index 86b6908..37bf4f2 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)
status = 0;
}
- if (status == 0 && dev->status)
+ if (status >= 0 && dev->status)
status = init_status (dev, udev);
if (status < 0)
goto out3;
OpenPOWER on IntegriCloud