linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: John Whitmore <johnfwhitmore@gmail.com>
To: linux-kernel@vger.kernel.org
Cc: devel@driverdev.osuosl.org, gregkh@linuxfoundation.org,
	John Whitmore <johnfwhitmore@gmail.com>
Subject: [PATCH 17/20] staging:rtl8192u: Rename RxAdmittedBARecord - Style
Date: Fri, 27 Jul 2018 18:31:20 +0100	[thread overview]
Message-ID: <20180727173123.27391-18-johnfwhitmore@gmail.com> (raw)
In-Reply-To: <20180727173123.27391-1-johnfwhitmore@gmail.com>

Rename the member variable RxAdmittedBARecord to rx_admitted_ba_record.
This change clears the checkpatch issue with CamelCase naming.

The resulting changes are coding style in nature and as such should have
no impact on runtime code execution.

Signed-off-by: John Whitmore <johnfwhitmore@gmail.com>
---
 drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c | 10 +++++-----
 drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h     |  2 +-
 drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c |  4 ++--
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c
index c2a51a6c2969..e296813203aa 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c
@@ -70,7 +70,7 @@ static u8 TxTsDeleteBA(struct ieee80211_device *ieee, struct tx_ts_record *pTxTs
  ********************************************************************************************************************/
 static u8 RxTsDeleteBA(struct ieee80211_device *ieee, struct rx_ts_record *pRxTs)
 {
-	PBA_RECORD		pBa = &pRxTs->RxAdmittedBARecord;
+	PBA_RECORD		pBa = &pRxTs->rx_admitted_ba_record;
 	u8			bSendDELBA = false;
 
 	if (pBa->bValid) {
@@ -369,7 +369,7 @@ int ieee80211_rx_ADDBAReq(struct ieee80211_device *ieee, struct sk_buff *skb)
 		IEEE80211_DEBUG(IEEE80211_DL_ERR, "can't get TS in %s()\n", __func__);
 		goto OnADDBAReq_Fail;
 	}
-	pBA = &pTS->RxAdmittedBARecord;
+	pBA = &pTS->rx_admitted_ba_record;
 	// To Determine the ADDBA Req content
 	// We can do much more check here, including BufferSize, AMSDU_Support, Policy, StartSeqCtrl...
 	// I want to check StartSeqCtrl to make sure when we start aggregation!!!
@@ -656,7 +656,7 @@ TsInitDelBA(struct ieee80211_device *ieee, struct ts_common_info *pTsCommonInfo,
 			ieee80211_send_DELBA(
 				ieee,
 				pTsCommonInfo->addr,
-				&pRxTs->RxAdmittedBARecord,
+				&pRxTs->rx_admitted_ba_record,
 				TxRxSelect,
 				DELBA_REASON_END_BA);
 	}
@@ -691,14 +691,14 @@ void TxBaInactTimeout(struct timer_list *t)
 
 void RxBaInactTimeout(struct timer_list *t)
 {
-	struct rx_ts_record *pRxTs = from_timer(pRxTs, t, RxAdmittedBARecord.Timer);
+	struct rx_ts_record *pRxTs = from_timer(pRxTs, t, rx_admitted_ba_record.Timer);
 	struct ieee80211_device *ieee = container_of(pRxTs, struct ieee80211_device, RxTsRecord[pRxTs->num]);
 
 	RxTsDeleteBA(ieee, pRxTs);
 	ieee80211_send_DELBA(
 		ieee,
 		pRxTs->ts_common_info.addr,
-		&pRxTs->RxAdmittedBARecord,
+		&pRxTs->rx_admitted_ba_record,
 		RX_DIR,
 		DELBA_REASON_TIMEOUT);
 }
diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
index a338d53ff88c..7cc13e5dfb2d 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h
@@ -43,7 +43,7 @@ struct rx_ts_record {
 	u16				rx_timeout_indicate_seq;
 	struct list_head		rx_pending_pkt_list;
 	struct timer_list		rx_pkt_pending_timer;
-	BA_RECORD			RxAdmittedBARecord;	 /*  For BA Recipient */
+	BA_RECORD			rx_admitted_ba_record;	 /*  For BA Recipient */
 	u16				RxLastSeqNum;
 	u8				RxLastFragNum;
 	u8				num;
diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c
index aad2704a7f46..a44536d8cd42 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c
@@ -126,7 +126,7 @@ static void ResetRxTsEntry(struct rx_ts_record *pTS)
 	ResetTsCommonInfo(&pTS->ts_common_info);
 	pTS->rx_indicate_seq = 0xffff; // This indicate the rx_indicate_seq is not used now!!
 	pTS->rx_timeout_indicate_seq = 0xffff; // This indicate the rx_timeout_indicate_seq is not used now!!
-	ResetBaEntry(&pTS->RxAdmittedBARecord);	  // For BA Recipient
+	ResetBaEntry(&pTS->rx_admitted_ba_record);	  // For BA Recipient
 }
 
 void TSInitialize(struct ieee80211_device *ieee)
@@ -171,7 +171,7 @@ void TSInitialize(struct ieee80211_device *ieee)
 			    0);
 		timer_setup(&pRxTS->ts_common_info.inact_timer, TsInactTimeout,
 			    0);
-		timer_setup(&pRxTS->RxAdmittedBARecord.Timer,
+		timer_setup(&pRxTS->rx_admitted_ba_record.Timer,
 			    RxBaInactTimeout, 0);
 		timer_setup(&pRxTS->rx_pkt_pending_timer, RxPktPendingTimeout, 0);
 		ResetRxTsEntry(pRxTS);
-- 
2.18.0


  parent reply	other threads:[~2018-07-27 17:32 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-27 17:31 [PATCH 00/20] staging:rtl8192u Header file cleanup - Style John Whitmore
2018-07-27 17:31 ` [PATCH 01/20] staging:rtl8192u: Remove typedef and rename TX_TS_RECORD " John Whitmore
2018-07-27 17:31 ` [PATCH 02/20] staging:rtl8192u: Rename TsCommonInfo " John Whitmore
2018-07-27 17:31 ` [PATCH 03/20] staging:rtl8192u: Rename TxCurSeq " John Whitmore
2018-07-27 17:31 ` [PATCH 04/20] staging:rtl8192u: Rename TxPendingBARecord " John Whitmore
2018-07-27 17:31 ` [PATCH 05/20] staging:rtl8192u: Rename TxAdmittedBARecord " John Whitmore
2018-07-27 17:31 ` [PATCH 06/20] staging:rtl8192u: Rename bAddBaReqInProgress " John Whitmore
2018-07-27 17:31 ` [PATCH 07/20] staging:rtl8192u: Rename bAddBaReqDelayed " John Whitmore
2018-07-27 17:31 ` [PATCH 08/20] staging:rtl8192u: Rename bUsingBa " John Whitmore
2018-07-27 17:31 ` [PATCH 09/20] staging:rtl8192u: Rename TsAddBaTimer " John Whitmore
2018-07-27 17:31 ` [PATCH 10/20] staging:rtl8192u: Remove typedef and rename RX_TS_RECORD " John Whitmore
2018-07-27 17:31 ` [PATCH 11/20] staging:rtl8192u: Remove commented out code " John Whitmore
2018-07-27 17:31 ` [PATCH 12/20] staging:rtl8192u: Remove unused timer values " John Whitmore
2018-07-27 17:31 ` [PATCH 13/20] staging:rtl8192u: Rename RxIndicateSeq " John Whitmore
2018-07-27 17:31 ` [PATCH 14/20] staging:rtl8192u: Rename RxTimeoutIndicateSeq _Style John Whitmore
2018-07-27 17:31 ` [PATCH 15/20] staging:rtl8192u: Rename RxPendingPktList - Style John Whitmore
2018-07-27 17:31 ` [PATCH 16/20] staging:rtl8192u: Rename RxPktPendingTimer " John Whitmore
2018-07-27 17:31 ` John Whitmore [this message]
2018-07-27 17:31 ` [PATCH 18/20] staging:rtl8192u: Rename RxLastSeqNum " John Whitmore
2018-07-27 17:31 ` [PATCH 19/20] staging:rtl8192u: Rename RxLastFragNum " John Whitmore
2018-07-27 17:31 ` [PATCH 20/20] staging:rtl8192u: Remove blank line " John Whitmore

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=20180727173123.27391-18-johnfwhitmore@gmail.com \
    --to=johnfwhitmore@gmail.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).