summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2017-11-07 12:21:40 +0000
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-11-07 15:52:29 +0100
commit15dfeb6dee4a2ea60efe2f94c7c293ab4ba3cb30 (patch)
treee0549eb9481a4347439ae0ae4d5cefbe71452ce5
parent8444efc4a052332d643ed5c8aebcca148c7de032 (diff)
downloadop-kernel-dev-15dfeb6dee4a2ea60efe2f94c7c293ab4ba3cb30.zip
op-kernel-dev-15dfeb6dee4a2ea60efe2f94c7c293ab4ba3cb30.tar.gz
usb: host: isp1362-hcd: remove a couple of redundant assignments
Variable index is being initialized and the value is never read, it is being updated a few statements later, so remove the redundant initialization. Variable total is being updated but the value is never read, so this is also redundant and can be removed. Cleans up two clang warnings: drivers/usb/host/isp1362-hcd.c:715:6: warning: Value stored to 'index' during its initialization is never read drivers/usb/host/isp1362-hcd.c:2254:2: warning: Value stored to 'total' is never read Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/usb/host/isp1362-hcd.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/usb/host/isp1362-hcd.c b/drivers/usb/host/isp1362-hcd.c
index c7e60b6..b21c386 100644
--- a/drivers/usb/host/isp1362-hcd.c
+++ b/drivers/usb/host/isp1362-hcd.c
@@ -713,7 +713,7 @@ static inline void enable_istl_transfers(struct isp1362_hcd *isp1362_hcd, int fl
static int submit_req(struct isp1362_hcd *isp1362_hcd, struct urb *urb,
struct isp1362_ep *ep, struct isp1362_ep_queue *epq)
{
- int index = epq->free_ptd;
+ int index;
prepare_ptd(isp1362_hcd, urb, ep, epq, 0);
index = claim_ptd_buffers(epq, ep, ep->length);
@@ -2253,7 +2253,6 @@ static int isp1362_mem_config(struct usb_hcd *hcd)
return -ENOMEM;
}
- total = istl_size + intl_size + atl_size;
spin_lock_irqsave(&isp1362_hcd->lock, flags);
for (i = 0; i < 2; i++) {
OpenPOWER on IntegriCloud