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

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 3da24a61282048dc6f4dbcd591d92a6465cafb9e..efc1db903a3a3543f08454d9f4e8abc0d336804c 100644 (file)
@@ -1373,29 +1373,32 @@ int ced_dbg_stop_loop(struct ced_data *ced)
 ** booked and a transfer to that area is in progress. Otherwise, we will
 ** release the area and re-assign it.
 ****************************************************************************/
-int ced_set_circular(struct ced_data *ced, struct transfer_area_desc __user *pTD)
+int ced_set_circular(struct ced_data *ced,
+                    struct transfer_area_desc __user *utd)
 {
-       int iReturn;
-       bool bToHost;
+       int ret;
+       bool to_host;
        struct transfer_area_desc td;
 
-       if (copy_from_user(&td, pTD, sizeof(td)))
+       if (copy_from_user(&td, utd, sizeof(td)))
                return -EFAULT;
 
        mutex_lock(&ced->io_mutex);
        dev_dbg(&ced->interface->dev, "%s: area:%d, size:%08x\n",
                __func__, td.wAreaNum, td.dwLength);
-       bToHost = td.eSize != 0;        /*  this is used as the tohost flag */
+       to_host = td.eSize != 0;        /*  this is used as the tohost flag */
 
-       /*  The strange cast is done so that we don't get warnings in 32-bit linux about the size of the */
-       /*  pointer. The pointer is always passed as a 64-bit object so that we don't have problems using */
-       /*  a 32-bit program on a 64-bit system. unsigned long is 64-bits on a 64-bit system. */
-       iReturn =
+       /* The strange cast is done so that we don't get warnings in 32-bit  */
+       /* linux about the size of the pointer. The pointer is always passed */
+       /* as a 64-bit object so that we don't have problems using a 32-bit  */
+       /* program on a 64-bit system. unsigned long is 64-bits on a 64-bit  */
+       /* system.                                                           */
+       ret =
            ced_set_area(ced, td.wAreaNum,
                    (char __user *)((unsigned long)td.lpvBuff), td.dwLength,
-                   true, bToHost);
+                   true, to_host);
        mutex_unlock(&ced->io_mutex);
-       return iReturn;
+       return ret;
 }
 
 /****************************************************************************
index c04f3857e134e31ccbbe29952d128289a2a1e706..a0d19aa6112747b2706bffa8451b6fe25e6e8b3c 100644 (file)
@@ -251,7 +251,8 @@ extern int ced_dbg_ramp_data(struct ced_data *ced, TDBGBLOCK __user *udb);
 extern int ced_dbg_ramp_addr(struct ced_data *ced, TDBGBLOCK __user *udb);
 extern int ced_dbg_get_data(struct ced_data *ced, TDBGBLOCK __user *udb);
 extern int ced_dbg_stop_loop(struct ced_data *ced);
-extern int ced_set_circular(struct ced_data *ced, struct transfer_area_desc __user *pTD);
+extern int ced_set_circular(struct ced_data *ced,
+                           struct transfer_area_desc __user *utd);
 extern int ced_get_circ_block(struct ced_data *ced, TCIRCBLOCK __user *pCB);
 extern int ced_free_circ_block(struct ced_data *ced, TCIRCBLOCK __user *pCB);
 extern int ced_wait_event(struct ced_data *ced, int area, int time_out);