mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/5] Staging: rtl8192e: Function name cleanup series 2
@ 2023-08-07  5:47 Tree Davies
  2023-08-07  5:47 ` [PATCH 1/5] Staging: rtl8192e: Rename function TsInitAddBA Tree Davies
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Tree Davies @ 2023-08-07  5:47 UTC (permalink / raw)
  To: gregkh, philipp.g.hortmann, anjan, error27
  Cc: linux-staging, linux-kernel, Tree Davies

This patch series renames functions to avoid checkpatch warning: 
Avoid CamelCase.

Thank you in advance to the reviewers.
~ Tree

Tree Davies (5):
  Staging: rtl8192e: Rename function TsInitAddBA
  Staging: rtl8192e: Rename function TsInitDelBA
  Staging: rtl8192e: Rename function BaSetupTimeOut
  Staging: rtl8192e: Rename function TxBaInactTimeout
  Staging: rtl8192e: Rename function RxBaInactTimeout

 drivers/staging/rtl8192e/rtl819x_BAProc.c | 16 ++++++++--------
 drivers/staging/rtl8192e/rtl819x_TSProc.c | 10 +++++-----
 drivers/staging/rtl8192e/rtllib.h         | 16 ++++++++--------
 3 files changed, 21 insertions(+), 21 deletions(-)

-- 
2.41.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/5] Staging: rtl8192e: Rename function TsInitAddBA
  2023-08-07  5:47 [PATCH 0/5] Staging: rtl8192e: Function name cleanup series 2 Tree Davies
@ 2023-08-07  5:47 ` Tree Davies
  2023-08-07  5:47 ` [PATCH 2/5] Staging: rtl8192e: Rename function TsInitDelBA Tree Davies
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Tree Davies @ 2023-08-07  5:47 UTC (permalink / raw)
  To: gregkh, philipp.g.hortmann, anjan, error27
  Cc: linux-staging, linux-kernel, Tree Davies

Rename function TsInitAddBA to rtllib_ts_init_add_ba in order to Fix
checkpatch warning: Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
---
 drivers/staging/rtl8192e/rtl819x_BAProc.c | 4 ++--
 drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
 drivers/staging/rtl8192e/rtllib.h         | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAProc.c
index 600802c795b9..9605d21bbf40 100644
--- a/drivers/staging/rtl8192e/rtl819x_BAProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_BAProc.c
@@ -461,8 +461,8 @@ int rtllib_rx_DELBA(struct rtllib_device *ieee, struct sk_buff *skb)
 	return 0;
 }
 
-void TsInitAddBA(struct rtllib_device *ieee, struct tx_ts_record *pTS,
-		 u8 Policy, u8	bOverwritePending)
+void rtllib_ts_init_add_ba(struct rtllib_device *ieee, struct tx_ts_record *pTS,
+			   u8 Policy, u8	bOverwritePending)
 {
 	struct ba_record *pBA = &pTS->TxPendingBARecord;
 
diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
index 14e0c000a31a..a36f175ba230 100644
--- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
@@ -88,7 +88,7 @@ static void TsAddBaProcess(struct timer_list *t)
 	struct rtllib_device *ieee = container_of(pTxTs, struct rtllib_device,
 				     TxTsRecord[num]);
 
-	TsInitAddBA(ieee, pTxTs, BA_POLICY_IMMEDIATE, false);
+	rtllib_ts_init_add_ba(ieee, pTxTs, BA_POLICY_IMMEDIATE, false);
 	netdev_dbg(ieee->dev, "%s(): ADDBA Req is started\n", __func__);
 }
 
diff --git a/drivers/staging/rtl8192e/rtllib.h b/drivers/staging/rtl8192e/rtllib.h
index c5a692dfcd17..c38dc5ae0b72 100644
--- a/drivers/staging/rtl8192e/rtllib.h
+++ b/drivers/staging/rtl8192e/rtllib.h
@@ -2002,8 +2002,8 @@ u16  TxCountToDataRate(struct rtllib_device *ieee, u8 nDataRate);
 int rtllib_rx_ADDBAReq(struct rtllib_device *ieee, struct sk_buff *skb);
 int rtllib_rx_ADDBARsp(struct rtllib_device *ieee, struct sk_buff *skb);
 int rtllib_rx_DELBA(struct rtllib_device *ieee, struct sk_buff *skb);
-void TsInitAddBA(struct rtllib_device *ieee, struct tx_ts_record *pTS,
-		 u8 Policy, u8 bOverwritePending);
+void rtllib_ts_init_add_ba(struct rtllib_device *ieee, struct tx_ts_record *pTS,
+			   u8 Policy, u8 bOverwritePending);
 void TsInitDelBA(struct rtllib_device *ieee,
 		 struct ts_common_info *pTsCommonInfo,
 		 enum tr_select TxRxSelect);
-- 
2.41.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 2/5] Staging: rtl8192e: Rename function TsInitDelBA
  2023-08-07  5:47 [PATCH 0/5] Staging: rtl8192e: Function name cleanup series 2 Tree Davies
  2023-08-07  5:47 ` [PATCH 1/5] Staging: rtl8192e: Rename function TsInitAddBA Tree Davies
