diff options
author | Colin Ian King <colin.king@canonical.com> | 2018-07-13 05:51:51 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-07-13 09:41:56 -0400 |
commit | 260560322616472e3e26ece9e9bd044b19031c6c (patch) | |
tree | 38feec76d76e1c07bc7f71eb9f1fe2136d138bf0 | |
parent | 795a8075e9a53f441229c3890f702bf96508406b (diff) |
usb: isp1760: remove redundant variable 'selector'
Variable 'selector' is being assigned but is never used hence it is
redundant and can be removed.
Cleans up clang warning:
warning: variable 'selector' 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>
-rw-r--r-- | drivers/usb/isp1760/isp1760-hcd.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/usb/isp1760/isp1760-hcd.c b/drivers/usb/isp1760/isp1760-hcd.c index 1045521be293..8142c6b4c4cf 100644 --- a/drivers/usb/isp1760/isp1760-hcd.c +++ b/drivers/usb/isp1760/isp1760-hcd.c | |||
@@ -1817,7 +1817,6 @@ static int isp1760_hub_control(struct usb_hcd *hcd, u16 typeReq, | |||
1817 | u32 temp, status; | 1817 | u32 temp, status; |
1818 | unsigned long flags; | 1818 | unsigned long flags; |
1819 | int retval = 0; | 1819 | int retval = 0; |
1820 | unsigned selector; | ||
1821 | 1820 | ||
1822 | /* | 1821 | /* |
1823 | * FIXME: support SetPortFeatures USB_PORT_FEAT_INDICATOR. | 1822 | * FIXME: support SetPortFeatures USB_PORT_FEAT_INDICATOR. |
@@ -2010,7 +2009,6 @@ static int isp1760_hub_control(struct usb_hcd *hcd, u16 typeReq, | |||
2010 | } | 2009 | } |
2011 | break; | 2010 | break; |
2012 | case SetPortFeature: | 2011 | case SetPortFeature: |
2013 | selector = wIndex >> 8; | ||
2014 | wIndex &= 0xff; | 2012 | wIndex &= 0xff; |
2015 | if (!wIndex || wIndex > ports) | 2013 | if (!wIndex || wIndex > ports) |
2016 | goto error; | 2014 | goto error; |