[staging:staging-testing 373/409] drivers/staging/rtl8188eu/core/rtw_mlme_ext.c:1771:6: warning: variable 'passing_time' set but not used

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



tree:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
head:   5b4f167ef3555ec4c334a8dc89c1b44bb2c6bff5
commit: e83e5dbbf4b79255cbda59522f8bcbda50d26f56 [373/409] staging: rtl8188eu: remove all DBG_88E calls from core/rtw_mlme_ext.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=e83e5dbbf4b79255cbda59522f8bcbda50d26f56
        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 e83e5dbbf4b79255cbda59522f8bcbda50d26f56
        # 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@xxxxxxxxx>

All warnings (new ones prefixed by >>):

   drivers/staging/rtl8188eu/core/rtw_mlme_ext.c: In function 'send_beacon':
>> drivers/staging/rtl8188eu/core/rtw_mlme_ext.c:1771:6: warning: variable 'passing_time' set but not used [-Wunused-but-set-variable]
    1771 |  u32 passing_time;
         |      ^~~~~~~~~~~~
   drivers/staging/rtl8188eu/core/rtw_mlme_ext.c: In function 'OnAssocReq':
>> drivers/staging/rtl8188eu/core/rtw_mlme_ext.c:2759:16: warning: variable 'reassoc' set but not used [-Wunused-but-set-variable]
    2759 |  unsigned char reassoc, *p, *pos, *wpa_ie;
         |                ^~~~~~~
   drivers/staging/rtl8188eu/core/rtw_mlme_ext.c: In function 'OnAction_back':
>> drivers/staging/rtl8188eu/core/rtw_mlme_ext.c:3400:30: warning: variable 'reason_code' set but not used [-Wunused-but-set-variable]
    3400 |  unsigned short tid, status, reason_code = 0;
         |                              ^~~~~~~~~~~


vim +/passing_time +1771 drivers/staging/rtl8188eu/core/rtw_mlme_ext.c

7b464c9fa5cc27 Larry Finger    2013-08-21  1764  
4e102344534354 Luca Ceresoli   2015-06-12  1765  unsigned int send_beacon(struct adapter *padapter)
4e102344534354 Luca Ceresoli   2015-06-12  1766  {
4e102344534354 Luca Ceresoli   2015-06-12  1767  	u8 bxmitok = false;
4e102344534354 Luca Ceresoli   2015-06-12  1768  	int issue = 0;
4e102344534354 Luca Ceresoli   2015-06-12  1769  	int poll = 0;
ed73749426deb2 Rémy Oudompheng 2015-11-02  1770  	unsigned long start = jiffies;
0b4d1d0da7e16f Michael Straube 2020-01-18 @1771  	u32 passing_time;
4e102344534354 Luca Ceresoli   2015-06-12  1772  
4e102344534354 Luca Ceresoli   2015-06-12  1773  	rtw_hal_set_hwreg(padapter, HW_VAR_BCN_VALID, NULL);
4e102344534354 Luca Ceresoli   2015-06-12  1774  	do {
4e102344534354 Luca Ceresoli   2015-06-12  1775  		issue_beacon(padapter, 100);
4e102344534354 Luca Ceresoli   2015-06-12  1776  		issue++;
4e102344534354 Luca Ceresoli   2015-06-12  1777  		do {
4e102344534354 Luca Ceresoli   2015-06-12  1778  			yield();
4e102344534354 Luca Ceresoli   2015-06-12  1779  			rtw_hal_get_hwreg(padapter, HW_VAR_BCN_VALID, (u8 *)(&bxmitok));
4e102344534354 Luca Ceresoli   2015-06-12  1780  			poll++;
4e102344534354 Luca Ceresoli   2015-06-12  1781  		} while ((poll % 10) != 0 && !bxmitok && !padapter->bSurpriseRemoved && !padapter->bDriverStopped);
4e102344534354 Luca Ceresoli   2015-06-12  1782  	} while (!bxmitok && issue < 100 && !padapter->bSurpriseRemoved && !padapter->bDriverStopped);
4e102344534354 Luca Ceresoli   2015-06-12  1783  
4e102344534354 Luca Ceresoli   2015-06-12  1784  	if (padapter->bSurpriseRemoved || padapter->bDriverStopped)
4e102344534354 Luca Ceresoli   2015-06-12  1785  		return _FAIL;
e83e5dbbf4b792 Phillip Potter  2021-06-15  1786  	if (!bxmitok)
4e102344534354 Luca Ceresoli   2015-06-12  1787  		return _FAIL;
0b4d1d0da7e16f Michael Straube 2020-01-18  1788  	passing_time = jiffies_to_msecs(jiffies - start);
7b464c9fa5cc27 Larry Finger    2013-08-21  1789  
4e102344534354 Luca Ceresoli   2015-06-12  1790  	return _SUCCESS;
4e102344534354 Luca Ceresoli   2015-06-12  1791  }
7b464c9fa5cc27 Larry Finger    2013-08-21  1792  

:::::: The code at line 1771 was first introduced by commit
:::::: 0b4d1d0da7e16f6faa572a8c7f865ff82e097819 staging: rtl8188eu: remove else after break or return

:::::: TO: Michael Straube <straube.linux@xxxxxxxxx>
:::::: CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux