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 86b6908..37bf4f2 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;