Commit d8b622b3 authored by Luca Ellero's avatar Luca Ellero Committed by Greg Kroah-Hartman

staging: ced1401: fix ced_get_string()

Rename camel case arguments and locals in function ced_get_string()
Signed-off-by: default avatarLuca Ellero <luca.ellero@brickedbrain.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 673b2ead
...@@ -471,10 +471,10 @@ int ced_get_char(struct ced_data *ced) ...@@ -471,10 +471,10 @@ int ced_get_char(struct ced_data *ced)
** returns the count of characters (including the terminator, or 0 if none ** returns the count of characters (including the terminator, or 0 if none
** or a negative error code. ** 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 available; /* character in the buffer */
int iReturn = U14ERR_NOIN; int ret = U14ERR_NOIN;
if (n <= 0) if (n <= 0)
return -ENOMEM; return -ENOMEM;
...@@ -483,49 +483,49 @@ int ced_get_string(struct ced_data *ced, char __user *pUser, int n) ...@@ -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 */ ced_send_chars(ced); /* and send any buffered chars */
spin_lock_irq(&ced->char_in_lock); spin_lock_irq(&ced->char_in_lock);
nAvailable = ced->num_input; /* characters available now */ available = ced->num_input; /* characters available now */
if (nAvailable > n) /* read max of space in pUser... */ if (available > n) /* read max of space in user... */
nAvailable = n; /* ...or input characters */ available = n; /* ...or input characters */
if (nAvailable > 0) { /* worth looking? */ if (available > 0) { /* worth looking? */
char buffer[INBUF_SZ + 1]; /* space for a linear copy of data */ char buffer[INBUF_SZ + 1]; /* space for a linear copy of data */
int nGot = 0; int got = 0;
int nCopyToUser; /* number to copy to user */ int n_copy_to_user; /* number to copy to user */
char cData; char data;
do { do {
cData = ced->input_buffer[ced->in_buff_get++]; data = ced->input_buffer[ced->in_buff_get++];
if (cData == CR_CHAR) /* replace CR with zero */ if (data == CR_CHAR) /* replace CR with zero */
cData = (char)0; data = (char)0;
if (ced->in_buff_get >= INBUF_SZ) 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 */ buffer[got++] = data; /* save the output */
} while ((nGot < nAvailable) && cData); } while ((got < available) && data);
nCopyToUser = nGot; /* what to copy... */ n_copy_to_user = got; /* what to copy... */
if (cData) { /* do we need null */ if (data) { /* do we need null */
buffer[nGot] = (char)0; /* make it tidy */ buffer[got] = (char)0; /* make it tidy */
if (nGot < n) /* if space in user buffer... */ if (got < n) /* if space in user buffer... */
++nCopyToUser; /* ...copy the 0 as well. */ ++n_copy_to_user; /* ...copy the 0 as well. */
} }
ced->num_input -= nGot; ced->num_input -= got;
spin_unlock_irq(&ced->char_in_lock); spin_unlock_irq(&ced->char_in_lock);
dev_dbg(&ced->interface->dev, "%s: read %d characters >%s<\n", dev_dbg(&ced->interface->dev, "%s: read %d characters >%s<\n",
__func__, nGot, buffer); __func__, got, buffer);
if (copy_to_user(pUser, buffer, nCopyToUser)) if (copy_to_user(user, buffer, n_copy_to_user))
iReturn = -EFAULT; ret = -EFAULT;
else else
iReturn = nGot; /* report characters read */ ret = got; /* report characters read */
} else } else
spin_unlock_irq(&ced->char_in_lock); spin_unlock_irq(&ced->char_in_lock);
ced_allowi(ced); /* Make sure char reads are running */ ced_allowi(ced); /* Make sure char reads are running */
mutex_unlock(&ced->io_mutex); /* Protect disconnect from new i/o */ mutex_unlock(&ced->io_mutex); /* Protect disconnect from new i/o */
return iReturn; return ret;
} }
/******************************************************************************* /*******************************************************************************
......
...@@ -228,7 +228,7 @@ extern int ced_get_state(struct ced_data *ced, __u32 *state, __u32 *error); ...@@ -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_read_write_cancel(struct ced_data *ced);
extern int ced_reset(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_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_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_unset_transfer(struct ced_data *ced, int nArea);
extern int ced_set_event(struct ced_data *ced, struct transfer_event __user *pTE); extern int ced_set_event(struct ced_data *ced, struct transfer_event __user *pTE);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment