[PATCH 093/115] staging: ced1401: fix ced_set_event()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Rename camel case arguments and locals in function ced_set_event()

Signed-off-by: Luca Ellero <luca.ellero@xxxxxxxxxxxxxxxx>
---
 drivers/staging/ced1401/ced_ioc.c |   33 ++++++++++++++++++++-------------
 drivers/staging/ced1401/usb1401.h |    3 ++-
 2 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/drivers/staging/ced1401/ced_ioc.c b/drivers/staging/ced1401/ced_ioc.c
index 74a765d..751dadb 100644
--- a/drivers/staging/ced1401/ced_ioc.c
+++ b/drivers/staging/ced1401/ced_ioc.c
@@ -821,33 +821,40 @@ int ced_unset_transfer(struct ced_data *ced, int area)
 ** pretend that whatever the user asked for was achieved, so we return 1 if
 ** try to create one, and 0 if they ask to remove (assuming all else was OK).
 ****************************************************************************/
-int ced_set_event(struct ced_data *ced, struct transfer_event __user *pTE)
+int ced_set_event(struct ced_data *ced, struct transfer_event __user *ute)
 {
-	int iReturn = U14ERR_NOERROR;
+	int ret = U14ERR_NOERROR;
 	struct transfer_event te;
 
 	/*  get a local copy of the data */
-	if (copy_from_user(&te, pTE, sizeof(te)))
+	if (copy_from_user(&te, ute, sizeof(te)))
 		return -EFAULT;
 
 	if (te.wAreaNum >= MAX_TRANSAREAS)	/*  the area must exist */
 		return U14ERR_BADAREA;
 	else {
-		struct transarea *pTA = &ced->trans_def[te.wAreaNum];
-		mutex_lock(&ced->io_mutex);	/*  make sure we have no competitor */
+		struct transarea *ta = &ced->trans_def[te.wAreaNum];
+
+		/* make sure we have no competitor */
+		mutex_lock(&ced->io_mutex);
 		spin_lock_irq(&ced->staged_lock);
-		if (pTA->used) {	/*  area must be in use */
-			pTA->event_st = te.dwStart;	/*  set area regions */
-			pTA->event_sz = te.dwLength;	/*  set size (0 cancels it) */
-			pTA->event_to_host = te.wFlags & 1;	/*  set the direction */
-			pTA->wake_up = 0;	/*  zero the wake up count */
+
+		if (ta->used) {	/* area must be in use */
+			ta->event_st = te.dwStart; /*  set area regions */
+
+			 /* set size (0 cancels it) */
+			ta->event_sz = te.dwLength;
+
+			 /*  set the direction */
+			ta->event_to_host = te.wFlags & 1;
+			ta->wake_up = 0;	/*  zero the wake up count */
 		} else
-			iReturn = U14ERR_NOTSET;
+			ret = U14ERR_NOTSET;
 		spin_unlock_irq(&ced->staged_lock);
 		mutex_unlock(&ced->io_mutex);
 	}
-	return iReturn ==
-	    U14ERR_NOERROR ? (te.iSetEvent ? 1 : U14ERR_NOERROR) : iReturn;
+	return ret ==
+	    U14ERR_NOERROR ? (te.iSetEvent ? 1 : U14ERR_NOERROR) : ret;
 }
 
 /****************************************************************************
diff --git a/drivers/staging/ced1401/usb1401.h b/drivers/staging/ced1401/usb1401.h
index bb58b79..6ae2e86 100644
--- a/drivers/staging/ced1401/usb1401.h
+++ b/drivers/staging/ced1401/usb1401.h
@@ -232,7 +232,8 @@ 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 *utd);
 extern int ced_unset_transfer(struct ced_data *ced, int area);
-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 *ute);
 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);
-- 
1.7.10.4

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux