* [PATCH 0/7] staging: rtl8723bs: remove dead functions
@ 2022-08-19 8:52 Nam Cao
2022-08-19 8:52 ` [PATCH 1/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
` (7 more replies)
0 siblings, 8 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
Remove some functions in this driver because they are not used anywhere.
Nam Cao (7):
staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
staging: rtl8723bs: remove function rtw_get_ch_setting_union
staging: rtl8723bs: remove function rtw_odm_ability_set
staging: rtl8723bs: remove function GetFractionValueFromString
staging: rtl8723bs: remove function IsCommentString
staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg
staging: rtl8723bs: remove function rtw_odm_dbg_comp_set
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 21 -------
drivers/staging/rtl8723bs/core/rtw_odm.c | 44 ---------------
drivers/staging/rtl8723bs/hal/hal_com.c | 55 -------------------
drivers/staging/rtl8723bs/include/hal_com.h | 5 --
.../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
drivers/staging/rtl8723bs/include/rtw_odm.h | 4 --
6 files changed, 130 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 1/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
@ 2022-08-19 8:52 ` Nam Cao
2022-08-19 13:07 ` kernel test robot
2022-08-19 8:52 ` [PATCH 2/7] staging: rtl8723bs: remove function rtw_get_ch_setting_union Nam Cao
` (6 subsequent siblings)
7 siblings, 1 reply; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 16 ----------------
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 17 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index f6b73a2a0270..abb111752169 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -90,22 +90,6 @@ static const char * const odm_dbg_level_str[] = {
#define RTW_ODM_DBG_LEVEL_NUM 6
-void rtw_odm_dbg_comp_msg(struct adapter *adapter)
-{
- u64 dbg_comp;
- int i;
-
- rtw_hal_get_def_var(adapter, HW_DEF_ODM_DBG_FLAG, &dbg_comp);
- netdev_dbg(adapter->pnetdev, "odm.DebugComponents = 0x%016llx\n",
- dbg_comp);
- for (i = 0; i < RTW_ODM_COMP_MAX; i++) {
- if (odm_comp_str[i])
- netdev_dbg(adapter->pnetdev, "%cBIT%-2d %s\n",
- (BIT0 << i) & dbg_comp ? '+' : ' ', i,
- odm_comp_str[i]);
- }
-}
-
inline void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps)
{
rtw_hal_set_def_var(adapter, HW_DEF_ODM_DBG_FLAG, &comps);
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index 94fc68a5c424..27e8240284b4 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -13,7 +13,6 @@
* This file provides utilities/wrappers for rtw driver to use ODM
*/
-void rtw_odm_dbg_comp_msg(struct adapter *adapter);
void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps);
void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter);
void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 2/7] staging: rtl8723bs: remove function rtw_get_ch_setting_union
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
2022-08-19 8:52 ` [PATCH 1/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
@ 2022-08-19 8:52 ` Nam Cao
2022-08-19 8:52 ` [PATCH 3/7] staging: rtl8723bs: remove function rtw_odm_ability_set Nam Cao
` (5 subsequent siblings)
7 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 21 -------------------
.../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
2 files changed, 22 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
index f878b04076d8..8e74b4f47b94 100644
--- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
@@ -5945,27 +5945,6 @@ int rtw_chk_start_clnt_join(struct adapter *padapter, u8 *ch, u8 *bw, u8 *offset
return connect_allow ? _SUCCESS : _FAIL;
}
-/* Find union about ch, bw, ch_offset of all linked/linking interfaces */
-int rtw_get_ch_setting_union(struct adapter *adapter, u8 *ch, u8 *bw, u8 *offset)
-{
- struct dvobj_priv *dvobj = adapter_to_dvobj(adapter);
- struct adapter *iface;
-
- if (ch)
- *ch = 0;
- if (bw)
- *bw = CHANNEL_WIDTH_20;
- if (offset)
- *offset = HAL_PRIME_CHNL_OFFSET_DONT_CARE;
-
- iface = dvobj->padapters;
-
- if (!check_fwstate(&iface->mlmepriv, _FW_LINKED|_FW_UNDER_LINKING))
- return 0;
-
- return 1;
-}
-
u8 set_ch_hdl(struct adapter *padapter, u8 *pbuf)
{
struct set_ch_parm *set_ch_parm;
diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
index 89b389d4c44b..65e138a5238f 100644
--- a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
+++ b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
@@ -662,7 +662,6 @@ extern void adaptive_early_32k(struct mlme_ext_priv *pmlmeext, u8 *pframe, uint
extern u8 traffic_status_watchdog(struct adapter *padapter, u8 from_timer);
int rtw_chk_start_clnt_join(struct adapter *padapter, u8 *ch, u8 *bw, u8 *offset);
-int rtw_get_ch_setting_union(struct adapter *adapter, u8 *ch, u8 *bw, u8 *offset);
struct cmd_hdl {
uint parmsize;
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 3/7] staging: rtl8723bs: remove function rtw_odm_ability_set
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
2022-08-19 8:52 ` [PATCH 1/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
2022-08-19 8:52 ` [PATCH 2/7] staging: rtl8723bs: remove function rtw_get_ch_setting_union Nam Cao
@ 2022-08-19 8:52 ` Nam Cao
2022-08-19 8:52 ` [PATCH 4/7] staging: rtl8723bs: remove function GetFractionValueFromString Nam Cao
` (4 subsequent siblings)
7 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 5 -----
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index abb111752169..ec8406c18b03 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -129,11 +129,6 @@ void rtw_odm_ability_msg(void *sel, struct adapter *adapter)
}
}
-inline void rtw_odm_ability_set(struct adapter *adapter, u32 ability)
-{
- rtw_hal_set_hwreg(adapter, HW_VAR_DM_FLAG, (u8 *)&ability);
-}
-
void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter)
{
struct hal_com_data *pHalData = GET_HAL_DATA(adapter);
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index 27e8240284b4..ea9c4de0f284 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -18,7 +18,6 @@ void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter);
void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
void rtw_odm_ability_msg(void *sel, struct adapter *adapter);
-void rtw_odm_ability_set(struct adapter *adapter, u32 ability);
void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter);
void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini, s8 TH_EDCCA_HL_diff,
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 4/7] staging: rtl8723bs: remove function GetFractionValueFromString
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
` (2 preceding siblings ...)
2022-08-19 8:52 ` [PATCH 3/7] staging: rtl8723bs: remove function rtw_odm_ability_set Nam Cao
@ 2022-08-19 8:52 ` Nam Cao
2022-08-19 8:52 ` [PATCH 5/7] staging: rtl8723bs: remove function IsCommentString Nam Cao
` (3 subsequent siblings)
7 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/hal/hal_com.c | 43 ---------------------
drivers/staging/rtl8723bs/include/hal_com.h | 3 --
2 files changed, 46 deletions(-)
diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
index 909b37bcc897..f82cbe5a77ff 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com.c
@@ -939,49 +939,6 @@ bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove)
return true;
}
-bool GetFractionValueFromString(
- char *szStr, u8 *pInteger, u8 *pFraction, u32 *pu4bMove
-)
-{
- char *szScan = szStr;
-
- /* Initialize output. */
- *pu4bMove = 0;
- *pInteger = 0;
- *pFraction = 0;
-
- /* Skip leading space. */
- while (*szScan != '\0' && (*szScan == ' ' || *szScan == '\t')) {
- ++szScan;
- ++(*pu4bMove);
- }
-
- /* Parse each digit. */
- do {
- (*pInteger) *= 10;
- *pInteger += (*szScan - '0');
-
- ++szScan;
- ++(*pu4bMove);
-
- if (*szScan == '.') {
- ++szScan;
- ++(*pu4bMove);
-
- if (*szScan < '0' || *szScan > '9')
- return false;
- else {
- *pFraction = *szScan - '0';
- ++szScan;
- ++(*pu4bMove);
- return true;
- }
- }
- } while (*szScan >= '0' && *szScan <= '9');
-
- return true;
-}
-
/* */
/* Description: */
/* Return true if szStr is comment out with leading "//". */
diff --git a/drivers/staging/rtl8723bs/include/hal_com.h b/drivers/staging/rtl8723bs/include/hal_com.h
index 7be0ea20bca4..b49b0a0355c6 100644
--- a/drivers/staging/rtl8723bs/include/hal_com.h
+++ b/drivers/staging/rtl8723bs/include/hal_com.h
@@ -153,9 +153,6 @@ u32 MapCharToHexDigit(char chTmp);
bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove);
-bool GetFractionValueFromString(char *szStr, u8 *pInteger, u8 *pFraction,
- u32 *pu4bMove);
-
bool IsCommentString(char *szStr);
bool ParseQualifiedString(char *In, u32 *Start, char *Out, char LeftQualifier,
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 5/7] staging: rtl8723bs: remove function IsCommentString
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
` (3 preceding siblings ...)
2022-08-19 8:52 ` [PATCH 4/7] staging: rtl8723bs: remove function GetFractionValueFromString Nam Cao
@ 2022-08-19 8:52 ` Nam Cao
2022-08-19 8:52 ` [PATCH 6/7] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg Nam Cao
` (2 subsequent siblings)
7 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/hal/hal_com.c | 12 ------------
drivers/staging/rtl8723bs/include/hal_com.h | 2 --
2 files changed, 14 deletions(-)
diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
index f82cbe5a77ff..1c744d0b4742 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com.c
@@ -939,18 +939,6 @@ bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove)
return true;
}
-/* */
-/* Description: */
-/* Return true if szStr is comment out with leading "//". */
-/* */
-bool IsCommentString(char *szStr)
-{
- if (*szStr == '/' && *(szStr+1) == '/')
- return true;
- else
- return false;
-}
-
bool GetU1ByteIntegerFromStringInDecimal(char *Str, u8 *pInt)
{
u16 i = 0;
diff --git a/drivers/staging/rtl8723bs/include/hal_com.h b/drivers/staging/rtl8723bs/include/hal_com.h
index b49b0a0355c6..406fccbcd4b6 100644
--- a/drivers/staging/rtl8723bs/include/hal_com.h
+++ b/drivers/staging/rtl8723bs/include/hal_com.h
@@ -153,8 +153,6 @@ u32 MapCharToHexDigit(char chTmp);
bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove);
-bool IsCommentString(char *szStr);
-
bool ParseQualifiedString(char *In, u32 *Start, char *Out, char LeftQualifier,
char RightQualifier);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 6/7] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
` (4 preceding siblings ...)
2022-08-19 8:52 ` [PATCH 5/7] staging: rtl8723bs: remove function IsCommentString Nam Cao
@ 2022-08-19 8:52 ` Nam Cao
2022-08-19 8:52 ` [PATCH 7/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set Nam Cao
2022-08-19 9:06 ` [PATCH 0/7] staging: rtl8723bs: remove dead functions Hans de Goede
7 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 18 ------------------
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 19 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index ec8406c18b03..2cba7825c9b3 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -129,24 +129,6 @@ void rtw_odm_ability_msg(void *sel, struct adapter *adapter)
}
}
-void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter)
-{
- struct hal_com_data *pHalData = GET_HAL_DATA(adapter);
- struct dm_odm_t *odm = &pHalData->odmpriv;
-
- netdev_dbg(adapter->pnetdev, "%10s %16s %8s %10s %11s %14s\n",
- "TH_L2H_ini", "TH_EDCCA_HL_diff", "IGI_Base", "ForceEDCCA",
- "AdapEn_RSSI", "IGI_LowerBound");
- netdev_dbg(adapter->pnetdev,
- "0x%-8x %-16d 0x%-6x %-10d %-11u %-14u\n",
- (u8)odm->TH_L2H_ini,
- odm->TH_EDCCA_HL_diff,
- odm->IGI_Base,
- odm->ForceEDCCA,
- odm->AdapEn_RSSI,
- odm->IGI_LowerBound);
-}
-
void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini,
s8 TH_EDCCA_HL_diff, s8 IGI_Base,
bool ForceEDCCA, u8 AdapEn_RSSI,
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index ea9c4de0f284..be0d47413fd2 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -19,7 +19,6 @@ void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
void rtw_odm_ability_msg(void *sel, struct adapter *adapter);
-void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter);
void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini, s8 TH_EDCCA_HL_diff,
s8 IGI_Base, bool ForceEDCCA, u8 AdapEn_RSSI, u8 IGI_LowerBound);
void rtw_odm_get_perpkt_rssi(void *sel, struct adapter *adapter);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 7/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
` (5 preceding siblings ...)
2022-08-19 8:52 ` [PATCH 6/7] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg Nam Cao
@ 2022-08-19 8:52 ` Nam Cao
2022-08-19 9:06 ` [PATCH 0/7] staging: rtl8723bs: remove dead functions Hans de Goede
7 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 8:52 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 5 -----
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index 2cba7825c9b3..58b6299ed9d7 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -90,11 +90,6 @@ static const char * const odm_dbg_level_str[] = {
#define RTW_ODM_DBG_LEVEL_NUM 6
-inline void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps)
-{
- rtw_hal_set_def_var(adapter, HW_DEF_ODM_DBG_FLAG, &comps);
-}
-
void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter)
{
u32 dbg_level;
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index be0d47413fd2..6a431c121285 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -13,7 +13,6 @@
* This file provides utilities/wrappers for rtw driver to use ODM
*/
-void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps);
void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter);
void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH 0/7] staging: rtl8723bs: remove dead functions
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
` (6 preceding siblings ...)
2022-08-19 8:52 ` [PATCH 7/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set Nam Cao
@ 2022-08-19 9:06 ` Hans de Goede
7 siblings, 0 replies; 22+ messages in thread
From: Hans de Goede @ 2022-08-19 9:06 UTC (permalink / raw)
To: Nam Cao, gregkh; +Cc: linux-kernel, linux-staging, Larry.Finger
Hi,
On 8/19/22 10:52, Nam Cao wrote:
> Remove some functions in this driver because they are not used anywhere.
>
> Nam Cao (7):
> staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
> staging: rtl8723bs: remove function rtw_get_ch_setting_union
> staging: rtl8723bs: remove function rtw_odm_ability_set
> staging: rtl8723bs: remove function GetFractionValueFromString
> staging: rtl8723bs: remove function IsCommentString
> staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg
> staging: rtl8723bs: remove function rtw_odm_dbg_comp_set
>
> drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 21 -------
> drivers/staging/rtl8723bs/core/rtw_odm.c | 44 ---------------
> drivers/staging/rtl8723bs/hal/hal_com.c | 55 -------------------
> drivers/staging/rtl8723bs/include/hal_com.h | 5 --
> .../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
> drivers/staging/rtl8723bs/include/rtw_odm.h | 4 --
> 6 files changed, 130 deletions(-)
Thanks, the entire series looks good to me:
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
for the series.
Regards,
Hans
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
2022-08-19 8:52 ` [PATCH 1/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
@ 2022-08-19 13:07 ` kernel test robot
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
0 siblings, 1 reply; 22+ messages in thread
From: kernel test robot @ 2022-08-19 13:07 UTC (permalink / raw)
To: Nam Cao, gregkh
Cc: kbuild-all, linux-kernel, linux-staging, namcaov, hdegoede, Larry.Finger
Hi Nam,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on staging/staging-testing]
url: https://github.com/intel-lab-lkp/linux/commits/Nam-Cao/staging-rtl8723bs-remove-dead-functions/20220819-165905
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 8379cf83fe6d57a12952de6dcaf7a7fbd7b364fc
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20220819/202208192018.BfgiZyOY-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-5) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/intel-lab-lkp/linux/commit/3b86aa03519d904d916cb62a1113c0c5549496cb
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Nam-Cao/staging-rtl8723bs-remove-dead-functions/20220819-165905
git checkout 3b86aa03519d904d916cb62a1113c0c5549496cb
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/staging/rtl8723bs/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/staging/rtl8723bs/core/rtw_odm.c:13:27: warning: 'odm_comp_str' defined but not used [-Wunused-const-variable=]
13 | static const char * const odm_comp_str[] = {
| ^~~~~~~~~~~~
vim +/odm_comp_str +13 drivers/staging/rtl8723bs/core/rtw_odm.c
554c0a3abf216c Hans de Goede 2017-03-29 12
05031914cb8f5b Tom Gardi 2017-08-12 @13 static const char * const odm_comp_str[] = {
554c0a3abf216c Hans de Goede 2017-03-29 14 /* BIT0 */"ODM_COMP_DIG",
554c0a3abf216c Hans de Goede 2017-03-29 15 /* BIT1 */"ODM_COMP_RA_MASK",
554c0a3abf216c Hans de Goede 2017-03-29 16 /* BIT2 */"ODM_COMP_DYNAMIC_TXPWR",
554c0a3abf216c Hans de Goede 2017-03-29 17 /* BIT3 */"ODM_COMP_FA_CNT",
554c0a3abf216c Hans de Goede 2017-03-29 18 /* BIT4 */"ODM_COMP_RSSI_MONITOR",
554c0a3abf216c Hans de Goede 2017-03-29 19 /* BIT5 */"ODM_COMP_CCK_PD",
554c0a3abf216c Hans de Goede 2017-03-29 20 /* BIT6 */"ODM_COMP_ANT_DIV",
554c0a3abf216c Hans de Goede 2017-03-29 21 /* BIT7 */"ODM_COMP_PWR_SAVE",
554c0a3abf216c Hans de Goede 2017-03-29 22 /* BIT8 */"ODM_COMP_PWR_TRAIN",
554c0a3abf216c Hans de Goede 2017-03-29 23 /* BIT9 */"ODM_COMP_RATE_ADAPTIVE",
554c0a3abf216c Hans de Goede 2017-03-29 24 /* BIT10 */"ODM_COMP_PATH_DIV",
554c0a3abf216c Hans de Goede 2017-03-29 25 /* BIT11 */"ODM_COMP_PSD",
554c0a3abf216c Hans de Goede 2017-03-29 26 /* BIT12 */"ODM_COMP_DYNAMIC_PRICCA",
554c0a3abf216c Hans de Goede 2017-03-29 27 /* BIT13 */"ODM_COMP_RXHP",
554c0a3abf216c Hans de Goede 2017-03-29 28 /* BIT14 */"ODM_COMP_MP",
554c0a3abf216c Hans de Goede 2017-03-29 29 /* BIT15 */"ODM_COMP_DYNAMIC_ATC",
554c0a3abf216c Hans de Goede 2017-03-29 30 /* BIT16 */"ODM_COMP_EDCA_TURBO",
554c0a3abf216c Hans de Goede 2017-03-29 31 /* BIT17 */"ODM_COMP_EARLY_MODE",
554c0a3abf216c Hans de Goede 2017-03-29 32 /* BIT18 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 33 /* BIT19 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 34 /* BIT20 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 35 /* BIT21 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 36 /* BIT22 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 37 /* BIT23 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 38 /* BIT24 */"ODM_COMP_TX_PWR_TRACK",
554c0a3abf216c Hans de Goede 2017-03-29 39 /* BIT25 */"ODM_COMP_RX_GAIN_TRACK",
554c0a3abf216c Hans de Goede 2017-03-29 40 /* BIT26 */"ODM_COMP_CALIBRATION",
554c0a3abf216c Hans de Goede 2017-03-29 41 /* BIT27 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 42 /* BIT28 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 43 /* BIT29 */NULL,
554c0a3abf216c Hans de Goede 2017-03-29 44 /* BIT30 */"ODM_COMP_COMMON",
554c0a3abf216c Hans de Goede 2017-03-29 45 /* BIT31 */"ODM_COMP_INIT",
554c0a3abf216c Hans de Goede 2017-03-29 46 };
554c0a3abf216c Hans de Goede 2017-03-29 47
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v2 0/8] remove some dead code
2022-08-19 13:07 ` kernel test robot
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
` (8 more replies)
0 siblings, 9 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
Remove some unused functions and variables.
Change in v2: add a new patch to fix kernel test robot's warning
about unused variable.
Nam Cao (8):
staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
staging: rtl8723bs: remove function rtw_get_ch_setting_union
staging: rtl8723bs: remove function rtw_odm_ability_set
staging: rtl8723bs: remove function GetFractionValueFromString
staging: rtl8723bs: remove function IsCommentString
staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg
staging: rtl8723bs: remove function rtw_odm_dbg_comp_set
staging: rtl8723bs: remove static const variable odm_comp_str
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 21 -----
drivers/staging/rtl8723bs/core/rtw_odm.c | 79 -------------------
drivers/staging/rtl8723bs/hal/hal_com.c | 55 -------------
drivers/staging/rtl8723bs/include/hal_com.h | 5 --
.../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
drivers/staging/rtl8723bs/include/rtw_odm.h | 4 -
6 files changed, 165 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-22 7:33 ` Dan Carpenter
2022-08-19 13:49 ` [PATCH v2 2/8] staging: rtl8723bs: remove function rtw_get_ch_setting_union Nam Cao
` (7 subsequent siblings)
8 siblings, 1 reply; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 16 ----------------
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 17 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index f6b73a2a0270..abb111752169 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -90,22 +90,6 @@ static const char * const odm_dbg_level_str[] = {
#define RTW_ODM_DBG_LEVEL_NUM 6
-void rtw_odm_dbg_comp_msg(struct adapter *adapter)
-{
- u64 dbg_comp;
- int i;
-
- rtw_hal_get_def_var(adapter, HW_DEF_ODM_DBG_FLAG, &dbg_comp);
- netdev_dbg(adapter->pnetdev, "odm.DebugComponents = 0x%016llx\n",
- dbg_comp);
- for (i = 0; i < RTW_ODM_COMP_MAX; i++) {
- if (odm_comp_str[i])
- netdev_dbg(adapter->pnetdev, "%cBIT%-2d %s\n",
- (BIT0 << i) & dbg_comp ? '+' : ' ', i,
- odm_comp_str[i]);
- }
-}
-
inline void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps)
{
rtw_hal_set_def_var(adapter, HW_DEF_ODM_DBG_FLAG, &comps);
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index 94fc68a5c424..27e8240284b4 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -13,7 +13,6 @@
* This file provides utilities/wrappers for rtw driver to use ODM
*/
-void rtw_odm_dbg_comp_msg(struct adapter *adapter);
void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps);
void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter);
void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2 2/8] staging: rtl8723bs: remove function rtw_get_ch_setting_union
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
2022-08-19 13:49 ` [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 3/8] staging: rtl8723bs: remove function rtw_odm_ability_set Nam Cao
` (6 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 21 -------------------
.../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
2 files changed, 22 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
index f878b04076d8..8e74b4f47b94 100644
--- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
@@ -5945,27 +5945,6 @@ int rtw_chk_start_clnt_join(struct adapter *padapter, u8 *ch, u8 *bw, u8 *offset
return connect_allow ? _SUCCESS : _FAIL;
}
-/* Find union about ch, bw, ch_offset of all linked/linking interfaces */
-int rtw_get_ch_setting_union(struct adapter *adapter, u8 *ch, u8 *bw, u8 *offset)
-{
- struct dvobj_priv *dvobj = adapter_to_dvobj(adapter);
- struct adapter *iface;
-
- if (ch)
- *ch = 0;
- if (bw)
- *bw = CHANNEL_WIDTH_20;
- if (offset)
- *offset = HAL_PRIME_CHNL_OFFSET_DONT_CARE;
-
- iface = dvobj->padapters;
-
- if (!check_fwstate(&iface->mlmepriv, _FW_LINKED|_FW_UNDER_LINKING))
- return 0;
-
- return 1;
-}
-
u8 set_ch_hdl(struct adapter *padapter, u8 *pbuf)
{
struct set_ch_parm *set_ch_parm;
diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
index 89b389d4c44b..65e138a5238f 100644
--- a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
+++ b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
@@ -662,7 +662,6 @@ extern void adaptive_early_32k(struct mlme_ext_priv *pmlmeext, u8 *pframe, uint
extern u8 traffic_status_watchdog(struct adapter *padapter, u8 from_timer);
int rtw_chk_start_clnt_join(struct adapter *padapter, u8 *ch, u8 *bw, u8 *offset);
-int rtw_get_ch_setting_union(struct adapter *adapter, u8 *ch, u8 *bw, u8 *offset);
struct cmd_hdl {
uint parmsize;
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2 3/8] staging: rtl8723bs: remove function rtw_odm_ability_set
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
2022-08-19 13:49 ` [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
2022-08-19 13:49 ` [PATCH v2 2/8] staging: rtl8723bs: remove function rtw_get_ch_setting_union Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 4/8] staging: rtl8723bs: remove function GetFractionValueFromString Nam Cao
` (5 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 5 -----
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index abb111752169..ec8406c18b03 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -129,11 +129,6 @@ void rtw_odm_ability_msg(void *sel, struct adapter *adapter)
}
}
-inline void rtw_odm_ability_set(struct adapter *adapter, u32 ability)
-{
- rtw_hal_set_hwreg(adapter, HW_VAR_DM_FLAG, (u8 *)&ability);
-}
-
void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter)
{
struct hal_com_data *pHalData = GET_HAL_DATA(adapter);
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index 27e8240284b4..ea9c4de0f284 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -18,7 +18,6 @@ void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter);
void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
void rtw_odm_ability_msg(void *sel, struct adapter *adapter);
-void rtw_odm_ability_set(struct adapter *adapter, u32 ability);
void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter);
void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini, s8 TH_EDCCA_HL_diff,
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2 4/8] staging: rtl8723bs: remove function GetFractionValueFromString
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
` (2 preceding siblings ...)
2022-08-19 13:49 ` [PATCH v2 3/8] staging: rtl8723bs: remove function rtw_odm_ability_set Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 5/8] staging: rtl8723bs: remove function IsCommentString Nam Cao
` (4 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/hal/hal_com.c | 43 ---------------------
drivers/staging/rtl8723bs/include/hal_com.h | 3 --
2 files changed, 46 deletions(-)
diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
index 909b37bcc897..f82cbe5a77ff 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com.c
@@ -939,49 +939,6 @@ bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove)
return true;
}
-bool GetFractionValueFromString(
- char *szStr, u8 *pInteger, u8 *pFraction, u32 *pu4bMove
-)
-{
- char *szScan = szStr;
-
- /* Initialize output. */
- *pu4bMove = 0;
- *pInteger = 0;
- *pFraction = 0;
-
- /* Skip leading space. */
- while (*szScan != '\0' && (*szScan == ' ' || *szScan == '\t')) {
- ++szScan;
- ++(*pu4bMove);
- }
-
- /* Parse each digit. */
- do {
- (*pInteger) *= 10;
- *pInteger += (*szScan - '0');
-
- ++szScan;
- ++(*pu4bMove);
-
- if (*szScan == '.') {
- ++szScan;
- ++(*pu4bMove);
-
- if (*szScan < '0' || *szScan > '9')
- return false;
- else {
- *pFraction = *szScan - '0';
- ++szScan;
- ++(*pu4bMove);
- return true;
- }
- }
- } while (*szScan >= '0' && *szScan <= '9');
-
- return true;
-}
-
/* */
/* Description: */
/* Return true if szStr is comment out with leading "//". */
diff --git a/drivers/staging/rtl8723bs/include/hal_com.h b/drivers/staging/rtl8723bs/include/hal_com.h
index 7be0ea20bca4..b49b0a0355c6 100644
--- a/drivers/staging/rtl8723bs/include/hal_com.h
+++ b/drivers/staging/rtl8723bs/include/hal_com.h
@@ -153,9 +153,6 @@ u32 MapCharToHexDigit(char chTmp);
bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove);
-bool GetFractionValueFromString(char *szStr, u8 *pInteger, u8 *pFraction,
- u32 *pu4bMove);
-
bool IsCommentString(char *szStr);
bool ParseQualifiedString(char *In, u32 *Start, char *Out, char LeftQualifier,
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2 5/8] staging: rtl8723bs: remove function IsCommentString
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
` (3 preceding siblings ...)
2022-08-19 13:49 ` [PATCH v2 4/8] staging: rtl8723bs: remove function GetFractionValueFromString Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 6/8] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg Nam Cao
` (3 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/hal/hal_com.c | 12 ------------
drivers/staging/rtl8723bs/include/hal_com.h | 2 --
2 files changed, 14 deletions(-)
diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
index f82cbe5a77ff..1c744d0b4742 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com.c
@@ -939,18 +939,6 @@ bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove)
return true;
}
-/* */
-/* Description: */
-/* Return true if szStr is comment out with leading "//". */
-/* */
-bool IsCommentString(char *szStr)
-{
- if (*szStr == '/' && *(szStr+1) == '/')
- return true;
- else
- return false;
-}
-
bool GetU1ByteIntegerFromStringInDecimal(char *Str, u8 *pInt)
{
u16 i = 0;
diff --git a/drivers/staging/rtl8723bs/include/hal_com.h b/drivers/staging/rtl8723bs/include/hal_com.h
index b49b0a0355c6..406fccbcd4b6 100644
--- a/drivers/staging/rtl8723bs/include/hal_com.h
+++ b/drivers/staging/rtl8723bs/include/hal_com.h
@@ -153,8 +153,6 @@ u32 MapCharToHexDigit(char chTmp);
bool GetHexValueFromString(char *szStr, u32 *pu4bVal, u32 *pu4bMove);
-bool IsCommentString(char *szStr);
-
bool ParseQualifiedString(char *In, u32 *Start, char *Out, char LeftQualifier,
char RightQualifier);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2 6/8] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
` (4 preceding siblings ...)
2022-08-19 13:49 ` [PATCH v2 5/8] staging: rtl8723bs: remove function IsCommentString Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 7/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set Nam Cao
` (2 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 18 ------------------
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 19 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index ec8406c18b03..2cba7825c9b3 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -129,24 +129,6 @@ void rtw_odm_ability_msg(void *sel, struct adapter *adapter)
}
}
-void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter)
-{
- struct hal_com_data *pHalData = GET_HAL_DATA(adapter);
- struct dm_odm_t *odm = &pHalData->odmpriv;
-
- netdev_dbg(adapter->pnetdev, "%10s %16s %8s %10s %11s %14s\n",
- "TH_L2H_ini", "TH_EDCCA_HL_diff", "IGI_Base", "ForceEDCCA",
- "AdapEn_RSSI", "IGI_LowerBound");
- netdev_dbg(adapter->pnetdev,
- "0x%-8x %-16d 0x%-6x %-10d %-11u %-14u\n",
- (u8)odm->TH_L2H_ini,
- odm->TH_EDCCA_HL_diff,
- odm->IGI_Base,
- odm->ForceEDCCA,
- odm->AdapEn_RSSI,
- odm->IGI_LowerBound);
-}
-
void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini,
s8 TH_EDCCA_HL_diff, s8 IGI_Base,
bool ForceEDCCA, u8 AdapEn_RSSI,
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index ea9c4de0f284..be0d47413fd2 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -19,7 +19,6 @@ void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
void rtw_odm_ability_msg(void *sel, struct adapter *adapter);
-void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter);
void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini, s8 TH_EDCCA_HL_diff,
s8 IGI_Base, bool ForceEDCCA, u8 AdapEn_RSSI, u8 IGI_LowerBound);
void rtw_odm_get_perpkt_rssi(void *sel, struct adapter *adapter);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2 7/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
` (5 preceding siblings ...)
2022-08-19 13:49 ` [PATCH v2 6/8] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 8/8] staging: rtl8723bs: remove static const variable odm_comp_str Nam Cao
2022-08-19 14:25 ` [PATCH v2 0/8] remove some dead code Hans de Goede
8 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because this function is not used.
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 5 -----
drivers/staging/rtl8723bs/include/rtw_odm.h | 1 -
2 files changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index 2cba7825c9b3..58b6299ed9d7 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -90,11 +90,6 @@ static const char * const odm_dbg_level_str[] = {
#define RTW_ODM_DBG_LEVEL_NUM 6
-inline void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps)
-{
- rtw_hal_set_def_var(adapter, HW_DEF_ODM_DBG_FLAG, &comps);
-}
-
void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter)
{
u32 dbg_level;
diff --git a/drivers/staging/rtl8723bs/include/rtw_odm.h b/drivers/staging/rtl8723bs/include/rtw_odm.h
index be0d47413fd2..6a431c121285 100644
--- a/drivers/staging/rtl8723bs/include/rtw_odm.h
+++ b/drivers/staging/rtl8723bs/include/rtw_odm.h
@@ -13,7 +13,6 @@
* This file provides utilities/wrappers for rtw driver to use ODM
*/
-void rtw_odm_dbg_comp_set(struct adapter *adapter, u64 comps);
void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter);
void rtw_odm_dbg_level_set(struct adapter *adapter, u32 level);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2 8/8] staging: rtl8723bs: remove static const variable odm_comp_str
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
` (6 preceding siblings ...)
2022-08-19 13:49 ` [PATCH v2 7/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set Nam Cao
@ 2022-08-19 13:49 ` Nam Cao
2022-08-19 14:25 ` [PATCH v2 0/8] remove some dead code Hans de Goede
8 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-19 13:49 UTC (permalink / raw)
To: lkp
Cc: Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging, namcaov
because it is not used.
Reported-by: kernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/linux-staging/202208192018.BfgiZyOY-lkp@intel.com/
Signed-off-by: Nam Cao <namcaov@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_odm.c | 35 ------------------------
1 file changed, 35 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_odm.c b/drivers/staging/rtl8723bs/core/rtw_odm.c
index 58b6299ed9d7..47fd2ee9bb9f 100644
--- a/drivers/staging/rtl8723bs/core/rtw_odm.c
+++ b/drivers/staging/rtl8723bs/core/rtw_odm.c
@@ -10,41 +10,6 @@
#include <rtw_odm.h>
#include <hal_data.h>
-static const char * const odm_comp_str[] = {
- /* BIT0 */"ODM_COMP_DIG",
- /* BIT1 */"ODM_COMP_RA_MASK",
- /* BIT2 */"ODM_COMP_DYNAMIC_TXPWR",
- /* BIT3 */"ODM_COMP_FA_CNT",
- /* BIT4 */"ODM_COMP_RSSI_MONITOR",
- /* BIT5 */"ODM_COMP_CCK_PD",
- /* BIT6 */"ODM_COMP_ANT_DIV",
- /* BIT7 */"ODM_COMP_PWR_SAVE",
- /* BIT8 */"ODM_COMP_PWR_TRAIN",
- /* BIT9 */"ODM_COMP_RATE_ADAPTIVE",
- /* BIT10 */"ODM_COMP_PATH_DIV",
- /* BIT11 */"ODM_COMP_PSD",
- /* BIT12 */"ODM_COMP_DYNAMIC_PRICCA",
- /* BIT13 */"ODM_COMP_RXHP",
- /* BIT14 */"ODM_COMP_MP",
- /* BIT15 */"ODM_COMP_DYNAMIC_ATC",
- /* BIT16 */"ODM_COMP_EDCA_TURBO",
- /* BIT17 */"ODM_COMP_EARLY_MODE",
- /* BIT18 */NULL,
- /* BIT19 */NULL,
- /* BIT20 */NULL,
- /* BIT21 */NULL,
- /* BIT22 */NULL,
- /* BIT23 */NULL,
- /* BIT24 */"ODM_COMP_TX_PWR_TRACK",
- /* BIT25 */"ODM_COMP_RX_GAIN_TRACK",
- /* BIT26 */"ODM_COMP_CALIBRATION",
- /* BIT27 */NULL,
- /* BIT28 */NULL,
- /* BIT29 */NULL,
- /* BIT30 */"ODM_COMP_COMMON",
- /* BIT31 */"ODM_COMP_INIT",
-};
-
#define RTW_ODM_COMP_MAX 32
static const char * const odm_ability_str[] = {
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v2 0/8] remove some dead code
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
` (7 preceding siblings ...)
2022-08-19 13:49 ` [PATCH v2 8/8] staging: rtl8723bs: remove static const variable odm_comp_str Nam Cao
@ 2022-08-19 14:25 ` Hans de Goede
8 siblings, 0 replies; 22+ messages in thread
From: Hans de Goede @ 2022-08-19 14:25 UTC (permalink / raw)
To: Nam Cao, lkp
Cc: Larry.Finger, gregkh, kbuild-all, linux-kernel, linux-staging
Hi,
On 8/19/22 15:49, Nam Cao wrote:
> Remove some unused functions and variables.
>
> Change in v2: add a new patch to fix kernel test robot's warning
> about unused variable.
>
> Nam Cao (8):
> staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
> staging: rtl8723bs: remove function rtw_get_ch_setting_union
> staging: rtl8723bs: remove function rtw_odm_ability_set
> staging: rtl8723bs: remove function GetFractionValueFromString
> staging: rtl8723bs: remove function IsCommentString
> staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg
> staging: rtl8723bs: remove function rtw_odm_dbg_comp_set
> staging: rtl8723bs: remove static const variable odm_comp_str
>
> drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 21 -----
> drivers/staging/rtl8723bs/core/rtw_odm.c | 79 -------------------
> drivers/staging/rtl8723bs/hal/hal_com.c | 55 -------------
> drivers/staging/rtl8723bs/include/hal_com.h | 5 --
> .../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
> drivers/staging/rtl8723bs/include/rtw_odm.h | 4 -
> 6 files changed, 165 deletions(-)
Thanks, the entire series looks good to me:
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
for the series.
Regards,
Hans
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
2022-08-19 13:49 ` [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
@ 2022-08-22 7:33 ` Dan Carpenter
2022-08-22 7:47 ` Nam Cao
0 siblings, 1 reply; 22+ messages in thread
From: Dan Carpenter @ 2022-08-22 7:33 UTC (permalink / raw)
To: Nam Cao
Cc: lkp, Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging
On Fri, Aug 19, 2022 at 03:49:35PM +0200, Nam Cao wrote:
> because this function is not used.
>
> Signed-off-by: Nam Cao <namcaov@gmail.com>
This patch is fine, but in the future can you please treat the subject
as separate from the commit message. "because it is not used" is a
sucky commit message. What is not used? What are you doing to it? It
doesn't start with a capital letter.
Here is how your email looks like on lore.
https://lore.kernel.org/all/7ff2d658863db4fd5eecc1a53f682510c2765c3f.1660916523.git.namcaov@gmail.com/
Try to find the subject in the middle of the block of text. People are
often only going to read the body of the email.
[PATCH v2 1/8] staging: rtl8723bs: remove rtw_odm_dbg_comp_msg()
The rtw_odm_dbg_comp_msg() function is not used. Delete it.
Signed-off-by:
regards,
dan carpenter
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg
2022-08-22 7:33 ` Dan Carpenter
@ 2022-08-22 7:47 ` Nam Cao
0 siblings, 0 replies; 22+ messages in thread
From: Nam Cao @ 2022-08-22 7:47 UTC (permalink / raw)
To: Dan Carpenter
Cc: lkp, Larry.Finger, gregkh, hdegoede, kbuild-all, linux-kernel,
linux-staging
Thank you so much for letting me know. I will try to make it more readable in
the future.
Best regards,
Nam
On Mon, Aug 22, 2022 at 9:34 AM Dan Carpenter <dan.carpenter@oracle.com> wrote:
>
> On Fri, Aug 19, 2022 at 03:49:35PM +0200, Nam Cao wrote:
> > because this function is not used.
> >
> > Signed-off-by: Nam Cao <namcaov@gmail.com>
>
> This patch is fine, but in the future can you please treat the subject
> as separate from the commit message. "because it is not used" is a
> sucky commit message. What is not used? What are you doing to it? It
> doesn't start with a capital letter.
>
> Here is how your email looks like on lore.
>
> https://lore.kernel.org/all/7ff2d658863db4fd5eecc1a53f682510c2765c3f.1660916523.git.namcaov@gmail.com/
>
> Try to find the subject in the middle of the block of text. People are
> often only going to read the body of the email.
>
> [PATCH v2 1/8] staging: rtl8723bs: remove rtw_odm_dbg_comp_msg()
>
> The rtw_odm_dbg_comp_msg() function is not used. Delete it.
>
> Signed-off-by:
>
> regards,
> dan carpenter
>
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2022-08-22 7:48 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-19 8:52 [PATCH 0/7] staging: rtl8723bs: remove dead functions Nam Cao
2022-08-19 8:52 ` [PATCH 1/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
2022-08-19 13:07 ` kernel test robot
2022-08-19 13:49 ` [PATCH v2 0/8] remove some dead code Nam Cao
2022-08-19 13:49 ` [PATCH v2 1/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_msg Nam Cao
2022-08-22 7:33 ` Dan Carpenter
2022-08-22 7:47 ` Nam Cao
2022-08-19 13:49 ` [PATCH v2 2/8] staging: rtl8723bs: remove function rtw_get_ch_setting_union Nam Cao
2022-08-19 13:49 ` [PATCH v2 3/8] staging: rtl8723bs: remove function rtw_odm_ability_set Nam Cao
2022-08-19 13:49 ` [PATCH v2 4/8] staging: rtl8723bs: remove function GetFractionValueFromString Nam Cao
2022-08-19 13:49 ` [PATCH v2 5/8] staging: rtl8723bs: remove function IsCommentString Nam Cao
2022-08-19 13:49 ` [PATCH v2 6/8] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg Nam Cao
2022-08-19 13:49 ` [PATCH v2 7/8] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set Nam Cao
2022-08-19 13:49 ` [PATCH v2 8/8] staging: rtl8723bs: remove static const variable odm_comp_str Nam Cao
2022-08-19 14:25 ` [PATCH v2 0/8] remove some dead code Hans de Goede
2022-08-19 8:52 ` [PATCH 2/7] staging: rtl8723bs: remove function rtw_get_ch_setting_union Nam Cao
2022-08-19 8:52 ` [PATCH 3/7] staging: rtl8723bs: remove function rtw_odm_ability_set Nam Cao
2022-08-19 8:52 ` [PATCH 4/7] staging: rtl8723bs: remove function GetFractionValueFromString Nam Cao
2022-08-19 8:52 ` [PATCH 5/7] staging: rtl8723bs: remove function IsCommentString Nam Cao
2022-08-19 8:52 ` [PATCH 6/7] staging: rtl8723bs: remove function rtw_odm_adaptivity_parm_msg Nam Cao
2022-08-19 8:52 ` [PATCH 7/7] staging: rtl8723bs: remove function rtw_odm_dbg_comp_set Nam Cao
2022-08-19 9:06 ` [PATCH 0/7] staging: rtl8723bs: remove dead functions Hans de Goede
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).