summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2019-06-26 17:35:17 +0100
committerFelipe Balbi <felipe.balbi@linux.intel.com>2019-07-03 11:00:36 +0300
commit4aef7966060439bb4f9d440feacd5ce7697bce37 (patch)
tree0a07af39875d96561dd3fa09d6b8386ed2cf7bf8
parentdfc4fdebc5d62ac4e2fe5428e59b273675515fb2 (diff)
downloadlinux-stable-4aef7966060439bb4f9d440feacd5ce7697bce37.tar.gz
linux-stable-4aef7966060439bb4f9d440feacd5ce7697bce37.tar.bz2
linux-stable-4aef7966060439bb4f9d440feacd5ce7697bce37.zip
usb: gadget: udc: renesas_usb3: remove redundant assignment to ret
Variable ret is being initialized with a value that is never read and ret is being re-assigned immediately after the initialization in both paths of an if statement. This is redundant and can be removed. Addresses-Coverity: ("Unused value") Reviewed-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com> Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
-rw-r--r--drivers/usb/gadget/udc/renesas_usb3.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/gadget/udc/renesas_usb3.c b/drivers/usb/gadget/udc/renesas_usb3.c
index 5a960fce31c5..87062d22134d 100644
--- a/drivers/usb/gadget/udc/renesas_usb3.c
+++ b/drivers/usb/gadget/udc/renesas_usb3.c
@@ -1168,7 +1168,7 @@ static void usb3_set_status_stage(struct renesas_usb3_ep *usb3_ep,
static void usb3_p0_xfer(struct renesas_usb3_ep *usb3_ep,
struct renesas_usb3_request *usb3_req)
{
- int ret = -EAGAIN;
+ int ret;
if (usb3_ep->dir_in)
ret = usb3_write_pipe(usb3_ep, usb3_req, USB3_P0_WRITE);