[alsa-devel] [asoc:for-next 9/9] sound/soc/codecs/wm9712.c:675:12: error: invalid storage class for function 'wm9712_soc_remove'

kbuild test robot fengguang.wu at intel.com
Tue Sep 19 19:11:19 CEST 2017


tree:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
head:   8c860a21cc365e822e70f970684179a50d22a77b
commit: 8c860a21cc365e822e70f970684179a50d22a77b [9/9] Merge remote-tracking branches 'asoc/topic/wm97xx' and 'asoc/topic/zte' into asoc-next
config: xtensa-allmodconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 4.9.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 8c860a21cc365e822e70f970684179a50d22a77b
        # save the attached .config to linux build tree
        make.cross ARCH=xtensa 

All error/warnings (new ones prefixed by >>):

   sound/soc/codecs/wm9712.c: In function 'wm9712_soc_probe':
>> sound/soc/codecs/wm9712.c:675:12: error: invalid storage class for function 'wm9712_soc_remove'
    static int wm9712_soc_remove(struct snd_soc_codec *codec)
               ^
>> sound/soc/codecs/wm9712.c:675:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
    static int wm9712_soc_remove(struct snd_soc_codec *codec)
    ^
>> sound/soc/codecs/wm9712.c:690:2: error: initializer element is not constant
     .remove =  wm9712_soc_remove,
     ^
   sound/soc/codecs/wm9712.c:690:2: error: (near initialization for 'soc_codec_dev_wm9712.remove')
>> sound/soc/codecs/wm9712.c:705:12: error: invalid storage class for function 'wm9712_probe'
    static int wm9712_probe(struct platform_device *pdev)
               ^
>> sound/soc/codecs/wm9712.c:722:12: error: invalid storage class for function 'wm9712_remove'
    static int wm9712_remove(struct platform_device *pdev)
               ^
   sound/soc/codecs/wm9712.c:733:2: error: initializer element is not constant
     .probe = wm9712_probe,
     ^
   sound/soc/codecs/wm9712.c:733:2: error: (near initialization for 'wm9712_codec_driver.probe')
   sound/soc/codecs/wm9712.c:734:2: error: initializer element is not constant
     .remove = wm9712_remove,
     ^
   sound/soc/codecs/wm9712.c:734:2: error: (near initialization for 'wm9712_codec_driver.remove')
   In file included from sound/soc/codecs/wm9712.c:18:0:
>> sound/soc/codecs/wm9712.c:737:24: error: invalid storage class for function 'wm9712_codec_driver_init'
    module_platform_driver(wm9712_codec_driver);
                           ^
   include/linux/device.h:1502:19: note: in definition of macro 'module_driver'
    static int __init __driver##_init(void) \
                      ^
>> sound/soc/codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
   In file included from sound/soc/codecs/wm9712.c:16:0:
