From: Colin Ian King Date: Wed, 11 Jul 2018 10:49:21 +0000 (+0100) Subject: staging: gdm724x: redundant variables idProduct and idVendor X-Git-Tag: v4.19-rc1~101^2~556 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d8353a75005d518293307b3da7c919b08a778864;p=thirdparty%2Fkernel%2Flinux.git staging: gdm724x: redundant variables idProduct and idVendor 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 Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/gdm724x/gdm_usb.c b/drivers/staging/gdm724x/gdm_usb.c index 0218782d1a08f..dc4da66c3695b 100644 --- a/drivers/staging/gdm724x/gdm_usb.c +++ b/drivers/staging/gdm724x/gdm_usb.c @@ -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;