OSDN Git Service

staging: gdm724x: redundant variables idProduct and idVendor
authorColin Ian King <colin.king@canonical.com>
Wed, 11 Jul 2018 10:49:21 +0000 (11:49 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 11 Jul 2018 11:12:34 +0000 (13:12 +0200)
Variable idProduct and idVendor are being assigned but are never used
hence they are redundant and can be removed.

Cleans up clang warnings:
warning: variable 'idProduct' set but not used [-Wunused-but-set-variable]
warning: variable 'idVendor' set but not used [-Wunused-but-set-variable]

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/gdm724x/gdm_usb.c

index 0218782..dc4da66 100644 (file)
@@ -879,14 +879,9 @@ static void gdm_usb_disconnect(struct usb_interface *intf)
 {
        struct phy_dev *phy_dev;
        struct lte_udev *udev;
-       u16 idVendor, idProduct;
        struct usb_device *usbdev;
 
        usbdev = interface_to_usbdev(intf);
-
-       idVendor = __le16_to_cpu(usbdev->descriptor.idVendor);
-       idProduct = __le16_to_cpu(usbdev->descriptor.idProduct);
-
        phy_dev = usb_get_intfdata(intf);
 
        udev = phy_dev->priv_dev;