All of lore.kernel.org
 help / color / mirror / Atom feed
From: Leo Kim <leo.kim@atmel.com>
To: <gregkh@linuxfoundation.org>
Cc: <devel@driverdev.osuosl.org>, <linux-wireless@vger.kernel.org>,
	<tony.cho@atmel.com>, <glen.lee@atmel.com>, <leo.kim@atmel.com>,
	<austin.shin@atmel.com>, <Chris.Park@atmel.com>,
	<adham.abozaeid@atmel.com>, <Nicolas.FERRE@atmel.com>
Subject: [PATCH 09/12] staging: wilc1000: wilc_parse_assoc_resp_info(): renames function variables
Date: Thu, 28 Jan 2016 16:13:43 +0900	[thread overview]
Message-ID: <1453965226-6158-9-git-send-email-leo.kim@atmel.com> (raw)
In-Reply-To: <1453965226-6158-1-git-send-email-leo.kim@atmel.com>

This patch renames to avoid camelcase, changes follow are:
 - pu8Buffer to buffer
 - u32BufferLen to buffer_len
 - ppstrConnectRespInfo to ret_connect_resp_info

Signed-off-by: Leo Kim <leo.kim@atmel.com>
---
 drivers/staging/wilc1000/coreconfigurator.c | 16 ++++++++--------
 drivers/staging/wilc1000/coreconfigurator.h |  5 ++---
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000/coreconfigurator.c
index 88e5661..48c8573 100644
--- a/drivers/staging/wilc1000/coreconfigurator.c
+++ b/drivers/staging/wilc1000/coreconfigurator.c
@@ -376,8 +376,8 @@ s32 wilc_dealloc_network_info(tstrNetworkInfo *pstrNetworkInfo)
 	return result;
 }
 
-s32 wilc_parse_assoc_resp_info(u8 *pu8Buffer, u32 u32BufferLen,
-			       tstrConnectRespInfo **ppstrConnectRespInfo)
+s32 wilc_parse_assoc_resp_info(u8 *buffer, u32 buffer_len,
+			       tstrConnectRespInfo **ret_connect_resp_info)
 {
 	tstrConnectRespInfo *pstrConnectRespInfo = NULL;
 	u16 u16AssocRespLen = 0;
@@ -388,14 +388,14 @@ s32 wilc_parse_assoc_resp_info(u8 *pu8Buffer, u32 u32BufferLen,
 	if (!pstrConnectRespInfo)
 		return -ENOMEM;
 
-	u16AssocRespLen = (u16)u32BufferLen;
+	u16AssocRespLen = (u16)buffer_len;
 
-	pstrConnectRespInfo->u16ConnectStatus = get_asoc_status(pu8Buffer);
+	pstrConnectRespInfo->u16ConnectStatus = get_asoc_status(buffer);
 	if (pstrConnectRespInfo->u16ConnectStatus == SUCCESSFUL_STATUSCODE) {
-		pstrConnectRespInfo->u16capability = get_assoc_resp_cap_info(pu8Buffer);
-		pstrConnectRespInfo->u16AssocID = get_asoc_id(pu8Buffer);
+		pstrConnectRespInfo->u16capability = get_assoc_resp_cap_info(buffer);
+		pstrConnectRespInfo->u16AssocID = get_asoc_id(buffer);
 
-		pu8IEs = &pu8Buffer[CAP_INFO_LEN + STATUS_CODE_LEN + AID_LEN];
+		pu8IEs = &buffer[CAP_INFO_LEN + STATUS_CODE_LEN + AID_LEN];
 		u16IEsLen = u16AssocRespLen - (CAP_INFO_LEN + STATUS_CODE_LEN + AID_LEN);
 
 		pstrConnectRespInfo->pu8RespIEs = kmemdup(pu8IEs, u16IEsLen, GFP_KERNEL);
@@ -405,7 +405,7 @@ s32 wilc_parse_assoc_resp_info(u8 *pu8Buffer, u32 u32BufferLen,
 		pstrConnectRespInfo->u16RespIEsLen = u16IEsLen;
 	}
 
-	*ppstrConnectRespInfo = pstrConnectRespInfo;
+	*ret_connect_resp_info = pstrConnectRespInfo;
 
 	return 0;
 }
diff --git a/drivers/staging/wilc1000/coreconfigurator.h b/drivers/staging/wilc1000/coreconfigurator.h
index aec0779..6a2c323 100644
--- a/drivers/staging/wilc1000/coreconfigurator.h
+++ b/drivers/staging/wilc1000/coreconfigurator.h
@@ -122,9 +122,8 @@ typedef struct {
 
 s32 wilc_parse_network_info(u8 *msg_buffer, tstrNetworkInfo **ret_network_info);
 s32 wilc_dealloc_network_info(tstrNetworkInfo *pstrNetworkInfo);
-
-s32 wilc_parse_assoc_resp_info(u8 *pu8Buffer, u32 u32BufferLen,
-		       tstrConnectRespInfo **ppstrConnectRespInfo);
+s32 wilc_parse_assoc_resp_info(u8 *buffer, u32 buffer_len,
+			       tstrConnectRespInfo **ret_connect_resp_info);
 s32 wilc_dealloc_assoc_resp_info(tstrConnectRespInfo *pstrConnectRespInfo);
 void wilc_scan_complete_received(struct wilc *wilc, u8 *pu8Buffer,
 				 u32 u32Length);
-- 
1.9.1


  parent reply	other threads:[~2016-01-28  7:16 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-28  7:13 [PATCH 01/12] staging: wilc1000: coreconfigurator.c : remove over-commenting Leo Kim
2016-01-28  7:13 ` [PATCH 02/12] staging: wilc1000: renames u16RxLen variable Leo Kim
2016-01-28  7:13 ` [PATCH 03/12] staging: wilc1000: renames u16TagParamOffset variable Leo Kim
2016-01-28  7:13 ` [PATCH 04/12] staging: wilc1000: renames u16index variable Leo Kim
2016-01-28  7:13 ` [PATCH 05/12] staging: wilc1000: wilc_parse_network_info(): renames function variables Leo Kim
2016-02-03 23:30   ` Greg KH
2016-01-28  7:13 ` [PATCH 06/12] staging: wilc1000: wilc_parse_network_info(): renames local variables Leo Kim
2016-01-28  7:13 ` [PATCH 07/12] staging: wilc1000: wilc_parse_network_info(): renames local inner variables Leo Kim
2016-01-28  7:13 ` [PATCH 08/12] staging: wilc1000: rename variable s32Error Leo Kim
2016-01-29  0:55   ` Julian Calaby
2016-01-29  2:55     ` Kim, Leo
2016-01-29  2:58       ` Julian Calaby
2016-01-28  7:13 ` Leo Kim [this message]
2016-01-28  7:13 ` [PATCH 10/12] staging: wilc1000: wilc_parse_assoc_resp_info(): renames local variables Leo Kim
2016-01-28  7:13 ` [PATCH 11/12] staging: wilc1000: wilc_dealloc_assoc_resp_info(): renames function variables Leo Kim
2016-01-28  7:13 ` [PATCH 12/12] staging: wilc1000: wilc_dealloc_network_info(): " Leo Kim

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1453965226-6158-9-git-send-email-leo.kim@atmel.com \
    --to=leo.kim@atmel.com \
    --cc=Chris.Park@atmel.com \
    --cc=Nicolas.FERRE@atmel.com \
    --cc=adham.abozaeid@atmel.com \
    --cc=austin.shin@atmel.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=glen.lee@atmel.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=tony.cho@atmel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.