[PATCH v2 16/60] staging: ced1401: fix ced_read_huff()

Luca Ellero luca.ellero at brickedbrain.com
Thu Jul 10 09:02:00 UTC 2014


Rename camel case arguments and locals in function ced_read_huff()

Signed-off-by: Luca Ellero <luca.ellero at brickedbrain.com>
---
 drivers/staging/ced1401/usb1401.c |   45 ++++++++++++++++++++-----------------
 1 file changed, 24 insertions(+), 21 deletions(-)

diff --git a/drivers/staging/ced1401/usb1401.c b/drivers/staging/ced1401/usb1401.c
index d062d5d..0078142 100644
--- a/drivers/staging/ced1401/usb1401.c
+++ b/drivers/staging/ced1401/usb1401.c
@@ -984,35 +984,38 @@ static bool ced_read_word(unsigned short *word, char *buf, unsigned int *n_done,
 ** to indicate three byte total.
 **
 *****************************************************************************/
-static bool ced_read_huff(volatile unsigned int *pDWord, char *pBuf,
-		     unsigned int *pdDone, unsigned int dGot)
+static bool ced_read_huff(volatile unsigned int *word, char *buf,
+		     unsigned int *n_done, unsigned int got)
 {
-	unsigned char ucData;	/* for each read to ced_read_char */
-	bool bReturn = true;	/* assume we will succeed */
-	unsigned int dwData = 0;	/* Accumulator for the data */
-
-	if (ced_read_char(&ucData, pBuf, pdDone, dGot)) {
-		dwData = ucData;	/* copy the data */
-		if ((dwData & 0x00000080) != 0) {	/* Bit set for more data ? */
-			dwData &= 0x0000007F;	/* Clear the relevant bit */
-			if (ced_read_char(&ucData, pBuf, pdDone, dGot)) {
-				dwData = (dwData << 8) | ucData;
-				if ((dwData & 0x00004000) != 0) {	/* three byte sequence ? */
-					dwData &= 0x00003FFF;	/* Clear the relevant bit */
+	unsigned char c;	/* for each read to ced_read_char */
+	bool retval = true;	/* assume we will succeed */
+	unsigned int data = 0;	/* Accumulator for the data */
+
+	if (ced_read_char(&c, buf, n_done, got)) {
+		data = c;	/* copy the data */
+		if ((data & 0x00000080) != 0) {	/* Bit set for more data ? */
+			data &= 0x0000007F;	/* Clear the relevant bit */
+			if (ced_read_char(&c, buf, n_done, got)) {
+				data = (data << 8) | c;
+
+				/* three byte sequence ? */
+				if ((data & 0x00004000) != 0) {
+					/* Clear the relevant bit */
+					data &= 0x00003FFF;
 					if (ced_read_char
-					    (&ucData, pBuf, pdDone, dGot))
-						dwData = (dwData << 8) | ucData;
+					    (&c, buf, n_done, got))
+						data = (data << 8) | c;
 					else
-						bReturn = false;
+						retval = false;
 				}
 			} else
-				bReturn = false;	/* couldn't read data */
+				retval = false;	/* couldn't read data */
 		}
 	} else
-		bReturn = false;
+		retval = false;
 
-	*pDWord = dwData;	/* return the data */
-	return bReturn;
+	*word = data;	/* return the data */
+	return retval;
 }
 
 /***************************************************************************
-- 
1.7.10.4



More information about the devel mailing list