Rename camel case arguments and locals in function ced_get_string() Signed-off-by: Luca Ellero <luca.ellero@xxxxxxxxxxxxxxxx> --- drivers/staging/ced1401/ced_ioc.c | 58 ++++++++++++++++++------------------- drivers/staging/ced1401/usb1401.h | 2 +- 2 files changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c b/drivers/staging/ced1401/ced_ioc.c index 29b4a7f..66d1c2f 100644 --- a/drivers/staging/ced1401/ced_ioc.c +++ b/drivers/staging/ced1401/ced_ioc.c @@ -471,10 +471,10 @@ int ced_get_char(struct ced_data *ced) ** returns the count of characters (including the terminator, or 0 if none ** or a negative error code. ****************************************************************************/ -int ced_get_string(struct ced_data *ced, char __user *pUser, int n) +int ced_get_string(struct ced_data *ced, char __user *user, int n) { - int nAvailable; /* character in the buffer */ - int iReturn = U14ERR_NOIN; + int available; /* character in the buffer */ + int ret = U14ERR_NOIN; if (n <= 0) return -ENOMEM; @@ -483,49 +483,49 @@ int ced_get_string(struct ced_data *ced, char __user *pUser, int n) ced_send_chars(ced); /* and send any buffered chars */ spin_lock_irq(&ced->char_in_lock); - nAvailable = ced->num_input; /* characters available now */ - if (nAvailable > n) /* read max of space in pUser... */ - nAvailable = n; /* ...or input characters */ - - if (nAvailable > 0) { /* worth looking? */ - char buffer[INBUF_SZ + 1]; /* space for a linear copy of data */ - int nGot = 0; - int nCopyToUser; /* number to copy to user */ - char cData; + available = ced->num_input; /* characters available now */ + if (available > n) /* read max of space in user... */ + available = n; /* ...or input characters */ + + if (available > 0) { /* worth looking? */ + char buffer[INBUF_SZ + 1]; /* space for a linear copy of data */ + int got = 0; + int n_copy_to_user; /* number to copy to user */ + char data; do { - cData = ced->input_buffer[ced->in_buff_get++]; - if (cData == CR_CHAR) /* replace CR with zero */ - cData = (char)0; + data = ced->input_buffer[ced->in_buff_get++]; + if (data == CR_CHAR) /* replace CR with zero */ + data = (char)0; if (ced->in_buff_get >= INBUF_SZ) - ced->in_buff_get = 0; /* wrap buffer pointer */ + ced->in_buff_get = 0; /* wrap buffer pointer */ - buffer[nGot++] = cData; /* save the output */ - } while ((nGot < nAvailable) && cData); + buffer[got++] = data; /* save the output */ + } while ((got < available) && data); - nCopyToUser = nGot; /* what to copy... */ - if (cData) { /* do we need null */ - buffer[nGot] = (char)0; /* make it tidy */ - if (nGot < n) /* if space in user buffer... */ - ++nCopyToUser; /* ...copy the 0 as well. */ + n_copy_to_user = got; /* what to copy... */ + if (data) { /* do we need null */ + buffer[got] = (char)0; /* make it tidy */ + if (got < n) /* if space in user buffer... */ + ++n_copy_to_user; /* ...copy the 0 as well. */ } - ced->num_input -= nGot; + ced->num_input -= got; spin_unlock_irq(&ced->char_in_lock); dev_dbg(&ced->interface->dev, "%s: read %d characters >%s<\n", - __func__, nGot, buffer); - if (copy_to_user(pUser, buffer, nCopyToUser)) - iReturn = -EFAULT; + __func__, got, buffer); + if (copy_to_user(user, buffer, n_copy_to_user)) + ret = -EFAULT; else - iReturn = nGot; /* report characters read */ + ret = got; /* report characters read */ } else spin_unlock_irq(&ced->char_in_lock); ced_allowi(ced); /* Make sure char reads are running */ mutex_unlock(&ced->io_mutex); /* Protect disconnect from new i/o */ - return iReturn; + return ret; } /******************************************************************************* diff --git a/drivers/staging/ced1401/usb1401.h b/drivers/staging/ced1401/usb1401.h index ed3508a..7ce2f02 100644 --- a/drivers/staging/ced1401/usb1401.h +++ b/drivers/staging/ced1401/usb1401.h @@ -228,7 +228,7 @@ extern int ced_get_state(struct ced_data *ced, __u32 *state, __u32 *error); extern int ced_read_write_cancel(struct ced_data *ced); extern int ced_reset(struct ced_data *ced); extern int ced_get_char(struct ced_data *ced); -extern int ced_get_string(struct ced_data *ced, char __user *pUser, int n); +extern int ced_get_string(struct ced_data *ced, char __user *user, int n); extern int ced_set_transfer(struct ced_data *ced, struct transfer_area_desc __user *pTD); extern int ced_unset_transfer(struct ced_data *ced, int nArea); extern int ced_set_event(struct ced_data *ced, struct transfer_event __user *pTE); -- 1.7.10.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel