USB: usb-storage: fix return values from init functions
authorAlan Stern <stern@rowland.harvard.edu>
Thu, 21 May 2009 21:37:58 +0000 (17:37 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 16 Jun 2009 04:44:46 +0000 (21:44 -0700)
This patch (as1242) fixes the return values from the special
init functions in usb-storage.  They are supposed to return 0 for
success, not USB_STOR_TRANSPORT_GOOD.

Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/storage/option_ms.c
drivers/usb/storage/sierra_ms.c

index 126ea3424c030ace82af3abf216a4a4729b6d4e6..0d8853339bb613f590c5eb981c4520e4674118ba 100644 (file)
@@ -96,7 +96,7 @@ int option_ms_init(struct us_data *us)
            udev->descriptor.bDeviceSubClass != 0 ||
            udev->descriptor.bDeviceProtocol != 0 ||
            udev->actconfig->desc.bNumInterfaces == 3)
-               return USB_STOR_TRANSPORT_GOOD;
+               return 0;
 
        US_DEBUGP("Option MS: option_ms_init called\n");
 
@@ -107,7 +107,7 @@ int option_ms_init(struct us_data *us)
            iface_desc->desc.bInterfaceProtocol != 0x50) {
                US_DEBUGP("Option MS: mass storage interface not found, no action "
                          "required\n");
-               return USB_STOR_TRANSPORT_GOOD;
+               return 0;
        }
 
        /* Find the mass storage bulk endpoints */
@@ -127,7 +127,7 @@ int option_ms_init(struct us_data *us)
        if (!ep_in_size || !ep_out_size) {
                US_DEBUGP("Option MS: mass storage endpoints not found, no action "
                          "required\n");
-               return USB_STOR_TRANSPORT_GOOD;
+               return 0;
        }
 
        /* Force Modem mode */
@@ -143,6 +143,6 @@ int option_ms_init(struct us_data *us)
                          " requests it\n");
        }
 
-       return USB_STOR_TRANSPORT_GOOD;
+       return 0;
 }
 
index 4359a2cb42df396d8e7996761814ff40755846dd..4395c4100ec2096313933d029e8b25191eefe6c9 100644 (file)
@@ -202,6 +202,6 @@ int sierra_ms_init(struct us_data *us)
 complete:
        result = device_create_file(&us->pusb_intf->dev, &dev_attr_truinst);
 
-       return USB_STOR_TRANSPORT_GOOD;
+       return 0;
 }