Re: [PATCH] usb: core: remove unnecessary goto in usb_create_newid_files()
From: kernel test robot
Date: Wed Mar 04 2026 - 03:57:22 EST
Hi Mark,
kernel test robot noticed the following build warnings:
[auto build test WARNING on usb/usb-testing]
[also build test WARNING on usb/usb-next usb/usb-linus westeri-thunderbolt/next linus/master v6.16-rc1 next-20260303]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Mark-Adamenko/usb-core-remove-unnecessary-goto-in-usb_create_newid_files/20260304-113844
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
patch link: https://lore.kernel.org/r/20260304033418.49357-1-marusik.adamenko%40gmail.com
patch subject: [PATCH] usb: core: remove unnecessary goto in usb_create_newid_files()
config: riscv-allnoconfig-bpf (https://download.01.org/0day-ci/archive/20260304/202603040948.m6LKXcFm-lkp@xxxxxxxxx/config)
compiler: riscv64-linux-gnu-gcc (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260304/202603040948.m6LKXcFm-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603040948.m6LKXcFm-lkp@xxxxxxxxx/
All warnings (new ones prefixed by >>):
drivers/usb/core/driver.c: In function 'usb_create_newid_files':
>> drivers/usb/core/driver.c:202:1: warning: control reaches end of non-void function [-Wreturn-type]
202 | }
| ^
vim +202 drivers/usb/core/driver.c
0c7a2b72746a96 CHENG Renquan 2009-11-22 183
ed283e9f0a2cc0 Alan Stern 2012-01-24 184 static int usb_create_newid_files(struct usb_driver *usb_drv)
733260ff9c45bd Greg Kroah-Hartman 2005-11-16 185 {
733260ff9c45bd Greg Kroah-Hartman 2005-11-16 186 int error = 0;
733260ff9c45bd Greg Kroah-Hartman 2005-11-16 187
ba9dc657af86d0 Greg Kroah-Hartman 2005-11-16 188 if (usb_drv->no_dynamic_id)
5523ca85805b3e Mark Adamenko 2026-03-03 189 return error;
ba9dc657af86d0 Greg Kroah-Hartman 2005-11-16 190
ed283e9f0a2cc0 Alan Stern 2012-01-24 191 if (usb_drv->probe != NULL) {
49a78b05d5ca1e Yajun Deng 2024-01-04 192 error = driver_create_file(&usb_drv->driver,
15147ffd57576f Greg Kroah-Hartman 2007-11-28 193 &driver_attr_new_id);
ed283e9f0a2cc0 Alan Stern 2012-01-24 194 if (error == 0) {
49a78b05d5ca1e Yajun Deng 2024-01-04 195 error = driver_create_file(&usb_drv->driver,
ed283e9f0a2cc0 Alan Stern 2012-01-24 196 &driver_attr_remove_id);
ed283e9f0a2cc0 Alan Stern 2012-01-24 197 if (error)
49a78b05d5ca1e Yajun Deng 2024-01-04 198 driver_remove_file(&usb_drv->driver,
ed283e9f0a2cc0 Alan Stern 2012-01-24 199 &driver_attr_new_id);
ed283e9f0a2cc0 Alan Stern 2012-01-24 200 }
ed283e9f0a2cc0 Alan Stern 2012-01-24 201 }
733260ff9c45bd Greg Kroah-Hartman 2005-11-16 @202 }
733260ff9c45bd Greg Kroah-Hartman 2005-11-16 203
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki