Re: [PATCH 5/5] staging: rtl8723bs: modify variable names to comply with kernel naming convention
kernel test robot <[email protected]> Thu, 3 Apr 2025 10:35:47 +0800
| Newsgroups | dev.linux.lists.outreachy,dev.linux.lists.linux-staging,dev.linux.lists.oe-kbuild-all,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
Hi Abraham, kernel test robot noticed the following build warnings: [auto build test WARNING on staging/staging-testing] url: https://github.com/intel-lab-lkp/linux/commits/Abraham-Samuel-Adekunle/staging-rtl8723bs-correct-coding-style-by-preventing-lines-from-ending-with/20250328-204628 base: staging/staging-testing patch link: https://lore.kernel.org/r/dd32dfe6c837d88dd13a546aadcb0bc207b997d6.1743163801.git.abrahamadekunle50%40gmail.com patch subject: [PATCH 5/5] staging: rtl8723bs: modify variable names to comply with kernel naming convention config: i386-randconfig-063-20250403 (https://download.01.org/0day-ci/archive/20250403/[email protected]/config) compiler: gcc-12 (Debian 12.2.0-14) 12.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250403/[email protected]/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <[email protected]> | Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/ sparse warnings: (new ones prefixed by >>) >> drivers/staging/rtl8723bs/core/rtw_efuse.c:13:4: sparse: sparse: symbol 'fake_efuse_bank' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:14:5: sparse: sparse: symbol 'fake_efuse_used_bytes' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:15:4: sparse: sparse: symbol 'fake_efuse_content' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:16:4: sparse: sparse: symbol 'fake_efuse_init_map' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:17:4: sparse: sparse: symbol 'fake_efuse_modified_map' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:19:5: sparse: sparse: symbol 'bte_fuse_used_bytes' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:20:4: sparse: sparse: symbol 'bte_fuse_content' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:21:4: sparse: sparse: symbol 'bte_use_init_map' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:22:4: sparse: sparse: symbol 'bte_use_modified_map' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:24:5: sparse: sparse: symbol 'fakebte_fuse_used_bytes' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:25:4: sparse: sparse: symbol 'fakebte_fuse_content' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:26:4: sparse: sparse: symbol 'fakebte_use_init_map' was not declared. Should it be static? >> drivers/staging/rtl8723bs/core/rtw_efuse.c:27:4: sparse: sparse: symbol 'fakebte_use_modified_map' was not declared. Should it be static? vim +/fake_efuse_bank +13 drivers/staging/rtl8723bs/core/rtw_efuse.c 10 11 12 /* Define global variables */ > 13 u8 fake_efuse_bank; > 14 u32 fake_efuse_used_bytes; > 15 u8 fake_efuse_content[EFUSE_MAX_HW_SIZE] = {0}; > 16 u8 fake_efuse_init_map[EFUSE_MAX_MAP_LEN] = {0}; > 17 u8 fake_efuse_modified_map[EFUSE_MAX_MAP_LEN] = {0}; 18 > 19 u32 bte_fuse_used_bytes; > 20 u8 bte_fuse_content[EFUSE_MAX_BT_BANK][EFUSE_MAX_HW_SIZE]; > 21 u8 bte_use_init_map[EFUSE_BT_MAX_MAP_LEN] = {0}; > 22 u8 bte_use_modified_map[EFUSE_BT_MAX_MAP_LEN] = {0}; 23 > 24 u32 fakebte_fuse_used_bytes; > 25 u8 fakebte_fuse_content[EFUSE_MAX_BT_BANK][EFUSE_MAX_HW_SIZE]; > 26 u8 fakebte_use_init_map[EFUSE_BT_MAX_MAP_LEN] = {0}; > 27 u8 fakebte_use_modified_map[EFUSE_BT_MAX_MAP_LEN] = {0}; 28 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki