Variable ar assigned a value that is never used. I have also removed all the code that thereby serves no purpose. This was found using a static code analysis program called cppcheck Signed-off-by: Rickard Strandqvist <rickard_strandqvist@xxxxxxxxxxxxxxxxxx> --- drivers/staging/emxx_udc/emxx_udc.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index eb178fc..b916fab 100644 --- a/drivers/staging/emxx_udc/emxx_udc.c +++ b/drivers/staging/emxx_udc/emxx_udc.c @@ -2974,10 +2974,10 @@ static int nbu2ss_ep_fifo_status(struct usb_ep *_ep) spin_lock_irqsave(&udc->lock, flags); if (ep->epnum == 0) { - data = _nbu2ss_readl(&preg->EP0_LENGTH) & EP0_LDATA; + _nbu2ss_readl(&preg->EP0_LENGTH) & EP0_LDATA; } else { - data = _nbu2ss_readl(&preg->EP_REGS[ep->epnum-1].EP_LEN_DCNT) + _nbu2ss_readl(&preg->EP_REGS[ep->epnum-1].EP_LEN_DCNT) & EPn_LDATA; } @@ -3264,12 +3264,11 @@ static void __init nbu2ss_drv_set_ep_info( if (isdigit(name[2])) { long num; - int res; char tempbuf[2]; tempbuf[0] = name[2]; tempbuf[1] = '\0'; - res = kstrtol(tempbuf, 16, &num); + kstrtol(tempbuf, 16, &num); if (num == 0) ep->ep.maxpacket = EP0_PACKETSIZE; -- 1.7.10.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel