diff options
| author | Afonso Bordado <afonsobordado@az8.co> | 2016-12-20 13:55:07 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-01-03 16:42:22 +0100 |
| commit | 7e026bfabbe479245aba4892863853ef44f8e981 (patch) | |
| tree | a5d47814bdb0ab70d317512920d0e1c0cc96200a /drivers/staging/emxx_udc | |
| parent | 86e2ba1c4527b708cae31fd9062a8d4bf4b964a6 (diff) | |
staging: emxx_udc: Remove unecessary temporary variable
This improves code readability.
Signed-off-by: Afonso Bordado <afonsobordado@az8.co>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/emxx_udc')
| -rw-r--r-- | drivers/staging/emxx_udc/emxx_udc.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index eab3dffd1e39..8e22c55ca698 100644 --- a/drivers/staging/emxx_udc/emxx_udc.c +++ b/drivers/staging/emxx_udc/emxx_udc.c @@ -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); } /*-------------------------------------------------------------------------*/ |