summaryrefslogtreecommitdiffstats
path: root/drivers/staging/ced1401
diff options
context:
space:
mode:
authorLuca Ellero <luca.ellero@brickedbrain.com>2014-07-10 11:02:16 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-07-10 15:09:26 -0700
commitd8b622b3aa2d164d954f4679d61fbc6a42790a5b (patch)
tree66c6e5ab78487306e1505a196ce146f1e49f3183 /drivers/staging/ced1401
parent673b2ead7aa9ea043d1fdb9d204191fc6733770b (diff)
downloadop-kernel-dev-d8b622b3aa2d164d954f4679d61fbc6a42790a5b.zip
op-kernel-dev-d8b622b3aa2d164d954f4679d61fbc6a42790a5b.tar.gz
staging: ced1401: fix ced_get_string()
Rename camel case arguments and locals in function ced_get_string() Signed-off-by: Luca Ellero <luca.ellero@brickedbrain.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/ced1401')
-rw-r--r--drivers/staging/ced1401/ced_ioc.c58
-rw-r--r--drivers/staging/ced1401/usb1401.h2
2 files changed, 30 insertions, 30 deletions
diff --git a/drivers/staging/ced1401/ced_ioc.c b/drivers/staging/ced1401/ced_ioc.c
index 9a19c84..28697c0 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 0c91562..fc650c5 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);
OpenPOWER on IntegriCloud