* [staging:staging-testing 371/409] drivers/staging/rtl8188eu/core/rtw_efuse.c:239:8: warning: variable 'aaa' set but not used
@ 2021-06-15 14:39 ` kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-06-15 14:39 UTC (permalink / raw)
To: Phillip Potter; +Cc: devel, Greg Kroah-Hartman, kbuild-all
[-- Attachment #1: Type: text/plain, Size: 8495 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
head: 5b4f167ef3555ec4c334a8dc89c1b44bb2c6bff5
commit: 9a6780227ece18b6ad1ae6514a46a6850dfa9971 [371/409] staging: rtl8188eu: remove all DBG_88E calls from core/rtw_efuse.c
config: mips-allyesconfig (attached as .config)
compiler: mips-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/commit/?id=9a6780227ece18b6ad1ae6514a46a6850dfa9971
git remote add staging https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
git fetch --no-tags staging staging-testing
git checkout 9a6780227ece18b6ad1ae6514a46a6850dfa9971
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=mips
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/staging/rtl8188eu/core/rtw_efuse.c: In function 'efuse_read_phymap_from_txpktbuf':
>> drivers/staging/rtl8188eu/core/rtw_efuse.c:239:8: warning: variable 'aaa' set but not used [-Wunused-but-set-variable]
239 | u16 aaa;
| ^~~
>> drivers/staging/rtl8188eu/core/rtw_efuse.c:238:16: warning: variable 'aaabak' set but not used [-Wunused-but-set-variable]
238 | u16 lenbak, aaabak;
| ^~~~~~
>> drivers/staging/rtl8188eu/core/rtw_efuse.c:238:8: warning: variable 'lenbak' set but not used [-Wunused-but-set-variable]
238 | u16 lenbak, aaabak;
| ^~~~~~
vim +/aaa +239 drivers/staging/rtl8188eu/core/rtw_efuse.c
ee5f8a431ead8f3 navin patidar 2014-06-20 199
ee5f8a431ead8f3 navin patidar 2014-06-20 200 static void efuse_read_phymap_from_txpktbuf(
ee5f8a431ead8f3 navin patidar 2014-06-20 201 struct adapter *adapter,
ee5f8a431ead8f3 navin patidar 2014-06-20 202 int bcnhead, /* beacon head, where FW store len(2-byte) and efuse physical map. */
ee5f8a431ead8f3 navin patidar 2014-06-20 203 u8 *content, /* buffer to store efuse physical map */
ee5f8a431ead8f3 navin patidar 2014-06-20 204 u16 *size /* for efuse content: the max byte to read. will update to byte read */
ee5f8a431ead8f3 navin patidar 2014-06-20 205 )
ee5f8a431ead8f3 navin patidar 2014-06-20 206 {
ee5f8a431ead8f3 navin patidar 2014-06-20 207 u16 dbg_addr = 0;
ed73749426deb28 Rémy Oudompheng 2015-11-02 208 unsigned long start = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 209 u8 reg_0x143 = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 210 u32 lo32 = 0, hi32 = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 211 u16 len = 0, count = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 212 int i = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 213 u16 limit = *size;
ee5f8a431ead8f3 navin patidar 2014-06-20 214
ee5f8a431ead8f3 navin patidar 2014-06-20 215 u8 *pos = content;
ee5f8a431ead8f3 navin patidar 2014-06-20 216
ee5f8a431ead8f3 navin patidar 2014-06-20 217 if (bcnhead < 0) /* if not valid */
ee5f8a431ead8f3 navin patidar 2014-06-20 218 bcnhead = usb_read8(adapter, REG_TDECTRL + 1);
ee5f8a431ead8f3 navin patidar 2014-06-20 219
ee5f8a431ead8f3 navin patidar 2014-06-20 220 usb_write8(adapter, REG_PKT_BUFF_ACCESS_CTRL, TXPKT_BUF_SELECT);
ee5f8a431ead8f3 navin patidar 2014-06-20 221
ee5f8a431ead8f3 navin patidar 2014-06-20 222 dbg_addr = bcnhead * 128 / 8; /* 8-bytes addressing */
ee5f8a431ead8f3 navin patidar 2014-06-20 223
ee5f8a431ead8f3 navin patidar 2014-06-20 224 while (1) {
ee5f8a431ead8f3 navin patidar 2014-06-20 225 usb_write16(adapter, REG_PKTBUF_DBG_ADDR, dbg_addr + i);
ee5f8a431ead8f3 navin patidar 2014-06-20 226
ee5f8a431ead8f3 navin patidar 2014-06-20 227 usb_write8(adapter, REG_TXPKTBUF_DBG, 0);
ee5f8a431ead8f3 navin patidar 2014-06-20 228 start = jiffies;
ee5f8a431ead8f3 navin patidar 2014-06-20 229 while (!(reg_0x143 = usb_read8(adapter, REG_TXPKTBUF_DBG)) &&
9a6780227ece18b Phillip Potter 2021-06-15 230 jiffies_to_msecs(jiffies - start) < 1000)
c8cb5f2c780a600 Jiayi Ye 2014-10-20 231 usleep_range(1000, 2000);
ee5f8a431ead8f3 navin patidar 2014-06-20 232
ee5f8a431ead8f3 navin patidar 2014-06-20 233 lo32 = usb_read32(adapter, REG_PKTBUF_DBG_DATA_L);
ee5f8a431ead8f3 navin patidar 2014-06-20 234 hi32 = usb_read32(adapter, REG_PKTBUF_DBG_DATA_H);
ee5f8a431ead8f3 navin patidar 2014-06-20 235
ee5f8a431ead8f3 navin patidar 2014-06-20 236 if (i == 0) {
ee5f8a431ead8f3 navin patidar 2014-06-20 237 u8 lenc[2];
ee5f8a431ead8f3 navin patidar 2014-06-20 @238 u16 lenbak, aaabak;
ee5f8a431ead8f3 navin patidar 2014-06-20 @239 u16 aaa;
7d2af82cc5f5713 Yamanappagouda Patil 2016-12-22 240
ee5f8a431ead8f3 navin patidar 2014-06-20 241 lenc[0] = usb_read8(adapter, REG_PKTBUF_DBG_DATA_L);
ee5f8a431ead8f3 navin patidar 2014-06-20 242 lenc[1] = usb_read8(adapter, REG_PKTBUF_DBG_DATA_L + 1);
ee5f8a431ead8f3 navin patidar 2014-06-20 243
ee5f8a431ead8f3 navin patidar 2014-06-20 244 aaabak = le16_to_cpup((__le16 *)lenc);
ee5f8a431ead8f3 navin patidar 2014-06-20 245 lenbak = le16_to_cpu(*((__le16 *)lenc));
ee5f8a431ead8f3 navin patidar 2014-06-20 246 aaa = le16_to_cpup((__le16 *)&lo32);
ee5f8a431ead8f3 navin patidar 2014-06-20 247 len = le16_to_cpu(*((__le16 *)&lo32));
ee5f8a431ead8f3 navin patidar 2014-06-20 248
530c9b1b21fc41f Ivan Safonov 2015-10-27 249 limit = min_t(u16, len - 2, limit);
ee5f8a431ead8f3 navin patidar 2014-06-20 250
ee5f8a431ead8f3 navin patidar 2014-06-20 251 memcpy(pos, ((u8 *)&lo32) + 2, (limit >= count + 2) ? 2 : limit - count);
ee5f8a431ead8f3 navin patidar 2014-06-20 252 count += (limit >= count + 2) ? 2 : limit - count;
ee5f8a431ead8f3 navin patidar 2014-06-20 253 pos = content + count;
ee5f8a431ead8f3 navin patidar 2014-06-20 254
ee5f8a431ead8f3 navin patidar 2014-06-20 255 } else {
ee5f8a431ead8f3 navin patidar 2014-06-20 256 memcpy(pos, ((u8 *)&lo32), (limit >= count + 4) ? 4 : limit - count);
ee5f8a431ead8f3 navin patidar 2014-06-20 257 count += (limit >= count + 4) ? 4 : limit - count;
ee5f8a431ead8f3 navin patidar 2014-06-20 258 pos = content + count;
ee5f8a431ead8f3 navin patidar 2014-06-20 259 }
ee5f8a431ead8f3 navin patidar 2014-06-20 260
ee5f8a431ead8f3 navin patidar 2014-06-20 261 if (limit > count && len - 2 > count) {
ee5f8a431ead8f3 navin patidar 2014-06-20 262 memcpy(pos, (u8 *)&hi32, (limit >= count + 4) ? 4 : limit - count);
ee5f8a431ead8f3 navin patidar 2014-06-20 263 count += (limit >= count + 4) ? 4 : limit - count;
ee5f8a431ead8f3 navin patidar 2014-06-20 264 pos = content + count;
ee5f8a431ead8f3 navin patidar 2014-06-20 265 }
ee5f8a431ead8f3 navin patidar 2014-06-20 266
ee5f8a431ead8f3 navin patidar 2014-06-20 267 if (limit <= count || len - 2 <= count)
ee5f8a431ead8f3 navin patidar 2014-06-20 268 break;
ee5f8a431ead8f3 navin patidar 2014-06-20 269 i++;
ee5f8a431ead8f3 navin patidar 2014-06-20 270 }
ee5f8a431ead8f3 navin patidar 2014-06-20 271 usb_write8(adapter, REG_PKT_BUFF_ACCESS_CTRL, DISABLE_TRXPKT_BUF_ACCESS);
ee5f8a431ead8f3 navin patidar 2014-06-20 272 *size = count;
ee5f8a431ead8f3 navin patidar 2014-06-20 273 }
ee5f8a431ead8f3 navin patidar 2014-06-20 274
:::::: The code at line 239 was first introduced by commit
:::::: ee5f8a431ead8f334f4c04f91ecafa34cb9d52ca staging: rtl8188eu: Move all efuse related code to rtw_efuse.c
:::::: TO: navin patidar <navin.patidar@gmail.com>
:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 71050 bytes --]
[-- Attachment #3: Type: text/plain, Size: 169 bytes --]
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
* [staging:staging-testing 371/409] drivers/staging/rtl8188eu/core/rtw_efuse.c:239:8: warning: variable 'aaa' set but not used
@ 2021-06-15 14:39 ` kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-06-15 14:39 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 8615 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
head: 5b4f167ef3555ec4c334a8dc89c1b44bb2c6bff5
commit: 9a6780227ece18b6ad1ae6514a46a6850dfa9971 [371/409] staging: rtl8188eu: remove all DBG_88E calls from core/rtw_efuse.c
config: mips-allyesconfig (attached as .config)
compiler: mips-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/commit/?id=9a6780227ece18b6ad1ae6514a46a6850dfa9971
git remote add staging https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
git fetch --no-tags staging staging-testing
git checkout 9a6780227ece18b6ad1ae6514a46a6850dfa9971
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=mips
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/staging/rtl8188eu/core/rtw_efuse.c: In function 'efuse_read_phymap_from_txpktbuf':
>> drivers/staging/rtl8188eu/core/rtw_efuse.c:239:8: warning: variable 'aaa' set but not used [-Wunused-but-set-variable]
239 | u16 aaa;
| ^~~
>> drivers/staging/rtl8188eu/core/rtw_efuse.c:238:16: warning: variable 'aaabak' set but not used [-Wunused-but-set-variable]
238 | u16 lenbak, aaabak;
| ^~~~~~
>> drivers/staging/rtl8188eu/core/rtw_efuse.c:238:8: warning: variable 'lenbak' set but not used [-Wunused-but-set-variable]
238 | u16 lenbak, aaabak;
| ^~~~~~
vim +/aaa +239 drivers/staging/rtl8188eu/core/rtw_efuse.c
ee5f8a431ead8f3 navin patidar 2014-06-20 199
ee5f8a431ead8f3 navin patidar 2014-06-20 200 static void efuse_read_phymap_from_txpktbuf(
ee5f8a431ead8f3 navin patidar 2014-06-20 201 struct adapter *adapter,
ee5f8a431ead8f3 navin patidar 2014-06-20 202 int bcnhead, /* beacon head, where FW store len(2-byte) and efuse physical map. */
ee5f8a431ead8f3 navin patidar 2014-06-20 203 u8 *content, /* buffer to store efuse physical map */
ee5f8a431ead8f3 navin patidar 2014-06-20 204 u16 *size /* for efuse content: the max byte to read. will update to byte read */
ee5f8a431ead8f3 navin patidar 2014-06-20 205 )
ee5f8a431ead8f3 navin patidar 2014-06-20 206 {
ee5f8a431ead8f3 navin patidar 2014-06-20 207 u16 dbg_addr = 0;
ed73749426deb28 Rémy Oudompheng 2015-11-02 208 unsigned long start = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 209 u8 reg_0x143 = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 210 u32 lo32 = 0, hi32 = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 211 u16 len = 0, count = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 212 int i = 0;
ee5f8a431ead8f3 navin patidar 2014-06-20 213 u16 limit = *size;
ee5f8a431ead8f3 navin patidar 2014-06-20 214
ee5f8a431ead8f3 navin patidar 2014-06-20 215 u8 *pos = content;
ee5f8a431ead8f3 navin patidar 2014-06-20 216
ee5f8a431ead8f3 navin patidar 2014-06-20 217 if (bcnhead < 0) /* if not valid */
ee5f8a431ead8f3 navin patidar 2014-06-20 218 bcnhead = usb_read8(adapter, REG_TDECTRL + 1);
ee5f8a431ead8f3 navin patidar 2014-06-20 219
ee5f8a431ead8f3 navin patidar 2014-06-20 220 usb_write8(adapter, REG_PKT_BUFF_ACCESS_CTRL, TXPKT_BUF_SELECT);
ee5f8a431ead8f3 navin patidar 2014-06-20 221
ee5f8a431ead8f3 navin patidar 2014-06-20 222 dbg_addr = bcnhead * 128 / 8; /* 8-bytes addressing */
ee5f8a431ead8f3 navin patidar 2014-06-20 223
ee5f8a431ead8f3 navin patidar 2014-06-20 224 while (1) {
ee5f8a431ead8f3 navin patidar 2014-06-20 225 usb_write16(adapter, REG_PKTBUF_DBG_ADDR, dbg_addr + i);
ee5f8a431ead8f3 navin patidar 2014-06-20 226
ee5f8a431ead8f3 navin patidar 2014-06-20 227 usb_write8(adapter, REG_TXPKTBUF_DBG, 0);
ee5f8a431ead8f3 navin patidar 2014-06-20 228 start = jiffies;
ee5f8a431ead8f3 navin patidar 2014-06-20 229 while (!(reg_0x143 = usb_read8(adapter, REG_TXPKTBUF_DBG)) &&
9a6780227ece18b Phillip Potter 2021-06-15 230 jiffies_to_msecs(jiffies - start) < 1000)
c8cb5f2c780a600 Jiayi Ye 2014-10-20 231 usleep_range(1000, 2000);
ee5f8a431ead8f3 navin patidar 2014-06-20 232
ee5f8a431ead8f3 navin patidar 2014-06-20 233 lo32 = usb_read32(adapter, REG_PKTBUF_DBG_DATA_L);
ee5f8a431ead8f3 navin patidar 2014-06-20 234 hi32 = usb_read32(adapter, REG_PKTBUF_DBG_DATA_H);
ee5f8a431ead8f3 navin patidar 2014-06-20 235
ee5f8a431ead8f3 navin patidar 2014-06-20 236 if (i == 0) {
ee5f8a431ead8f3 navin patidar 2014-06-20 237 u8 lenc[2];
ee5f8a431ead8f3 navin patidar 2014-06-20 @238 u16 lenbak, aaabak;
ee5f8a431ead8f3 navin patidar 2014-06-20 @239 u16 aaa;
7d2af82cc5f5713 Yamanappagouda Patil 2016-12-22 240
ee5f8a431ead8f3 navin patidar 2014-06-20 241 lenc[0] = usb_read8(adapter, REG_PKTBUF_DBG_DATA_L);
ee5f8a431ead8f3 navin patidar 2014-06-20 242 lenc[1] = usb_read8(adapter, REG_PKTBUF_DBG_DATA_L + 1);
ee5f8a431ead8f3 navin patidar 2014-06-20 243
ee5f8a431ead8f3 navin patidar 2014-06-20 244 aaabak = le16_to_cpup((__le16 *)lenc);
ee5f8a431ead8f3 navin patidar 2014-06-20 245 lenbak = le16_to_cpu(*((__le16 *)lenc));
ee5f8a431ead8f3 navin patidar 2014-06-20 246 aaa = le16_to_cpup((__le16 *)&lo32);
ee5f8a431ead8f3 navin patidar 2014-06-20 247 len = le16_to_cpu(*((__le16 *)&lo32));
ee5f8a431ead8f3 navin patidar 2014-06-20 248
530c9b1b21fc41f Ivan Safonov 2015-10-27 249 limit = min_t(u16, len - 2, limit);
ee5f8a431ead8f3 navin patidar 2014-06-20 250
ee5f8a431ead8f3 navin patidar 2014-06-20 251 memcpy(pos, ((u8 *)&lo32) + 2, (limit >= count + 2) ? 2 : limit - count);
ee5f8a431ead8f3 navin patidar 2014-06-20 252 count += (limit >= count + 2) ? 2 : limit - count;
ee5f8a431ead8f3 navin patidar 2014-06-20 253 pos = content + count;
ee5f8a431ead8f3 navin patidar 2014-06-20 254
ee5f8a431ead8f3 navin patidar 2014-06-20 255 } else {
ee5f8a431ead8f3 navin patidar 2014-06-20 256 memcpy(pos, ((u8 *)&lo32), (limit >= count + 4) ? 4 : limit - count);
ee5f8a431ead8f3 navin patidar 2014-06-20 257 count += (limit >= count + 4) ? 4 : limit - count;
ee5f8a431ead8f3 navin patidar 2014-06-20 258 pos = content + count;
ee5f8a431ead8f3 navin patidar 2014-06-20 259 }
ee5f8a431ead8f3 navin patidar 2014-06-20 260
ee5f8a431ead8f3 navin patidar 2014-06-20 261 if (limit > count && len - 2 > count) {
ee5f8a431ead8f3 navin patidar 2014-06-20 262 memcpy(pos, (u8 *)&hi32, (limit >= count + 4) ? 4 : limit - count);
ee5f8a431ead8f3 navin patidar 2014-06-20 263 count += (limit >= count + 4) ? 4 : limit - count;
ee5f8a431ead8f3 navin patidar 2014-06-20 264 pos = content + count;
ee5f8a431ead8f3 navin patidar 2014-06-20 265 }
ee5f8a431ead8f3 navin patidar 2014-06-20 266
ee5f8a431ead8f3 navin patidar 2014-06-20 267 if (limit <= count || len - 2 <= count)
ee5f8a431ead8f3 navin patidar 2014-06-20 268 break;
ee5f8a431ead8f3 navin patidar 2014-06-20 269 i++;
ee5f8a431ead8f3 navin patidar 2014-06-20 270 }
ee5f8a431ead8f3 navin patidar 2014-06-20 271 usb_write8(adapter, REG_PKT_BUFF_ACCESS_CTRL, DISABLE_TRXPKT_BUF_ACCESS);
ee5f8a431ead8f3 navin patidar 2014-06-20 272 *size = count;
ee5f8a431ead8f3 navin patidar 2014-06-20 273 }
ee5f8a431ead8f3 navin patidar 2014-06-20 274
:::::: The code@line 239 was first introduced by commit
:::::: ee5f8a431ead8f334f4c04f91ecafa34cb9d52ca staging: rtl8188eu: Move all efuse related code to rtw_efuse.c
:::::: TO: navin patidar <navin.patidar@gmail.com>
:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 71050 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-06-15 14:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-15 14:39 [staging:staging-testing 371/409] drivers/staging/rtl8188eu/core/rtw_efuse.c:239:8: warning: variable 'aaa' set but not used kernel test robot
2021-06-15 14:39 ` kernel test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.