Re: [PATCH v4 4/9] spi: Add API to count spi acpi resources

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2992 bytes --]

Hi Stefan,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on next-20220120]
[also build test WARNING on v5.16]
[cannot apply to broonie-spi/for-next rafael-pm/linux-next tiwai-sound/for-next linus/master v5.16 v5.16-rc8 v5.16-rc7]
[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]

url:    https://github.com/0day-ci/linux/commits/Stefan-Binding/Support-Spi-in-i2c-multi-instantiate-driver/20220120-214608
base:    7fc5253f5a13271e9df35d6b936ff97b74540a59
config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20220121/202201210109.tNnI8P2j-lkp(a)intel.com/config)
compiler: alpha-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/684295c62acdefb3fab20a0e7633793ba748c6d7
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Stefan-Binding/Support-Spi-in-i2c-multi-instantiate-driver/20220120-214608
        git checkout 684295c62acdefb3fab20a0e7633793ba748c6d7
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=alpha SHELL=/bin/bash drivers/char/tpm/st33zp24/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from drivers/char/tpm/st33zp24/spi.c:8:
   include/linux/spi/spi.h:772:1: error: expected identifier or '(' before '{' token
     772 | {
         | ^
>> include/linux/spi/spi.h:776:5: warning: no previous prototype for 'acpi_spi_count_resources' [-Wmissing-prototypes]
     776 | int acpi_spi_count_resources(struct acpi_device *adev)
         |     ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/spi/spi.h:769:34: warning: 'acpi_spi_device_alloc' declared 'static' but never defined [-Wunused-function]
     769 | static inline struct spi_device *acpi_spi_device_alloc(struct spi_controller *ctlr,
         |                                  ^~~~~~~~~~~~~~~~~~~~~
   drivers/char/tpm/st33zp24/spi.c:408:36: warning: 'st33zp24_spi_acpi_match' defined but not used [-Wunused-const-variable=]
     408 | static const struct acpi_device_id st33zp24_spi_acpi_match[] = {
         |                                    ^~~~~~~~~~~~~~~~~~~~~~~


vim +/acpi_spi_count_resources +776 include/linux/spi/spi.h

   775	
 > 776	int acpi_spi_count_resources(struct acpi_device *adev)
   777	{
   778		return -EOPNOTSUPP;
   779	}
   780	#endif
   781	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org