[android-common:android16-6.12 1/1] drivers/usb/host/xhci-sideband.c:263: warning: Function parameter or struct member 'intr_num' not described in 'xhci_sideband_create_interrupter'

[android-common:android16-6.12 1/1] drivers/usb/host/xhci-sideband.c:263: warning: Function parameter or struct member 'intr_num' not described in 'xhci_sideband_create_interrupter'

[kernel test robot]

Hi Wesley,

FYI, the error/warning still remains.

tree:   https://android.googlesource.com/kernel/common android16-6.12
head:   6417b973d924236b080ef26fbfee42586a7b3704
commit: cca9824fd10fa5cf91be0a83f71a9ced6adc7c2a [1/1] FROMLIST: usb: host: xhci-mem: Allow for interrupter clients to choose specific index
config: x86_64-buildonly-randconfig-002-20260708 (https://download.01.org/0day-ci/archive/20260709/202607090508.c5sbtn7d-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260709/202607090508.c5sbtn7d-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/202607090508.c5sbtn7d-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/usb/host/xhci-sideband.c:263: warning: Function parameter or struct member 'imod_interval' not described in 'xhci_sideband_create_interrupter'
>> drivers/usb/host/xhci-sideband.c:263: warning: Function parameter or struct member 'intr_num' not described in 'xhci_sideband_create_interrupter'
   drivers/usb/host/xhci-sideband.c:346: warning: Function parameter or struct member 'type' not described in 'xhci_sideband_register'


vim +263 drivers/usb/host/xhci-sideband.c

7b946a32948f90 Mathias Nyman 2023-02-06  244  
7b946a32948f90 Mathias Nyman 2023-02-06  245  /**
7b946a32948f90 Mathias Nyman 2023-02-06  246   * xhci_sideband_create_interrupter - creates a new interrupter for this sideband
7b946a32948f90 Mathias Nyman 2023-02-06  247   * @sb: sideband instance for this usb device
7b946a32948f90 Mathias Nyman 2023-02-06  248   * @num_seg: number of event ring segments to allocate
7b946a32948f90 Mathias Nyman 2023-02-06  249   * @ip_autoclear: IP autoclearing support such as MSI implemented
7b946a32948f90 Mathias Nyman 2023-02-06  250   *
7b946a32948f90 Mathias Nyman 2023-02-06  251   * Sets up a xhci interrupter that can be used for this sideband accessed usb
7b946a32948f90 Mathias Nyman 2023-02-06  252   * device. Transfer events for this device can be routed to this interrupters
7b946a32948f90 Mathias Nyman 2023-02-06  253   * event ring by setting the 'Interrupter Target' field correctly when queueing
7b946a32948f90 Mathias Nyman 2023-02-06  254   * the transfer TRBs.
7b946a32948f90 Mathias Nyman 2023-02-06  255   * Once this interrupter is created the interrupter target ID can be obtained
7b946a32948f90 Mathias Nyman 2023-02-06  256   * by calling xhci_sideband_interrupter_id()
7b946a32948f90 Mathias Nyman 2023-02-06  257   *
7b946a32948f90 Mathias Nyman 2023-02-06  258   * Returns 0 on success, negative error otherwise
7b946a32948f90 Mathias Nyman 2023-02-06  259   */
7b946a32948f90 Mathias Nyman 2023-02-06  260  int
7b946a32948f90 Mathias Nyman 2023-02-06  261  xhci_sideband_create_interrupter(struct xhci_sideband *sb, int num_seg,
cca9824fd10fa5 Wesley Cheng  2023-05-09  262  				 bool ip_autoclear, u32 imod_interval, int intr_num)
7b946a32948f90 Mathias Nyman 2023-02-06 @263  {
7b946a32948f90 Mathias Nyman 2023-02-06  264  	int ret = 0;
7b946a32948f90 Mathias Nyman 2023-02-06  265  
7b946a32948f90 Mathias Nyman 2023-02-06  266  	if (!sb || !sb->xhci)
7b946a32948f90 Mathias Nyman 2023-02-06  267  		return -ENODEV;
7b946a32948f90 Mathias Nyman 2023-02-06  268  
7b946a32948f90 Mathias Nyman 2023-02-06  269  	mutex_lock(&sb->mutex);
7b946a32948f90 Mathias Nyman 2023-02-06  270  	if (sb->ir) {
7b946a32948f90 Mathias Nyman 2023-02-06  271  		ret = -EBUSY;
7b946a32948f90 Mathias Nyman 2023-02-06  272  		goto out;
7b946a32948f90 Mathias Nyman 2023-02-06  273  	}
7b946a32948f90 Mathias Nyman 2023-02-06  274  
7b946a32948f90 Mathias Nyman 2023-02-06  275  	sb->ir = xhci_create_secondary_interrupter(xhci_to_hcd(sb->xhci),
cca9824fd10fa5 Wesley Cheng  2023-05-09  276  						   num_seg, imod_interval,
cca9824fd10fa5 Wesley Cheng  2023-05-09  277  						   intr_num);
7b946a32948f90 Mathias Nyman 2023-02-06  278  	if (!sb->ir) {
7b946a32948f90 Mathias Nyman 2023-02-06  279  		ret = -ENOMEM;
7b946a32948f90 Mathias Nyman 2023-02-06  280  		goto out;
7b946a32948f90 Mathias Nyman 2023-02-06  281  	}
7b946a32948f90 Mathias Nyman 2023-02-06  282  
7b946a32948f90 Mathias Nyman 2023-02-06  283  	sb->ir->ip_autoclear = ip_autoclear;
7b946a32948f90 Mathias Nyman 2023-02-06  284  
7b946a32948f90 Mathias Nyman 2023-02-06  285  out:
7b946a32948f90 Mathias Nyman 2023-02-06  286  	mutex_unlock(&sb->mutex);
7b946a32948f90 Mathias Nyman 2023-02-06  287  
7b946a32948f90 Mathias Nyman 2023-02-06  288  	return ret;
7b946a32948f90 Mathias Nyman 2023-02-06  289  }
7b946a32948f90 Mathias Nyman 2023-02-06  290  EXPORT_SYMBOL_GPL(xhci_sideband_create_interrupter);
7b946a32948f90 Mathias Nyman 2023-02-06  291  

:::::: The code at line 263 was first introduced by commit
:::::: 7b946a32948f90b198e3e51c824576458ceda3a4 FROMLIST: xhci: sideband: add initial api to register a secondary interrupter entity

:::::: TO: Mathias Nyman <mathias.nyman@linux.intel.com>
:::::: CC: Neill Kapron <nkapron@google.com>

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