linux-kernel - drivers/usb/renesas_usbhs/mod.c:195:6: warning: variable 'intenb0' set but not used

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <202412050247.vVthPDpE-lkp@intel.com>
Date: Thu, 5 Dec 2024 02:15:36 +0800
From: kernel test robot <lkp@...el.com>
To: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
Cc: llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev,
	linux-kernel@...r.kernel.org
Subject: drivers/usb/renesas_usbhs/mod.c:195:6: warning: variable 'intenb0'
 set but not used

Hi Lad,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   feffde684ac29a3b7aec82d2df850fbdbdee55e4
commit: 22ae6415c702680b1d3170717ce34ee31656f8b6 usb: renesas_usbhs: Use platform_get_irq() to get the interrupt
date:   2 years, 11 months ago
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20241205/202412050247.vVthPDpE-lkp@intel.com/config)
compiler: ClangBuiltLinux clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241205/202412050247.vVthPDpE-lkp@intel.com/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 <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202412050247.vVthPDpE-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/usb/renesas_usbhs/mod.c:195:6: warning: variable 'intenb0' set but not used [-Wunused-but-set-variable]
     195 |         u16 intenb0, intenb1;
         |             ^
   1 warning generated.


vim +/intenb0 +195 drivers/usb/renesas_usbhs/mod.c

f1407d5c66240b3 Kuninori Morimoto  2011-04-04  190  
697d5c004e39010 Shimoda, Yoshihiro 2012-08-20  191  static int usbhs_status_get_each_irq(struct usbhs_priv *priv,
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  192  				     struct usbhs_irq_state *state)
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  193  {
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  194  	struct usbhs_mod *mod = usbhs_mod_get_current(priv);
697d5c004e39010 Shimoda, Yoshihiro 2012-08-20 @195  	u16 intenb0, intenb1;
c4d8199ba1a7aa3 Yoshihiro Shimoda  2014-08-22  196  	unsigned long flags;
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  197  
c4d8199ba1a7aa3 Yoshihiro Shimoda  2014-08-22  198  	/********************  spin lock ********************/
c4d8199ba1a7aa3 Yoshihiro Shimoda  2014-08-22  199  	usbhs_lock(priv, flags);
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  200  	state->intsts0 = usbhs_read(priv, INTSTS0);
697d5c004e39010 Shimoda, Yoshihiro 2012-08-20  201  	intenb0 = usbhs_read(priv, INTENB0);
88a25e02f35e56a Nobuhiro Iwamatsu  2015-01-09  202  
88a25e02f35e56a Nobuhiro Iwamatsu  2015-01-09  203  	if (usbhs_mod_is_host(priv)) {
88a25e02f35e56a Nobuhiro Iwamatsu  2015-01-09  204  		state->intsts1 = usbhs_read(priv, INTSTS1);
697d5c004e39010 Shimoda, Yoshihiro 2012-08-20  205  		intenb1 = usbhs_read(priv, INTENB1);
672bfdaa3100043 Arnd Bergmann      2015-05-22  206  	} else {
672bfdaa3100043 Arnd Bergmann      2015-05-22  207  		state->intsts1 = intenb1 = 0;
88a25e02f35e56a Nobuhiro Iwamatsu  2015-01-09  208  	}
697d5c004e39010 Shimoda, Yoshihiro 2012-08-20  209  
5ea68d541df200d Kuninori Morimoto  2011-04-28  210  	/* mask */
5ea68d541df200d Kuninori Morimoto  2011-04-28  211  	if (mod) {
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  212  		state->brdysts = usbhs_read(priv, BRDYSTS);
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  213  		state->nrdysts = usbhs_read(priv, NRDYSTS);
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  214  		state->bempsts = usbhs_read(priv, BEMPSTS);
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  215  
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  216  		state->bempsts &= mod->irq_bempsts;
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  217  		state->brdysts &= mod->irq_brdysts;
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  218  	}
c4d8199ba1a7aa3 Yoshihiro Shimoda  2014-08-22  219  	usbhs_unlock(priv, flags);
c4d8199ba1a7aa3 Yoshihiro Shimoda  2014-08-22  220  	/********************  spin unlock ******************/
697d5c004e39010 Shimoda, Yoshihiro 2012-08-20  221  
697d5c004e39010 Shimoda, Yoshihiro 2012-08-20  222  	return 0;
5ea68d541df200d Kuninori Morimoto  2011-04-28  223  }
f1407d5c66240b3 Kuninori Morimoto  2011-04-04  224  

:::::: The code at line 195 was first introduced by commit
:::::: 697d5c004e390102efbf9320a5416873679bea81 usb: renesas_usbhs: modify the irq handler for sharing irq

:::::: TO: Shimoda, Yoshihiro <yoshihiro.shimoda.uh@...esas.com>
:::::: CC: Greg Kroah-Hartman <gregkh@...uxfoundation.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki