diff options
author | Alan Cox <alan@linux.intel.com> | 2012-09-17 06:55:23 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-17 18:10:57 -0400 |
commit | ac08de32d2e2b2b56bfe85720ec9e0b06e75350a (patch) | |
tree | 0d2dfb416b6a5ec02758f037f1e6cbc0d8177ab5 | |
parent | 2bcb132c693566bcb8208cc7ce66b72a4f852ecf (diff) |
usb: remove junk from store_remove_id
retval is 0, and carefully assigned - and tested as non zero.
This is not useful. While we are at it remove some other bogus initialisation
in the function
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/usb/core/driver.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/drivers/usb/core/driver.c b/drivers/usb/core/driver.c index 445455a4429b..ddd820d25288 100644 --- a/drivers/usb/core/driver.c +++ b/drivers/usb/core/driver.c | |||
@@ -125,10 +125,9 @@ store_remove_id(struct device_driver *driver, const char *buf, size_t count) | |||
125 | { | 125 | { |
126 | struct usb_dynid *dynid, *n; | 126 | struct usb_dynid *dynid, *n; |
127 | struct usb_driver *usb_driver = to_usb_driver(driver); | 127 | struct usb_driver *usb_driver = to_usb_driver(driver); |
128 | u32 idVendor = 0; | 128 | u32 idVendor; |
129 | u32 idProduct = 0; | 129 | u32 idProduct; |
130 | int fields = 0; | 130 | int fields; |
131 | int retval = 0; | ||
132 | 131 | ||
133 | fields = sscanf(buf, "%x %x", &idVendor, &idProduct); | 132 | fields = sscanf(buf, "%x %x", &idVendor, &idProduct); |
134 | if (fields < 2) | 133 | if (fields < 2) |
@@ -141,14 +140,10 @@ store_remove_id(struct device_driver *driver, const char *buf, size_t count) | |||
141 | (id->idProduct == idProduct)) { | 140 | (id->idProduct == idProduct)) { |
142 | list_del(&dynid->node); | 141 | list_del(&dynid->node); |
143 | kfree(dynid); | 142 | kfree(dynid); |
144 | retval = 0; | ||
145 | break; | 143 | break; |
146 | } | 144 | } |
147 | } | 145 | } |
148 | spin_unlock(&usb_driver->dynids.lock); | 146 | spin_unlock(&usb_driver->dynids.lock); |
149 | |||
150 | if (retval) | ||
151 | return retval; | ||
152 | return count; | 147 | return count; |
153 | } | 148 | } |
154 | static DRIVER_ATTR(remove_id, S_IRUGO | S_IWUSR, show_dynids, store_remove_id); | 149 | static DRIVER_ATTR(remove_id, S_IRUGO | S_IWUSR, show_dynids, store_remove_id); |