@ 2023-08-07  5:47 ` Tree Davies
  2023-08-07  5:47 ` [PATCH 3/5] Staging: rtl8192e: Rename function BaSetupTimeOut Tree Davies
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Tree Davies @ 2023-08-07  5:47 UTC (permalink / raw)
  To: gregkh, philipp.g.hortmann, anjan, error27
  Cc: linux-staging, linux-kernel, Tree Davies

Rename function TsInitDelBA to rtllib_ts_init_del_ba in order to Fix
checkpatch warning: Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
---
 drivers/staging/rtl8192e/rtl819x_BAProc.c | 6 +++---
 drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
 drivers/staging/rtl8192e/rtllib.h         | 6 +++---
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAProc.c
index 9605d21bbf40..19c845ec863c 100644
--- a/drivers/staging/rtl8192e/rtl819x_BAProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_BAProc.c
@@ -484,9 +484,9 @@ void rtllib_ts_init_add_ba(struct rtllib_device *ieee, struct tx_ts_record *pTS,
 	rtllib_send_ADDBAReq(ieee, pTS->TsCommonInfo.Addr, pBA);
 }
 
-void TsInitDelBA(struct rtllib_device *ieee,
-		 struct ts_common_info *pTsCommonInfo,
-		 enum tr_select TxRxSelect)
+void rtllib_ts_init_del_ba(struct rtllib_device *ieee,
+			   struct ts_common_info *pTsCommonInfo,
+			   enum tr_select TxRxSelect)
 {
 	if (TxRxSelect == TX_DIR) {
 		struct tx_ts_record *pTxTs =
diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
index a36f175ba230..759778787afa 100644
--- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
@@ -361,7 +361,7 @@ bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS,
 static void RemoveTsEntry(struct rtllib_device *ieee,
 			  struct ts_common_info *pTs, enum tr_select TxRxSelect)
 {
-	TsInitDelBA(ieee, pTs, TxRxSelect);
+	rtllib_ts_init_del_ba(ieee, pTs, TxRxSelect);
 
 	if (TxRxSelect == RX_DIR) {
 		struct rx_reorder_entry *pRxReorderEntry;
diff --git a/drivers/staging/rtl8192e/rtllib.h b/drivers/staging/rtl8192e/rtllib.h
index c38dc5ae0b72..a4238db2fd56 100644
--- a/drivers/staging/rtl8192e/rtllib.h
+++ b/drivers/staging/rtl8192e/rtllib.h
@@ -2004,9 +2004,9 @@ int rtllib_rx_ADDBARsp(struct rtllib_device *ieee, struct sk_buff *skb);
 int rtllib_rx_DELBA(struct rtllib_device *ieee, struct sk_buff *skb);
 void rtllib_ts_init_add_ba(struct rtllib_device *ieee, struct tx_ts_record *pTS,
 			   u8 Policy, u8 bOverwritePending);
-void TsInitDelBA(struct rtllib_device *ieee,
-		 struct ts_common_info *pTsCommonInfo,
-		 enum tr_select TxRxSelect);
+void rtllib_ts_init_del_ba(struct rtllib_device *ieee,
+			   struct ts_common_info *pTsCommonInfo,
+			   enum tr_select TxRxSelect);
 void BaSetupTimeOut(struct timer_list *t);
 void TxBaInactTimeout(struct timer_list *t);
 void RxBaInactTimeout(struct timer_list *t);
-- 
2.41.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 3/5] Staging: rtl8192e: Rename function BaSetupTimeOut
  2023-08-07  5:47 [PATCH 0/5] Staging: rtl8192e: Function name cleanup series 2 Tree Davies
  2023-08-07  5:47 ` [PATCH 1/5] Staging: rtl8192e: Rename function TsInitAddBA Tree Davies
  2023-08-07  5:47 ` [PATCH 2/5] Staging: rtl8192e: Rename function TsInitDelBA Tree Davies
@ 2023-08-07  5:47 ` Tree Davies
  2023-08-07  5:47 ` [PATCH 4/5] Staging: rtl8192e: Rename function TxBaInactTimeout Tree Davies
  2023-08-07  5:47 ` [PATCH 5/5] Staging: rtl8192e: Rename function RxBaInactTimeout Tree Davies
  4 siblings, 0 replies; 6+ messages in thread
From: Tree Davies @ 2023-08-07  5:47 UTC (permalink / raw)
  To: gregkh, philipp.g.hortmann, anjan, error27
  Cc: linux-staging, linux-kernel, Tree Davies

Rename function BaSetupTimeOut to rtllib_ba_setup_timeout in order to Fix
checkpatch warning: Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
---
 drivers/staging/rtl8192e/rtl819x_BAProc.c | 2 +-
 drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
 drivers/staging/rtl8192e/rtllib.h         | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAProc.c
index 19c845ec863c..f07c9e7ed893 100644
--- a/drivers/staging/rtl8192e/rtl819x_BAProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_BAProc.c
@@ -508,7 +508,7 @@ void rtllib_ts_init_del_ba(struct rtllib_device *ieee,
 	}
 }
 
-void BaSetupTimeOut(struct timer_list *t)
+void rtllib_ba_setup_timeout(struct timer_list *t)
 {
 	struct tx_ts_record *pTxTs = from_timer(pTxTs, t,
 					      TxPendingBARecord.timer);
diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
index 759778787afa..1784fa676a65 100644
--- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
@@ -136,7 +136,7 @@ void TSInitialize(struct rtllib_device *ieee)
 		pTxTS->num = count;
 		timer_setup(&pTxTS->TsAddBaTimer, TsAddBaProcess, 0);
 
-		timer_setup(&pTxTS->TxPendingBARecord.timer, BaSetupTimeOut,
+		timer_setup(&pTxTS->TxPendingBARecord.timer, rtllib_ba_setup_timeout,
 			    0);
 		timer_setup(&pTxTS->TxAdmittedBARecord.timer,
 			    TxBaInactTimeout, 0);
diff --git a/drivers/staging/rtl8192e/rtllib.h b/drivers/staging/rtl8192e/rtllib.h
index a4238db2fd56..53bdf2c429a7 100644
--- a/drivers/staging/rtl8192e/rtllib.h
+++ b/drivers/staging/rtl8192e/rtllib.h
@@ -2007,7 +2007,7 @@ void rtllib_ts_init_add_ba(struct rtllib_device *ieee, struct tx_ts_record *pTS,
 void rtllib_ts_init_del_ba(struct rtllib_device *ieee,
 			   struct ts_common_info *pTsCommonInfo,
 			   enum tr_select TxRxSelect);
-void BaSetupTimeOut(struct timer_list *t);
+void rtllib_ba_setup_timeout(struct timer_list *t);
 void TxBaInactTimeout(struct timer_list *t);
 void RxBaInactTimeout(struct timer_list *t);
 void rtllib_reset_ba_entry(struct ba_record *pBA);
-- 
2.41.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 4/5] Staging: rtl8192e: Rename function TxBaInactTimeout
  2023-08-07  5:47 [PATCH 0/5] Staging: rtl8192e: Function name cleanup series 2 Tree Davies
                   ` (2 preceding siblings ...)
  2023-08-07  5:47 ` [PATCH 3/5] Staging: rtl8192e: Rename function BaSetupTimeOut Tree Davies
@ 2023-08-07  5:47 ` Tree Davies
  2023-08-07  5:47 ` [PATCH 5/5] Staging: rtl8192e: Rename function RxBaInactTimeout Tree Davies
  4 siblings, 0 replies; 6+ messages in thread
From: Tree Davies @ 2023-08-07  5:47 UTC (permalink / raw)
  To: gregkh, philipp.g.hortmann, anjan, error27
  Cc: linux-staging, linux-kernel, Tree Davies

Rename function TxBaInactTimeout to rtllib_tx_ba_inact_timeout in order
to Fix checkpatch warning: Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
---
 drivers/staging/rtl8192e/rtl819x_BAProc.c | 2 +-
 drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
 drivers/staging/rtl8192e/rtllib.h         | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAProc.c
index f07c9e7ed893..1eac56fa97f9 100644
--- a/drivers/staging/rtl8192e/rtl819x_BAProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_BAProc.c
@@ -518,7 +518,7 @@ void rtllib_ba_setup_timeout(struct timer_list *t)
 	pTxTs->TxPendingBARecord.b_valid = false;
 }
 
-void TxBaInactTimeout(struct timer_list *t)
+void rtllib_tx_ba_inact_timeout(struct timer_list *t)
 {
 	struct tx_ts_record *pTxTs = from_timer(pTxTs, t,
 					      TxAdmittedBARecord.timer);
diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
index 1784fa676a65..c57bc0c8816b 100644
--- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
@@ -139,7 +139,7 @@ void TSInitialize(struct rtllib_device *ieee)
 		timer_setup(&pTxTS->TxPendingBARecord.timer, rtllib_ba_setup_timeout,
 			    0);
 		timer_setup(&pTxTS->TxAdmittedBARecord.timer,
-			    TxBaInactTimeout, 0);
+			    rtllib_tx_ba_inact_timeout, 0);
 
 		ResetTxTsEntry(pTxTS);
 		list_add_tail(&pTxTS->TsCommonInfo.List,
diff --git a/drivers/staging/rtl8192e/rtllib.h b/drivers/staging/rtl8192e/rtllib.h
index 53bdf2c429a7..3383dab0d1ca 100644
--- a/drivers/staging/rtl8192e/rtllib.h
+++ b/drivers/staging/rtl8192e/rtllib.h
@@ -2008,7 +2008,7 @@ void rtllib_ts_init_del_ba(struct rtllib_device *ieee,
 			   struct ts_common_info *pTsCommonInfo,
 			   enum tr_select TxRxSelect);
 void rtllib_ba_setup_timeout(struct timer_list *t);
-void TxBaInactTimeout(struct timer_list *t);
+void rtllib_tx_ba_inact_timeout(struct timer_list *t);
 void RxBaInactTimeout(struct timer_list *t);
 void rtllib_reset_ba_entry(struct ba_record *pBA);
 bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS, u8 *Addr,
-- 
2.41.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 5/5] Staging: rtl8192e: Rename function RxBaInactTimeout
  2023-08-07  5:47 [PATCH 0/5] Staging: rtl8192e: Function name cleanup series 2 Tree Davies
                   ` (3 preceding siblings ...)
  2023-08-07  5:47 ` [PATCH 4/5] Staging: rtl8192e: Rename function TxBaInactTimeout Tree Davies
@ 2023-08-07  5:47 ` Tree Davies
  4 siblings, 0 replies; 6+ messages in thread
From: Tree Davies @ 2023-08-07  5:47 UTC (permalink / raw)
  To: gregkh, philipp.g.hortmann, anjan, error27
  Cc: linux-staging, linux-kernel, Tree Davies

Rename function RxBaInactTimeout to rtllib_rx_ba_inact_timeout in order
to Fix checkpatch warning: Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
---
 drivers/staging/rtl8192e/rtl819x_BAProc.c | 2 +-
 drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
 drivers/staging/rtl8192e/rtllib.h         | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAProc.c
index 1eac56fa97f9..0e3372868f97 100644
--- a/drivers/staging/rtl8192e/rtl819x_BAProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_BAProc.c
@@ -530,7 +530,7 @@ void rtllib_tx_ba_inact_timeout(struct timer_list *t)
 			  DELBA_REASON_TIMEOUT);
 }
 
-void RxBaInactTimeout(struct timer_list *t)
+void rtllib_rx_ba_inact_timeout(struct timer_list *t)
 {
 	struct rx_ts_record *pRxTs = from_timer(pRxTs, t,
 					      rx_admitted_ba_record.timer);
diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
index c57bc0c8816b..24a8b9fc0168 100644
--- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
@@ -154,7 +154,7 @@ void TSInitialize(struct rtllib_device *ieee)
 		pRxTS->num = count;
 		INIT_LIST_HEAD(&pRxTS->rx_pending_pkt_list);
 		timer_setup(&pRxTS->rx_admitted_ba_record.timer,
-			    RxBaInactTimeout, 0);
+			    rtllib_rx_ba_inact_timeout, 0);
 
 		timer_setup(&pRxTS->rx_pkt_pending_timer, RxPktPendingTimeout, 0);
 
diff --git a/drivers/staging/rtl8192e/rtllib.h b/drivers/staging/rtl8192e/rtllib.h
index 3383dab0d1ca..0c812eb02ba6 100644
--- a/drivers/staging/rtl8192e/rtllib.h
+++ b/drivers/staging/rtl8192e/rtllib.h
@@ -2009,7 +2009,7 @@ void rtllib_ts_init_del_ba(struct rtllib_device *ieee,
 			   enum tr_select TxRxSelect);
 void rtllib_ba_setup_timeout(struct timer_list *t);
 void rtllib_tx_ba_inact_timeout(struct timer_list *t);
-void RxBaInactTimeout(struct timer_list *t);
+void rtllib_rx_ba_inact_timeout(struct timer_list *t);
 void rtllib_reset_ba_entry(struct ba_record *pBA);
 bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS, u8 *Addr,
 	   u8 TID, enum tr_select TxRxSelect, bool bAddNewTs);
-- 
2.41.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-08-07  5:46 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-07  5:47 [PATCH 0/5] Staging: rtl8192e: Function name cleanup series 2 Tree Davies
2023-08-07  5:47 ` [PATCH 1/5] Staging: rtl8192e: Rename function TsInitAddBA Tree Davies
2023-08-07  5:47 ` [PATCH 2/5] Staging: rtl8192e: Rename function TsInitDelBA Tree Davies
2023-08-07  5:47 ` [PATCH 3/5] Staging: rtl8192e: Rename function BaSetupTimeOut Tree Davies
2023-08-07  5:47 ` [PATCH 4/5] Staging: rtl8192e: Rename function TxBaInactTimeout Tree Davies
2023-08-07  5:47 ` [PATCH 5/5] Staging: rtl8192e: Rename function RxBaInactTimeout Tree Davies

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®