From: kernel test robot <lkp@intel.com>
To: Phillip Potter <phil@philpotter.co.uk>
Cc: devel@driverdev.osuosl.org,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
kbuild-all@lists.01.org
Subject: [staging:staging-testing 371/409] drivers/staging/rtl8188eu/core/rtw_efuse.c:239:8: warning: variable 'aaa' set but not used
Date: Tue, 15 Jun 2021 22:39:22 +0800 [thread overview]
Message-ID: <202106152218.X9kSeARr-lkp@intel.com> (raw)
[-- 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
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [staging:staging-testing 371/409] drivers/staging/rtl8188eu/core/rtw_efuse.c:239:8: warning: variable 'aaa' set but not used
Date: Tue, 15 Jun 2021 22:39:22 +0800 [thread overview]
Message-ID: <202106152218.X9kSeARr-lkp@intel.com> (raw)
[-- 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 --]
next reply other threads:[~2021-06-15 14:39 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-15 14:39 kernel test robot [this message]
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
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202106152218.X9kSeARr-lkp@intel.com \
--to=lkp@intel.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=kbuild-all@lists.01.org \
--cc=phil@philpotter.co.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.