Re: [PATCH v4 2/2] fpga: Add support for Lattice iCE40 FPGAs

[kbuild test robot]

Hi Joel,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.9-rc3 next-20161028]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
[Suggest to use git(>=2.9.0) format-patch --base=<commit> (or --base=auto for convenience) to record what (public, well-known) commit your patch series was built on]
[Check https://git-scm.com/docs/git-format-patch for more information]

url:    https://github.com/0day-ci/linux/commits/Joel-Holdsworth/of-Add-vendor-prefix-for-Lattice-Semiconductor/20161030-053525
config: i386-allyesconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/fpga/ice40-spi.c:203:1: warning: data definition has no type or storage class
    MODULE_DEVICE_TABLE(of, ice40_fpga_of_match);
    ^~~~~~~~~~~~~~~~~~~
>> drivers/fpga/ice40-spi.c:203:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int]
   drivers/fpga/ice40-spi.c:203:1: warning: parameter names (without types) in function declaration
   drivers/fpga/ice40-spi.c:215:229: warning: data definition has no type or storage class
    module_spi_driver(ice40_fpga_driver);
                                                                                                                                                                                                                                        ^          
   drivers/fpga/ice40-spi.c:215:229: error: type defaults to 'int' in declaration of 'module_init' [-Werror=implicit-int]
   drivers/fpga/ice40-spi.c:215:191: warning: parameter names (without types) in function declaration
    module_spi_driver(ice40_fpga_driver);
                                                                                                                                                                                                  ^     
   drivers/fpga/ice40-spi.c:215:492: warning: data definition has no type or storage class
    module_spi_driver(ice40_fpga_driver);
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               ^          
   drivers/fpga/ice40-spi.c:215:492: error: type defaults to 'int' in declaration of 'module_exit' [-Werror=implicit-int]
   drivers/fpga/ice40-spi.c:215:191: warning: parameter names (without types) in function declaration
    module_spi_driver(ice40_fpga_driver);
                                                                                                                                                                                                  ^     
>> drivers/fpga/ice40-spi.c:217:15: error: expected declaration specifiers or '...' before string constant
    MODULE_AUTHOR("Joel Holdsworth <joel@xxxxxxxxxxxxxxxxxxx>");
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/fpga/ice40-spi.c:218:20: error: expected declaration specifiers or '...' before string constant
    MODULE_DESCRIPTION("Lattice iCE40 FPGA Manager");
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/fpga/ice40-spi.c:219:16: error: expected declaration specifiers or '...' before string constant
    MODULE_LICENSE("GPL v2");
                   ^~~~~~~~
   drivers/fpga/ice40-spi.c:215:122: warning: 'ice40_fpga_driver_init' defined but not used [-Wunused-function]
    module_spi_driver(ice40_fpga_driver);
                                                                                                                             ^
    
                                                                                                                              
   cc1: some warnings being treated as errors

vim +203 drivers/fpga/ice40-spi.c

   197	
   198	#ifdef CONFIG_OF
   199	static const struct of_device_id ice40_fpga_of_match[] = {
   200		{ .compatible = "lattice,ice40-fpga-mgr", },
   201		{},
   202	};
 > 203	MODULE_DEVICE_TABLE(of, ice40_fpga_of_match);
   204	#endif
   205	
   206	static struct spi_driver ice40_fpga_driver = {
   207		.probe = ice40_fpga_probe,
   208		.remove = ice40_fpga_remove,
   209		.driver = {
   210			.name = "ice40spi",
   211			.of_match_table = of_match_ptr(ice40_fpga_of_match),
   212		},
   213	};
   214	
 > 215	module_spi_driver(ice40_fpga_driver);
   216	
 > 217	MODULE_AUTHOR("Joel Holdsworth <joel@xxxxxxxxxxxxxxxxxxx>");
   218	MODULE_DESCRIPTION("Lattice iCE40 FPGA Manager");
   219	MODULE_LICENSE("GPL v2");

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip