Cleanup usbnet_probe() return value handling
authorPeter Korsgaard <jacmet@sunsite.dk>
Mon, 2 Jul 2007 22:46:42 +0000 (00:46 +0200)
committerJeff Garzik <jeff@garzik.org>
Tue, 10 Jul 2007 16:41:18 +0000 (12:41 -0400)
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>
drivers/net/usb/usbnet.c

index 86b6908433620428c345077b0ac3c86144afcfa7..37bf4f2c0a44c07204dcdc991cecee505462dcf1 100644 (file)
@@ -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;