diff options
author | Alan Stern <stern@rowland.harvard.edu> | 2009-05-21 17:37:58 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-06-16 00:44:46 -0400 |
commit | be475d9027a07069407a19980d1a4fcb7b18b23c (patch) | |
tree | 7af3bcb99fb70d12c2c0f0b9c47e196560241eff /drivers/usb/storage/option_ms.c | |
parent | 9a4b5e36ce58febdd2039dd4d129ee7edf4f5536 (diff) |
USB: usb-storage: fix return values from init functions
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>
Diffstat (limited to 'drivers/usb/storage/option_ms.c')
-rw-r--r-- | drivers/usb/storage/option_ms.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/usb/storage/option_ms.c b/drivers/usb/storage/option_ms.c index 126ea3424c03..0d8853339bb6 100644 --- a/drivers/usb/storage/option_ms.c +++ b/drivers/usb/storage/option_ms.c | |||
@@ -96,7 +96,7 @@ int option_ms_init(struct us_data *us) | |||
96 | udev->descriptor.bDeviceSubClass != 0 || | 96 | udev->descriptor.bDeviceSubClass != 0 || |
97 | udev->descriptor.bDeviceProtocol != 0 || | 97 | udev->descriptor.bDeviceProtocol != 0 || |
98 | udev->actconfig->desc.bNumInterfaces == 3) | 98 | udev->actconfig->desc.bNumInterfaces == 3) |
99 | return USB_STOR_TRANSPORT_GOOD; | 99 | return 0; |
100 | 100 | ||
101 | US_DEBUGP("Option MS: option_ms_init called\n"); | 101 | US_DEBUGP("Option MS: option_ms_init called\n"); |
102 | 102 | ||
@@ -107,7 +107,7 @@ int option_ms_init(struct us_data *us) | |||
107 | iface_desc->desc.bInterfaceProtocol != 0x50) { | 107 | iface_desc->desc.bInterfaceProtocol != 0x50) { |
108 | US_DEBUGP("Option MS: mass storage interface not found, no action " | 108 | US_DEBUGP("Option MS: mass storage interface not found, no action " |
109 | "required\n"); | 109 | "required\n"); |
110 | return USB_STOR_TRANSPORT_GOOD; | 110 | return 0; |
111 | } | 111 | } |
112 | 112 | ||
113 | /* Find the mass storage bulk endpoints */ | 113 | /* Find the mass storage bulk endpoints */ |
@@ -127,7 +127,7 @@ int option_ms_init(struct us_data *us) | |||
127 | if (!ep_in_size || !ep_out_size) { | 127 | if (!ep_in_size || !ep_out_size) { |
128 | US_DEBUGP("Option MS: mass storage endpoints not found, no action " | 128 | US_DEBUGP("Option MS: mass storage endpoints not found, no action " |
129 | "required\n"); | 129 | "required\n"); |
130 | return USB_STOR_TRANSPORT_GOOD; | 130 | return 0; |
131 | } | 131 | } |
132 | 132 | ||
133 | /* Force Modem mode */ | 133 | /* Force Modem mode */ |
@@ -143,6 +143,6 @@ int option_ms_init(struct us_data *us) | |||
143 | " requests it\n"); | 143 | " requests it\n"); |
144 | } | 144 | } |
145 | 145 | ||
146 | return USB_STOR_TRANSPORT_GOOD; | 146 | return 0; |
147 | } | 147 | } |
148 | 148 | ||