[PATCH v2 12/60] staging: ced1401: fix ced_stage_chunk()
Luca Ellero
luca.ellero at brickedbrain.com
Thu Jul 10 09:01:56 UTC 2014
Rename camel case arguments and locals in function ced_stage_chunk()
Signed-off-by: Luca Ellero <luca.ellero at brickedbrain.com>
---
drivers/staging/ced1401/usb1401.c | 53 ++++++++++++++++++++++---------------
1 file changed, 31 insertions(+), 22 deletions(-)
diff --git a/drivers/staging/ced1401/usb1401.c b/drivers/staging/ced1401/usb1401.c
index 99c8d2f..e0d7aa0 100644
--- a/drivers/staging/ced1401/usb1401.c
+++ b/drivers/staging/ced1401/usb1401.c
@@ -757,16 +757,19 @@ static void staged_callback(struct urb *urb)
** Generates the next chunk of data making up a staged transfer.
**
** The calling code must have acquired the staging spinlock before calling
-** this function, and is responsible for releasing it. We are at callback level.
+** this function, and is responsible for releasing it. We are at callback level.
****************************************************************************/
static int ced_stage_chunk(struct ced_data *ced)
{
- int iReturn = U14ERR_NOERROR;
- unsigned int ChunkSize;
- int nPipe = ced->staged_read ? 3 : 2; /* The pipe number to use for reads or writes */
+ int retval = U14ERR_NOERROR;
+ unsigned int chunk_size;
+ int pipe = ced->staged_read ? 3 : 2; /* The pipe number to use for */
+ /* reads or writes */
+
if (ced->n_pipes == 3)
- nPipe--; /* Adjust for the 3-pipe case */
- if (nPipe < 0) /* and trap case that should never happen */
+ pipe--; /* Adjust for the 3-pipe case */
+
+ if (pipe < 0) /* and trap case that should never happen */
return U14ERR_FAIL;
if (!can_accept_io_requests(ced)) { /* got sudden remove? */
@@ -775,34 +778,40 @@ static int ced_stage_chunk(struct ced_data *ced)
return U14ERR_FAIL; /* could do with a better error */
}
- ChunkSize = (ced->staged_length - ced->staged_done); /* transfer length remaining */
- if (ChunkSize > STAGED_SZ) /* make sure to keep legal */
- ChunkSize = STAGED_SZ; /* limit to max allowed */
+ /* transfer length remaining */
+ chunk_size = (ced->staged_length - ced->staged_done);
+ if (chunk_size > STAGED_SZ) /* make sure to keep legal */
+ chunk_size = STAGED_SZ; /* limit to max allowed */
if (!ced->staged_read) /* if writing... */
- ced_copy_user_space(ced, ChunkSize); /* ...copy data into the buffer */
+ /* ...copy data into the buffer */
+ ced_copy_user_space(ced, chunk_size);
usb_fill_bulk_urb(ced->staged_urb, ced->udev,
ced->staged_read ? usb_rcvbulkpipe(ced->udev,
ced->
- ep_addr[nPipe]) :
- usb_sndbulkpipe(ced->udev, ced->ep_addr[nPipe]),
- ced->coher_staged_io, ChunkSize,
+ ep_addr[pipe]) :
+ usb_sndbulkpipe(ced->udev, ced->ep_addr[pipe]),
+ ced->coher_staged_io, chunk_size,
staged_callback, ced);
ced->staged_urb->transfer_flags |= URB_NO_TRANSFER_DMA_MAP;
- usb_anchor_urb(ced->staged_urb, &ced->submitted); /* in case we need to kill it */
- iReturn = usb_submit_urb(ced->staged_urb, GFP_ATOMIC);
- if (iReturn) {
+ /* in case we need to kill it */
+ usb_anchor_urb(ced->staged_urb, &ced->submitted);
+ retval = usb_submit_urb(ced->staged_urb, GFP_ATOMIC);
+ if (retval) {
usb_unanchor_urb(ced->staged_urb); /* kill it */
- ced->pipe_error[nPipe] = 1; /* Flag an error to be handled later */
- dev_err(&ced->interface->dev, "%s: submit urb failed, code %d\n",
- __func__, iReturn);
+ ced->pipe_error[pipe] = 1; /* Flag an error to be */
+ /* handled later */
+ dev_err(&ced->interface->dev,
+ "%s: submit urb failed, code %d\n",
+ __func__, retval);
} else
- ced->staged_urb_pending = true; /* Set the flag for staged URB pending */
+ /* Set the flag for staged URB pending */
+ ced->staged_urb_pending = true;
dev_dbg(&ced->interface->dev, "%s: done so far:%d, this size:%d\n",
- __func__, ced->staged_done, ChunkSize);
+ __func__, ced->staged_done, chunk_size);
- return iReturn;
+ return retval;
}
/***************************************************************************
--
1.7.10.4
More information about the devel
mailing list