usb: remove junk from store_remove_id
authorAlan Cox <alan@linux.intel.com>
Mon, 17 Sep 2012 10:55:23 +0000 (11:55 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Sep 2012 22:10:57 +0000 (15:10 -0700)
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>
drivers/usb/core/driver.c

index 445455a4429bef2f9a26cff9a3aa88ac4629c1ac..ddd820d25288b57da71336bf4eed397a4cc16410 100644 (file)
@@ -125,10 +125,9 @@ store_remove_id(struct device_driver *driver, const char *buf, size_t count)
 {
        struct usb_dynid *dynid, *n;
        struct usb_driver *usb_driver = to_usb_driver(driver);
-       u32 idVendor = 0;
-       u32 idProduct = 0;
-       int fields = 0;
-       int retval = 0;
+       u32 idVendor;
+       u32 idProduct;
+       int fields;
 
        fields = sscanf(buf, "%x %x", &idVendor, &idProduct);
        if (fields < 2)
@@ -141,14 +140,10 @@ store_remove_id(struct device_driver *driver, const char *buf, size_t count)
                    (id->idProduct == idProduct)) {
                        list_del(&dynid->node);
                        kfree(dynid);
-                       retval = 0;
                        break;
                }
        }
        spin_unlock(&usb_driver->dynids.lock);
-
-       if (retval)
-               return retval;
        return count;
 }
 static DRIVER_ATTR(remove_id, S_IRUGO | S_IWUSR, show_dynids, store_remove_id);