staging: emxx_udc: Remove unecessary temporary variable
authorAfonso Bordado <afonsobordado@az8.co>
Tue, 20 Dec 2016 13:55:07 +0000 (13:55 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 3 Jan 2017 15:42:22 +0000 (16:42 +0100)
This improves code readability.

Signed-off-by: Afonso Bordado <afonsobordado@az8.co>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/emxx_udc/emxx_udc.c

index eab3dff..8e22c55 100644 (file)
@@ -556,21 +556,19 @@ static void _nbu2ss_dma_unmap_single(
 static int ep0_out_pio(struct nbu2ss_udc *udc, u8 *pBuf, u32 length)
 {
        u32             i;
-       int             nret   = 0;
        u32 numreads = length / sizeof(u32);
        union usb_reg_access *pBuf32 = (union usb_reg_access *)pBuf;
 
-       /*------------------------------------------------------------*/
+       if (!numreads)
+               return 0;
+
        /* PIO Read */
-       if (numreads) {
-               for (i = 0; i < numreads; i++) {
-                       pBuf32->dw = _nbu2ss_readl(&udc->p_regs->EP0_READ);
-                       pBuf32++;
-               }
-               nret = numreads * sizeof(u32);
+       for (i = 0; i < numreads; i++) {
+               pBuf32->dw = _nbu2ss_readl(&udc->p_regs->EP0_READ);
+               pBuf32++;
        }
 
-       return nret;
+       return  numreads * sizeof(u32);
 }
 
 /*-------------------------------------------------------------------------*/