[alsa-devel] [asoc:for-4.21 60/62] sound/soc/amd/raven/pci-acp3x.c:59:8: error: implicit declaration of function 'pci_enable_msi'
kbuild test robot
lkp at intel.com
Wed Nov 14 04:31:07 CET 2018
tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-4.21
head: 9d4b17efa51f3bb0918021090c09b0013039df00
commit: 818ba93b0e9ed4a08cba575dd49c1391ef4b7776 [60/62] ASoC: amd: enable acp3x drivers build
config: openrisc-allmodconfig (attached as .config)
compiler: or1k-linux-gcc (GCC) 6.0.0 20160327 (experimental)
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout 818ba93b0e9ed4a08cba575dd49c1391ef4b7776
# save the attached .config to linux build tree
make.cross ARCH=openrisc
All errors (new ones prefixed by >>):
sound/soc/amd/raven/pci-acp3x.c: In function 'snd_acp3x_probe':
>> sound/soc/amd/raven/pci-acp3x.c:59:8: error: implicit declaration of function 'pci_enable_msi' [-Werror=implicit-function-declaration]
ret = pci_enable_msi(pci);
^~~~~~~~~~~~~~
>> sound/soc/amd/raven/pci-acp3x.c:124:2: error: implicit declaration of function 'pci_disable_msi' [-Werror=implicit-function-declaration]
pci_disable_msi(pci);
^~~~~~~~~~~~~~~
sound/soc/amd/raven/pci-acp3x.c: At top level:
sound/soc/amd/raven/pci-acp3x.c:161:1: warning: data definition has no type or storage class
module_pci_driver(acp3x_driver);
^~~~~~~~~~~~~~~~~
sound/soc/amd/raven/pci-acp3x.c:161:1: error: type defaults to 'int' in declaration of 'module_pci_driver' [-Werror=implicit-int]
sound/soc/amd/raven/pci-acp3x.c:161:1: warning: parameter names (without types) in function declaration
sound/soc/amd/raven/pci-acp3x.c:154:26: warning: 'acp3x_driver' defined but not used [-Wunused-variable]
static struct pci_driver acp3x_driver = {
^~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/pci_enable_msi +59 sound/soc/amd/raven/pci-acp3x.c
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 30
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 31 static int snd_acp3x_probe(struct pci_dev *pci,
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 32 const struct pci_device_id *pci_id)
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 33 {
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 34 int ret;
7894a7e7 Vijendar Mukunda 2018-11-12 35 u32 addr, val;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 36 struct acp3x_dev_data *adata;
7894a7e7 Vijendar Mukunda 2018-11-12 37 struct platform_device_info pdevinfo;
7894a7e7 Vijendar Mukunda 2018-11-12 38 unsigned int irqflags;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 39
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 40 if (pci_enable_device(pci)) {
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 41 dev_err(&pci->dev, "pci_enable_device failed\n");
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 42 return -ENODEV;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 43 }
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 44
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 45 ret = pci_request_regions(pci, "AMD ACP3x audio");
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 46 if (ret < 0) {
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 47 dev_err(&pci->dev, "pci_request_regions failed\n");
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 48 goto disable_pci;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 49 }
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 50
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 51 adata = devm_kzalloc(&pci->dev, sizeof(struct acp3x_dev_data),
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 52 GFP_KERNEL);
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 53 if (!adata) {
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 54 ret = -ENOMEM;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 55 goto release_regions;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 56 }
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 57
7894a7e7 Vijendar Mukunda 2018-11-12 58 /* check for msi interrupt support */
7894a7e7 Vijendar Mukunda 2018-11-12 @59 ret = pci_enable_msi(pci);
7894a7e7 Vijendar Mukunda 2018-11-12 60 if (ret)
7894a7e7 Vijendar Mukunda 2018-11-12 61 /* msi is not enabled */
7894a7e7 Vijendar Mukunda 2018-11-12 62 irqflags = IRQF_SHARED;
7894a7e7 Vijendar Mukunda 2018-11-12 63 else
7894a7e7 Vijendar Mukunda 2018-11-12 64 /* msi is enabled */
7894a7e7 Vijendar Mukunda 2018-11-12 65 irqflags = 0;
7894a7e7 Vijendar Mukunda 2018-11-12 66
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 67 addr = pci_resource_start(pci, 0);
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 68 adata->acp3x_base = ioremap(addr, pci_resource_len(pci, 0));
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 69 if (!adata->acp3x_base) {
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 70 ret = -ENOMEM;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 71 goto release_regions;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 72 }
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 73 pci_set_master(pci);
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 74 pci_set_drvdata(pci, adata);
7894a7e7 Vijendar Mukunda 2018-11-12 75
7894a7e7 Vijendar Mukunda 2018-11-12 76 val = rv_readl(adata->acp3x_base + mmACP_I2S_PIN_CONFIG);
7894a7e7 Vijendar Mukunda 2018-11-12 77 switch (val) {
7894a7e7 Vijendar Mukunda 2018-11-12 78 case I2S_MODE:
7894a7e7 Vijendar Mukunda 2018-11-12 79 adata->res = devm_kzalloc(&pci->dev,
7894a7e7 Vijendar Mukunda 2018-11-12 80 sizeof(struct resource) * 2,
7894a7e7 Vijendar Mukunda 2018-11-12 81 GFP_KERNEL);
7894a7e7 Vijendar Mukunda 2018-11-12 82 if (!adata->res) {
7894a7e7 Vijendar Mukunda 2018-11-12 83 ret = -ENOMEM;
7894a7e7 Vijendar Mukunda 2018-11-12 84 goto unmap_mmio;
7894a7e7 Vijendar Mukunda 2018-11-12 85 }
7894a7e7 Vijendar Mukunda 2018-11-12 86
7894a7e7 Vijendar Mukunda 2018-11-12 87 adata->res[0].name = "acp3x_i2s_iomem";
7894a7e7 Vijendar Mukunda 2018-11-12 88 adata->res[0].flags = IORESOURCE_MEM;
7894a7e7 Vijendar Mukunda 2018-11-12 89 adata->res[0].start = addr;
7894a7e7 Vijendar Mukunda 2018-11-12 90 adata->res[0].end = addr + (ACP3x_REG_END - ACP3x_REG_START);
7894a7e7 Vijendar Mukunda 2018-11-12 91
7894a7e7 Vijendar Mukunda 2018-11-12 92 adata->res[1].name = "acp3x_i2s_irq";
7894a7e7 Vijendar Mukunda 2018-11-12 93 adata->res[1].flags = IORESOURCE_IRQ;
7894a7e7 Vijendar Mukunda 2018-11-12 94 adata->res[1].start = pci->irq;
7894a7e7 Vijendar Mukunda 2018-11-12 95 adata->res[1].end = pci->irq;
7894a7e7 Vijendar Mukunda 2018-11-12 96
7894a7e7 Vijendar Mukunda 2018-11-12 97 adata->acp3x_audio_mode = ACP3x_I2S_MODE;
7894a7e7 Vijendar Mukunda 2018-11-12 98
7894a7e7 Vijendar Mukunda 2018-11-12 99 memset(&pdevinfo, 0, sizeof(pdevinfo));
7894a7e7 Vijendar Mukunda 2018-11-12 100 pdevinfo.name = "acp3x_rv_i2s";
7894a7e7 Vijendar Mukunda 2018-11-12 101 pdevinfo.id = 0;
7894a7e7 Vijendar Mukunda 2018-11-12 102 pdevinfo.parent = &pci->dev;
7894a7e7 Vijendar Mukunda 2018-11-12 103 pdevinfo.num_res = 2;
7894a7e7 Vijendar Mukunda 2018-11-12 104 pdevinfo.res = adata->res;
7894a7e7 Vijendar Mukunda 2018-11-12 105 pdevinfo.data = &irqflags;
7894a7e7 Vijendar Mukunda 2018-11-12 106 pdevinfo.size_data = sizeof(irqflags);
7894a7e7 Vijendar Mukunda 2018-11-12 107
7894a7e7 Vijendar Mukunda 2018-11-12 108 adata->pdev = platform_device_register_full(&pdevinfo);
7894a7e7 Vijendar Mukunda 2018-11-12 109 if (!adata->pdev) {
7894a7e7 Vijendar Mukunda 2018-11-12 110 dev_err(&pci->dev, "cannot register %s device\n",
7894a7e7 Vijendar Mukunda 2018-11-12 111 pdevinfo.name);
7894a7e7 Vijendar Mukunda 2018-11-12 112 ret = -ENODEV;
7894a7e7 Vijendar Mukunda 2018-11-12 113 goto unmap_mmio;
7894a7e7 Vijendar Mukunda 2018-11-12 114 }
7894a7e7 Vijendar Mukunda 2018-11-12 115 break;
7894a7e7 Vijendar Mukunda 2018-11-12 116 default:
7894a7e7 Vijendar Mukunda 2018-11-12 117 dev_err(&pci->dev, "Inavlid ACP audio mode : %d\n", val);
7894a7e7 Vijendar Mukunda 2018-11-12 118 ret = -ENODEV;
7894a7e7 Vijendar Mukunda 2018-11-12 119 goto unmap_mmio;
7894a7e7 Vijendar Mukunda 2018-11-12 120 }
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 121 return 0;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 122
7894a7e7 Vijendar Mukunda 2018-11-12 123 unmap_mmio:
7894a7e7 Vijendar Mukunda 2018-11-12 @124 pci_disable_msi(pci);
7894a7e7 Vijendar Mukunda 2018-11-12 125 iounmap(adata->acp3x_base);
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 126 release_regions:
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 127 pci_release_regions(pci);
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 128 disable_pci:
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 129 pci_disable_device(pci);
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 130
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 131 return ret;
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 132 }
e30d9128 Maruthi Srinivas Bayyavarapu 2018-11-12 133
:::::: The code at line 59 was first introduced by commit
:::::: 7894a7e7ea3de6c1fd5b2b43f3a73106c94ef584 ASoC: amd: create ACP3x PCM platform device
:::::: TO: Vijendar Mukunda <Vijendar.Mukunda at amd.com>
:::::: CC: Mark Brown <broonie at kernel.org>
---
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: 47016 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20181114/5b76ddd6/attachment-0001.bin>
More information about the Alsa-devel
mailing list