projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
e566215
)
usb: host: u132-hcd: remove redundant continue statements
author
Colin Ian King
<colin.king@canonical.com>
Thu, 17 Jun 2021 12:38:26 +0000
(13:38 +0100)
committer
Greg Kroah-Hartman
<gregkh@linuxfoundation.org>
Thu, 17 Jun 2021 13:31:50 +0000
(15:31 +0200)
There are continue statements at the end of loops that have
no effect and are redundant. Remove them.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Addresses-Coverity: ("Continue has no effect")
Link:
https://lore.kernel.org/r/20210617123826.13764-1-colin.king@canonical.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/u132-hcd.c
patch
|
blob
|
history
diff --git
a/drivers/usb/host/u132-hcd.c
b/drivers/usb/host/u132-hcd.c
index 5a783c423d8e7e48298fb384880d36ad264bf480..ae882d76612bccb782e88dc78f0323b8ca553205 100644
(file)
--- a/
drivers/usb/host/u132-hcd.c
+++ b/
drivers/usb/host/u132-hcd.c
@@
-2392,8
+2392,7
@@
static int dequeue_from_overflow_chain(struct u132 *u132,
urb->error_count = 0;
usb_hcd_giveback_urb(hcd, urb, 0);
return 0;
- } else
- continue;
+ }
}
dev_err(&u132->platform_dev->dev, "urb=%p not found in endp[%d]=%p ring"
"[%d] %c%c usb_endp=%d usb_addr=%d size=%d next=%04X last=%04X"
@@
-2448,8
+2447,7
@@
static int u132_endp_urb_dequeue(struct u132 *u132, struct u132_endp *endp,
urb_slot = &endp->urb_list[ENDP_QUEUE_MASK &
queue_scan];
break;
- } else
- continue;
+ }
}
while (++queue_list < ENDP_QUEUE_SIZE && --queue_size > 0) {
*urb_slot = endp->urb_list[ENDP_QUEUE_MASK &