Re: [usb:usb-testing 36/37] drivers/usb/core/usb.c:648:18: warning: variable 'root_hub' set but not used

[Greg KH <gregkh@linuxfoundation.org>]

On Wed, Aug 09, 2023 at 11:35:04AM -0400, Alan Stern wrote:
> On Wed, Aug 09, 2023 at 11:11:29PM +0800, kernel test robot wrote:
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
> > head:   f176638af476c6d46257cc3303f5c7cf47d5967d
> > commit: 1e4c574225cc5a0553115e5eb5787d1474db5b0f [36/37] USB: Remove remnants of Wireless USB and UWB
> > config: loongarch-allyesconfig (https://download.01.org/0day-ci/archive/20230809/202308092350.HR4PVHUt-lkp@intel.com/config)
> > compiler: loongarch64-linux-gcc (GCC) 12.3.0
> > reproduce: (https://download.01.org/0day-ci/archive/20230809/202308092350.HR4PVHUt-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@intel.com>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202308092350.HR4PVHUt-lkp@intel.com/
> > 
> > All warnings (new ones prefixed by >>):
> > 
> >    drivers/usb/core/usb.c: In function 'usb_alloc_dev':
> > >> drivers/usb/core/usb.c:648:18: warning: variable 'root_hub' set but not used [-Wunused-but-set-variable]
> >      648 |         unsigned root_hub = 0;
> >          |                  ^~~~~~~~
> 
> Odd.  I'm quite sure that I built this with no problems.  [Does some 
> checking...]  Aha!  When I built it, I didn't use W=1. 
> 
> Greg, would you prefer a new version of this patch or a separate 
> correction patch?

A new version, let me go drop the three patches from you for this as you
found a different issue on the first one already.  Just resend a new
series please.

thanks,

greg k-h