staging: ced1401: fix ced_get_transfer()
authorLuca Ellero <luca.ellero@brickedbrain.com>
Thu, 10 Jul 2014 09:03:59 +0000 (11:03 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 10 Jul 2014 22:10:05 +0000 (15:10 -0700)
Rename camel case arguments and locals in function ced_get_transfer()

Signed-off-by: Luca Ellero <luca.ellero@brickedbrain.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ced1401/ced_ioc.c
drivers/staging/ced1401/usb1401.h

index 47ef5ddcb4ce2c1b9d0f92878c305ac0336a43fb..80bc213f1604432a04f6c79cb79cf3381a7102d7 100644 (file)
@@ -939,17 +939,18 @@ int ced_test_event(struct ced_data *ced, int area)
 ** ced_get_transferInfo
 ** Puts the current state of the 1401 in a TGET_TX_BLOCK.
 *****************************************************************************/
-int ced_get_transfer(struct ced_data *ced, TGET_TX_BLOCK __user *pTX)
+int ced_get_transfer(struct ced_data *ced, TGET_TX_BLOCK __user *utx)
 {
-       int iReturn = U14ERR_NOERROR;
+       int ret = U14ERR_NOERROR;
        unsigned int dwIdent;
 
        mutex_lock(&ced->io_mutex);
        dwIdent = ced->staged_id;       /*  area ident for last xfer */
        if (dwIdent >= MAX_TRANSAREAS)
-               iReturn = U14ERR_BADAREA;
+               ret = U14ERR_BADAREA;
        else {
-               /*  Return the best information we have - we don't have physical addresses */
+               /* Return the best information we have - we */
+               /* don't have physical addresses            */
                TGET_TX_BLOCK *tx;
 
                tx = kzalloc(sizeof(*tx), GFP_KERNEL);
@@ -959,18 +960,19 @@ int ced_get_transfer(struct ced_data *ced, TGET_TX_BLOCK __user *pTX)
                }
                tx->size = ced->trans_def[dwIdent].length;
                tx->linear = (long long)((long)ced->trans_def[dwIdent].buff);
-               tx->avail = GET_TX_MAXENTRIES;  /*  how many blocks we could return */
+               /* how many blocks we could return */
+               tx->avail = GET_TX_MAXENTRIES;
                tx->used = 1;   /*  number we actually return */
                tx->entries[0].physical =
                    (long long)(tx->linear + ced->staged_offset);
                tx->entries[0].size = tx->size;
 
-               if (copy_to_user(pTX, tx, sizeof(*tx)))
-                       iReturn = -EFAULT;
+               if (copy_to_user(utx, tx, sizeof(*tx)))
+                       ret = -EFAULT;
                kfree(tx);
        }
        mutex_unlock(&ced->io_mutex);
-       return iReturn;
+       return ret;
 }
 
 /****************************************************************************
index 30dbe5bf9c8a461acfd805327e545e46d14526b4..c5a4118454a2f63d0adf2e1f71385f79e799ca53 100644 (file)
@@ -237,7 +237,7 @@ extern int ced_set_event(struct ced_data *ced,
 extern int ced_stat_1401(struct ced_data *ced);
 extern int ced_line_count(struct ced_data *ced);
 extern int ced_get_out_buf_space(struct ced_data *ced);
-extern int ced_get_transfer(struct ced_data *ced, TGET_TX_BLOCK __user *pGTB);
+extern int ced_get_transfer(struct ced_data *ced, TGET_TX_BLOCK __user *utx);
 extern int ced_kill_io(struct ced_data *ced);
 extern int ced_state_of_1401(struct ced_data *ced);
 extern int ced_start_self_test(struct ced_data *ced);