Re: [PATCH v2 8/8] PCI: rzg3s-host: Add 100 ms delay after link training

Hi Hans,

kernel test robot noticed the following build warnings:

[auto build test WARNING on a293ec25d59dd96309058c70df5a4dd0f889a1e4]

url:    https://github.com/intel-lab-lkp/linux/commits/Hans-Zhang/PCI-Add-pcie_wait_after_link_train-helper/20260514-132815
base:   a293ec25d59dd96309058c70df5a4dd0f889a1e4
patch link:    https://lore.kernel.org/r/20260506152346.166056-9-18255117159%40163.com
patch subject: [PATCH v2 8/8] PCI: rzg3s-host: Add 100 ms delay after link training
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20260514/[email protected]/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/20260514/[email protected]/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 <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

All warnings (new ones prefixed by >>):

   In file included from drivers/pci/hotplug/pciehp_ctrl.c:24:
   drivers/pci/hotplug/../pci.h: In function 'pcie_wait_after_link_train':
   drivers/pci/hotplug/../pci.h:73:17: error: implicit declaration of function 'msleep' [-Wimplicit-function-declaration]
      73 |                 msleep(PCIE_RESET_CONFIG_WAIT_MS);
         |                 ^~~~~~
   In file included from drivers/pci/hotplug/pciehp.h:21,
                    from drivers/pci/hotplug/pciehp_ctrl.c:25:
   include/linux/delay.h: At top level:
>> include/linux/delay.h:61:6: warning: conflicting types for 'msleep'; have 'void(unsigned int)'
      61 | void msleep(unsigned int msecs);
         |      ^~~~~~
   drivers/pci/hotplug/../pci.h:73:17: note: previous implicit declaration of 'msleep' with type 'void(unsigned int)'
      73 |                 msleep(PCIE_RESET_CONFIG_WAIT_MS);
         |                 ^~~~~~


vim +61 include/linux/delay.h

^1da177e4c3f41 Linus Torvalds   2005-04-16  56  
f3f3149f35b919 Alok Kataria     2008-06-23  57  extern unsigned long lpj_fine;
^1da177e4c3f41 Linus Torvalds   2005-04-16  58  void calibrate_delay(void);
ad1a48301f659a Arnd Bergmann    2023-05-17  59  unsigned long calibrate_delay_is_known(void);
8496ecd0bed4c7 Valdis Kletnieks 2019-03-07  60  void __attribute__((weak)) calibration_delay_done(void);
^1da177e4c3f41 Linus Torvalds   2005-04-16 @61  void msleep(unsigned int msecs);
^1da177e4c3f41 Linus Torvalds   2005-04-16  62  unsigned long msleep_interruptible(unsigned int msecs);
e4779015fd5d2f SeongJae Park    2021-12-10  63  void usleep_range_state(unsigned long min, unsigned long max,
e4779015fd5d2f SeongJae Park    2021-12-10  64  			unsigned int state);
e4779015fd5d2f SeongJae Park    2021-12-10  65  

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