* [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
* 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
* 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
* [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
* [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
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).