>> include/linux/module.h:129:42: error: invalid storage class for function '__inittest'
     static inline initcall_t __maybe_unused __inittest(void)  \
                                             ^
   include/linux/device.h:1506:1: note: in expansion of macro 'module_init'
    module_init(__driver##_init); \
    ^
   include/linux/platform_device.h:228:2: note: in expansion of macro 'module_driver'
     module_driver(__platform_driver, platform_driver_register, \
     ^
>> sound/soc/codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
>> sound/soc/codecs/wm9712.c:737:1: warning: 'alias' attribute ignored [-Wattributes]
   In file included from sound/soc/codecs/wm9712.c:18:0:
>> sound/soc/codecs/wm9712.c:737:24: error: invalid storage class for function 'wm9712_codec_driver_exit'
    module_platform_driver(wm9712_codec_driver);
                           ^
   include/linux/device.h:1507:20: note: in definition of macro 'module_driver'
    static void __exit __driver##_exit(void) \
                       ^
>> sound/soc/codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
>> include/linux/device.h:1507:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
    static void __exit __driver##_exit(void) \
    ^
   include/linux/platform_device.h:228:2: note: in expansion of macro 'module_driver'
     module_driver(__platform_driver, platform_driver_register, \
     ^
>> sound/soc/codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
   In file included from sound/soc/codecs/wm9712.c:16:0:
>> include/linux/module.h:135:42: error: invalid storage class for function '__exittest'
     static inline exitcall_t __maybe_unused __exittest(void)  \
                                             ^
   include/linux/device.h:1511:1: note: in expansion of macro 'module_exit'
    module_exit(__driver##_exit);
    ^
   include/linux/platform_device.h:228:2: note: in expansion of macro 'module_driver'
     module_driver(__platform_driver, platform_driver_register, \
     ^
>> sound/soc/codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
>> sound/soc/codecs/wm9712.c:737:1: warning: 'alias' attribute ignored [-Wattributes]
   In file included from include/linux/module.h:18:0,
                    from sound/soc/codecs/wm9712.c:16:
   include/linux/moduleparam.h:21:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
    static const char __UNIQUE_ID(name)[]       \
    ^
   include/linux/module.h:160:32: note: in expansion of macro '__MODULE_INFO'
    #define MODULE_INFO(tag, info) __MODULE_INFO(tag, tag, info)
                                   ^
   include/linux/module.h:207:42: note: in expansion of macro 'MODULE_INFO'
    #define MODULE_DESCRIPTION(_description) MODULE_INFO(description, _description)
                                             ^
>> sound/soc/codecs/wm9712.c:739:1: note: in expansion of macro 'MODULE_DESCRIPTION'
    MODULE_DESCRIPTION("ASoC WM9711/WM9712 driver");
    ^
>> sound/soc/codecs/wm9712.c:741:1: error: expected declaration or statement at end of input
    MODULE_LICENSE("GPL");
    ^
   sound/soc/codecs/wm9712.c: At top level:
   sound/soc/codecs/wm9712.c:637:12: warning: 'wm9712_soc_probe' defined but not used [-Wunused-function]
    static int wm9712_soc_probe(struct snd_soc_codec *codec)
               ^
--
   sound/soc//codecs/wm9712.c: In function 'wm9712_soc_probe':
   sound/soc//codecs/wm9712.c:675:12: error: invalid storage class for function 'wm9712_soc_remove'
    static int wm9712_soc_remove(struct snd_soc_codec *codec)
               ^
   sound/soc//codecs/wm9712.c:675:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
    static int wm9712_soc_remove(struct snd_soc_codec *codec)
    ^
   sound/soc//codecs/wm9712.c:690:2: error: initializer element is not constant
     .remove =  wm9712_soc_remove,
     ^
   sound/soc//codecs/wm9712.c:690:2: error: (near initialization for 'soc_codec_dev_wm9712.remove')
   sound/soc//codecs/wm9712.c:705:12: error: invalid storage class for function 'wm9712_probe'
    static int wm9712_probe(struct platform_device *pdev)
               ^
   sound/soc//codecs/wm9712.c:722:12: error: invalid storage class for function 'wm9712_remove'
    static int wm9712_remove(struct platform_device *pdev)
               ^
   sound/soc//codecs/wm9712.c:733:2: error: initializer element is not constant
     .probe = wm9712_probe,
     ^
   sound/soc//codecs/wm9712.c:733:2: error: (near initialization for 'wm9712_codec_driver.probe')
   sound/soc//codecs/wm9712.c:734:2: error: initializer element is not constant
     .remove = wm9712_remove,
     ^
   sound/soc//codecs/wm9712.c:734:2: error: (near initialization for 'wm9712_codec_driver.remove')
   In file included from sound/soc//codecs/wm9712.c:18:0:
   sound/soc//codecs/wm9712.c:737:24: error: invalid storage class for function 'wm9712_codec_driver_init'
    module_platform_driver(wm9712_codec_driver);
                           ^
   include/linux/device.h:1502:19: note: in definition of macro 'module_driver'
    static int __init __driver##_init(void) \
                      ^
   sound/soc//codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
   In file included from sound/soc//codecs/wm9712.c:16:0:
>> include/linux/module.h:129:42: error: invalid storage class for function '__inittest'
     static inline initcall_t __maybe_unused __inittest(void)  \
                                             ^
   include/linux/device.h:1506:1: note: in expansion of macro 'module_init'
    module_init(__driver##_init); \
    ^
   include/linux/platform_device.h:228:2: note: in expansion of macro 'module_driver'
     module_driver(__platform_driver, platform_driver_register, \
     ^
   sound/soc//codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
   sound/soc//codecs/wm9712.c:737:1: warning: 'alias' attribute ignored [-Wattributes]
   In file included from sound/soc//codecs/wm9712.c:18:0:
   sound/soc//codecs/wm9712.c:737:24: error: invalid storage class for function 'wm9712_codec_driver_exit'
    module_platform_driver(wm9712_codec_driver);
                           ^
   include/linux/device.h:1507:20: note: in definition of macro 'module_driver'
    static void __exit __driver##_exit(void) \
                       ^
   sound/soc//codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
>> include/linux/device.h:1507:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
    static void __exit __driver##_exit(void) \
    ^
   include/linux/platform_device.h:228:2: note: in expansion of macro 'module_driver'
     module_driver(__platform_driver, platform_driver_register, \
     ^
   sound/soc//codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
   In file included from sound/soc//codecs/wm9712.c:16:0:
>> include/linux/module.h:135:42: error: invalid storage class for function '__exittest'
     static inline exitcall_t __maybe_unused __exittest(void)  \
                                             ^
   include/linux/device.h:1511:1: note: in expansion of macro 'module_exit'
    module_exit(__driver##_exit);
    ^
   include/linux/platform_device.h:228:2: note: in expansion of macro 'module_driver'
     module_driver(__platform_driver, platform_driver_register, \
     ^
   sound/soc//codecs/wm9712.c:737:1: note: in expansion of macro 'module_platform_driver'
    module_platform_driver(wm9712_codec_driver);
    ^
   sound/soc//codecs/wm9712.c:737:1: warning: 'alias' attribute ignored [-Wattributes]
   In file included from include/linux/module.h:18:0,
                    from sound/soc//codecs/wm9712.c:16:
   include/linux/moduleparam.h:21:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
    static const char __UNIQUE_ID(name)[]       \
    ^
   include/linux/module.h:160:32: note: in expansion of macro '__MODULE_INFO'
    #define MODULE_INFO(tag, info) __MODULE_INFO(tag, tag, info)
                                   ^
   include/linux/module.h:207:42: note: in expansion of macro 'MODULE_INFO'
    #define MODULE_DESCRIPTION(_description) MODULE_INFO(description, _description)
                                             ^
   sound/soc//codecs/wm9712.c:739:1: note: in expansion of macro 'MODULE_DESCRIPTION'
    MODULE_DESCRIPTION("ASoC WM9711/WM9712 driver");
    ^
   sound/soc//codecs/wm9712.c:741:1: error: expected declaration or statement at end of input
    MODULE_LICENSE("GPL");
    ^
   sound/soc//codecs/wm9712.c: At top level:
   sound/soc//codecs/wm9712.c:637:12: warning: 'wm9712_soc_probe' defined but not used [-Wunused-function]
    static int wm9712_soc_probe(struct snd_soc_codec *codec)
               ^

vim +/wm9712_soc_remove +675 sound/soc/codecs/wm9712.c

10c5cf304 Richard Purdie     2006-10-06  674  
f0fba2ad1 Liam Girdwood      2010-03-17 @675  static int wm9712_soc_remove(struct snd_soc_codec *codec)
10c5cf304 Richard Purdie     2006-10-06  676  {
2ed1a8e0c Robert Jarzmik     2017-09-13  677  #ifdef CONFIG_SND_SOC_AC97_BUS
358a8bb56 Lars-Peter Clausen 2014-11-10  678  	struct wm9712_priv *wm9712 = snd_soc_codec_get_drvdata(codec);
358a8bb56 Lars-Peter Clausen 2014-11-10  679  
2ed1a8e0c Robert Jarzmik     2017-09-13  680  	if (!wm9712->mfd_pdata) {
b4f4f2eba Lars-Peter Clausen 2016-09-30  681  		snd_soc_codec_exit_regmap(codec);
358a8bb56 Lars-Peter Clausen 2014-11-10  682  		snd_soc_free_ac97_codec(wm9712->ac97);
2ed1a8e0c Robert Jarzmik     2017-09-13  683  	}
2ed1a8e0c Robert Jarzmik     2017-09-13  684  #endif
10c5cf304 Richard Purdie     2006-10-06  685  	return 0;
10c5cf304 Richard Purdie     2006-10-06  686  }
10c5cf304 Richard Purdie     2006-10-06  687  
f802d6c02 Julia Lawall       2016-08-31  688  static const struct snd_soc_codec_driver soc_codec_dev_wm9712 = {
10c5cf304 Richard Purdie     2006-10-06  689  	.probe = 	wm9712_soc_probe,
10c5cf304 Richard Purdie     2006-10-06 @690  	.remove = 	wm9712_soc_remove,
10c5cf304 Richard Purdie     2006-10-06  691  	.resume =	wm9712_soc_resume,
f0fba2ad1 Liam Girdwood      2010-03-17  692  	.set_bias_level = wm9712_set_bias_level,
ab492b86b Lars-Peter Clausen 2014-11-23  693  	.suspend_bias_off = true,
9a812c6b7 Lars-Peter Clausen 2014-10-30  694  
3123e54b3 Kuninori Morimoto  2016-08-08  695  	.component_driver = {
9a812c6b7 Lars-Peter Clausen 2014-10-30  696  		.controls		= wm9712_snd_ac97_controls,
9a812c6b7 Lars-Peter Clausen 2014-10-30  697  		.num_controls		= ARRAY_SIZE(wm9712_snd_ac97_controls),
983347785 Lu Guanqun         2011-03-30  698  		.dapm_widgets		= wm9712_dapm_widgets,
983347785 Lu Guanqun         2011-03-30  699  		.num_dapm_widgets	= ARRAY_SIZE(wm9712_dapm_widgets),
983347785 Lu Guanqun         2011-03-30  700  		.dapm_routes		= wm9712_audio_map,
983347785 Lu Guanqun         2011-03-30  701  		.num_dapm_routes	= ARRAY_SIZE(wm9712_audio_map),
3123e54b3 Kuninori Morimoto  2016-08-08  702  	},
10c5cf304 Richard Purdie     2006-10-06  703  };
f0fba2ad1 Liam Girdwood      2010-03-17  704  
7a79e94e9 Bill Pemberton     2012-12-07 @705  static int wm9712_probe(struct platform_device *pdev)
f0fba2ad1 Liam Girdwood      2010-03-17  706  {
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  707  	struct wm9712_priv *wm9712;
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  708  
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  709  	wm9712 = devm_kzalloc(&pdev->dev, sizeof(*wm9712), GFP_KERNEL);
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  710  	if (wm9712 == NULL)
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  711  		return -ENOMEM;
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  712  
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  713  	mutex_init(&wm9712->lock);
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  714  
2ed1a8e0c Robert Jarzmik     2017-09-13  715  	wm9712->mfd_pdata = dev_get_platdata(&pdev->dev);
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  716  	platform_set_drvdata(pdev, wm9712);
cf1f2ebe8 Lars-Peter Clausen 2014-11-03  717  
f0fba2ad1 Liam Girdwood      2010-03-17  718  	return snd_soc_register_codec(&pdev->dev,
f0fba2ad1 Liam Girdwood      2010-03-17  719  			&soc_codec_dev_wm9712, wm9712_dai, ARRAY_SIZE(wm9712_dai));
f0fba2ad1 Liam Girdwood      2010-03-17  720  }
f0fba2ad1 Liam Girdwood      2010-03-17  721  
7a79e94e9 Bill Pemberton     2012-12-07 @722  static int wm9712_remove(struct platform_device *pdev)
f0fba2ad1 Liam Girdwood      2010-03-17  723  {
f0fba2ad1 Liam Girdwood      2010-03-17  724  	snd_soc_unregister_codec(&pdev->dev);
f0fba2ad1 Liam Girdwood      2010-03-17  725  	return 0;
f0fba2ad1 Liam Girdwood      2010-03-17  726  }
f0fba2ad1 Liam Girdwood      2010-03-17  727  
f0fba2ad1 Liam Girdwood      2010-03-17  728  static struct platform_driver wm9712_codec_driver = {
f0fba2ad1 Liam Girdwood      2010-03-17  729  	.driver = {
f0fba2ad1 Liam Girdwood      2010-03-17  730  		.name = "wm9712-codec",
f0fba2ad1 Liam Girdwood      2010-03-17  731  	},
f0fba2ad1 Liam Girdwood      2010-03-17  732  
f0fba2ad1 Liam Girdwood      2010-03-17 @733  	.probe = wm9712_probe,
7a79e94e9 Bill Pemberton     2012-12-07 @734  	.remove = wm9712_remove,
f0fba2ad1 Liam Girdwood      2010-03-17  735  };
f0fba2ad1 Liam Girdwood      2010-03-17  736  
5bbcc3c0d Mark Brown         2011-11-23 @737  module_platform_driver(wm9712_codec_driver);
10c5cf304 Richard Purdie     2006-10-06  738  
10c5cf304 Richard Purdie     2006-10-06 @739  MODULE_DESCRIPTION("ASoC WM9711/WM9712 driver");
10c5cf304 Richard Purdie     2006-10-06  740  MODULE_AUTHOR("Liam Girdwood");
10c5cf304 Richard Purdie     2006-10-06 @741  MODULE_LICENSE("GPL");

:::::: The code at line 675 was first introduced by commit
:::::: f0fba2ad1b6b53d5360125c41953b7afcd6deff0 ASoC: multi-component - ASoC Multi-Component Support

:::::: TO: Liam Girdwood <lrg at slimlogic.co.uk>
:::::: CC: Liam Girdwood <lrg at slimlogic.co.uk>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 51614 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20170920/5f795241/attachment-0001.bin>


More information about the Alsa-devel mailing list