[radeon-alex:amd-staging-drm-next 374/955] sound/soc/amd/raven/acp3x-pcm-dma.c:638:22: error: implicit declaration of function 'devm_ioremap'; did you mean 'of_ioremap'?

kbuild test robot fengguang.wu at intel.com
Thu Jan 11 05:48:06 UTC 2018


Hi Maruthi,

FYI, the error/warning still remains.

tree:   git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head:   22531a0d76e105084ebb3b076c93e2923254c9c8
commit: c1888183e1764d55d51ae051bd8651e634febe4d [374/955] ASoC: AMD: enable ACP3x drivers build
config: sparc-allyesconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.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 c1888183e1764d55d51ae051bd8651e634febe4d
        # save the attached .config to linux build tree
        make.cross ARCH=sparc 

All errors (new ones prefixed by >>):

   In file included from sound/soc/amd/raven/acp3x-pcm-dma.c:26:0:
   sound/soc/amd/raven/acp3x.h: In function 'rv_readl':
   sound/soc/amd/raven/acp3x.h:28:9: error: implicit declaration of function 'readl'; did you mean 'vread'? [-Werror=implicit-function-declaration]
     return readl(base_addr - ACP3x_PHY_BASE_ADDRESS);
            ^~~~~
            vread
   sound/soc/amd/raven/acp3x.h: In function 'rv_writel':
   sound/soc/amd/raven/acp3x.h:33:2: error: implicit declaration of function 'writel'; did you mean 'vwrite'? [-Werror=implicit-function-declaration]
     writel(val, base_addr - ACP3x_PHY_BASE_ADDRESS);
     ^~~~~~
     vwrite
   sound/soc/amd/raven/acp3x-pcm-dma.c: In function 'acp3x_audio_probe':
>> sound/soc/amd/raven/acp3x-pcm-dma.c:638:22: error: implicit declaration of function 'devm_ioremap'; did you mean 'of_ioremap'? [-Werror=implicit-function-declaration]
     adata->acp3x_base = devm_ioremap(&pdev->dev, res->start,
                         ^~~~~~~~~~~~
                         of_ioremap
   sound/soc/amd/raven/acp3x-pcm-dma.c:638:20: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
     adata->acp3x_base = devm_ioremap(&pdev->dev, res->start,
                       ^
   cc1: some warnings being treated as errors

vim +638 sound/soc/amd/raven/acp3x-pcm-dma.c

f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  616  
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  617  static int acp3x_audio_probe(struct platform_device *pdev)
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  618  {
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  619  	int status;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  620  	struct resource *res;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  621  	struct i2s_dev_data *adata;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  622  	unsigned int irqflags;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  623  
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  624  	if (pdev->dev.platform_data == NULL) {
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  625  		dev_err(&pdev->dev, "platform_data not retrieved\n");
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  626  		return -ENODEV;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  627  	}
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  628  	irqflags = *((unsigned int *)(pdev->dev.platform_data));
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  629  
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  630  	adata = devm_kzalloc(&pdev->dev, sizeof(struct i2s_dev_data),
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  631  				GFP_KERNEL);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  632  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  633  	if (!res) {
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  634  		dev_err(&pdev->dev, "IORESOURCE_IRQ FAILED\n");
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  635  			return -ENODEV;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  636  	}
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  637  
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29 @638  	adata->acp3x_base = devm_ioremap(&pdev->dev, res->start,
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  639  			resource_size(res));
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  640  
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  641  	res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  642  	if (!res) {
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  643  		dev_err(&pdev->dev, "IORESOURCE_IRQ FAILED\n");
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  644  		return -ENODEV;
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  645  	}
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  646  
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  647  	adata->i2s_irq = res->start;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  648  	adata->play_stream = NULL;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  649  	adata->capture_stream = NULL;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  650  
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  651  	dev_set_drvdata(&pdev->dev, adata);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  652  	/* Initialize ACP */
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  653  	status = acp3x_init(adata->acp3x_base);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  654  	if (status)
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  655  		return -ENODEV;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  656  
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  657  	status = snd_soc_register_platform(&pdev->dev, &acp3x_asoc_platform);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  658  	if (status != 0) {
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  659  		dev_err(&pdev->dev, "Fail to register ALSA platform device\n");
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  660  		goto dev_err;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  661  	}
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  662  
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  663  	status = devm_snd_soc_register_component(&pdev->dev,
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  664  			&acp3x_i2s_component, &acp3x_i2s_dai_driver, 1);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  665  	if (status != 0) {
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  666  		dev_err(&pdev->dev, "Fail to register acp i2s dai\n");
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  667  		snd_soc_unregister_platform(&pdev->dev);
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  668  		goto dev_err;
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  669  	}
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  670  
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  671  	status = devm_request_irq(&pdev->dev, adata->i2s_irq, i2s_irq_handler,
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  672  					irqflags, "ACP3x_I2S_IRQ", adata);
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  673  	if (status) {
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  674  		dev_err(&pdev->dev, "ACP3x I2S IRQ request failed\n");
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  675  		snd_soc_unregister_platform(&pdev->dev);
31058cda Maruthi Srinivas Bayyavarapu 2017-03-30  676  		snd_soc_unregister_component(&pdev->dev);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  677  		goto dev_err;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  678  	}
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  679  
d8cdff57 Maruthi Srinivas Bayyavarapu 2017-03-30  680  	pm_runtime_set_autosuspend_delay(&pdev->dev, 10000);
d8cdff57 Maruthi Srinivas Bayyavarapu 2017-03-30  681  	pm_runtime_use_autosuspend(&pdev->dev);
d8cdff57 Maruthi Srinivas Bayyavarapu 2017-03-30  682  	pm_runtime_enable(&pdev->dev);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  683  	return 0;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  684  dev_err:
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  685  	status = acp3x_deinit(adata->acp3x_base);
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  686  	if (status)
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  687  		dev_err(&pdev->dev, "ACP de-init failed\n");
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  688  	else
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  689  		dev_info(&pdev->dev, "ACP de-initialized\n");
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  690  	/*ignore device status and return driver probe error*/
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  691  	return -ENODEV;
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  692  }
f1e465ee Maruthi Srinivas Bayyavarapu 2017-03-29  693  

:::::: The code at line 638 was first introduced by commit
:::::: f1e465ee291eed1843c209c6cf2ad5b482b2b703 ASoC: AMD: add ACP3x PCM platform driver

:::::: TO: Maruthi Srinivas Bayyavarapu <Maruthi.Bayyavarapu at amd.com>
:::::: CC: Alex Deucher <alexander.deucher at amd.com>

---
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: 53051 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20180111/35629399/attachment-0001.gz>


More information about the dri-devel mailing list