Re: drivers/crypto/inside-secure/safexcel.c:840:9: error: implicit declaration of function 'pci_irq_vector'; did you mean 'rcu_irq_enter'?

[Herbert Xu <herbert@gondor.apana.org.au>]

On Fri, Sep 20, 2019 at 01:45:21PM +0800, Yuehaibing wrote:
> Herbert,
> 
> This has been fixed in below patch, but I can't find it in linux-next.
> 
> https://patchwork.kernel.org/patch/11129983/

I'm waiting for Bjorn to ack it.  Bjorn?

> On 2019/9/20 9:03, kbuild test robot wrote:
> > Hi Pascal,
> > 
> > FYI, the error/warning still remains.
> > 
> > tree:   https://kernel.googlesource.com/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head:   574cc4539762561d96b456dbc0544d8898bd4c6e
> > commit: 625f269a5a7a3643771320387e474bd0a61d9654 crypto: inside-secure - add support for PCI based FPGA development board
> > date:   3 weeks ago
> > config: sh-allmodconfig (attached as .config)
> > compiler: sh4-linux-gcc (GCC) 7.4.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 625f269a5a7a3643771320387e474bd0a61d9654
> >         # save the attached .config to linux build tree
> >         GCC_VERSION=7.4.0 make.cross ARCH=sh 
> > 
> > If you fix the issue, kindly add following tag
> > Reported-by: kbuild test robot <lkp@intel.com>
> > 
> > All errors (new ones prefixed by >>):
> > 
> >    drivers/crypto/inside-secure/safexcel.c: In function 'safexcel_request_ring_irq':
> >>> drivers/crypto/inside-secure/safexcel.c:840:9: error: implicit declaration of function 'pci_irq_vector'; did you mean 'rcu_irq_enter'? [-Werror=implicit-function-declaration]
> >       irq = pci_irq_vector(pci_pdev, irqid);
> >             ^~~~~~~~~~~~~~
> >             rcu_irq_enter
> >    drivers/crypto/inside-secure/safexcel.c: In function 'safexcel_probe_generic':
> >>> drivers/crypto/inside-secure/safexcel.c:1043:9: error: implicit declaration of function 'pci_alloc_irq_vectors'; did you mean 'pci_alloc_consistent'? [-Werror=implicit-function-declaration]
> >       ret = pci_alloc_irq_vectors(pci_pdev,
> >             ^~~~~~~~~~~~~~~~~~~~~
> >             pci_alloc_consistent
> >>> drivers/crypto/inside-secure/safexcel.c:1046:10: error: 'PCI_IRQ_MSI' undeclared (first use in this function); did you mean 'IRQ_MSK'?
> >              PCI_IRQ_MSI | PCI_IRQ_MSIX);
> >              ^~~~~~~~~~~
> >              IRQ_MSK
> >    drivers/crypto/inside-secure/safexcel.c:1046:10: note: each undeclared identifier is reported only once for each function it appears in
> >>> drivers/crypto/inside-secure/safexcel.c:1046:24: error: 'PCI_IRQ_MSIX' undeclared (first use in this function); did you mean 'PCI_IRQ_MSI'?
> >              PCI_IRQ_MSI | PCI_IRQ_MSIX);
> >                            ^~~~~~~~~~~~
> >                            PCI_IRQ_MSI
> >    drivers/crypto/inside-secure/safexcel.c: In function 'safexcel_init':
> >    drivers/crypto/inside-secure/safexcel.c:1402:6: warning: unused variable 'rc' [-Wunused-variable]
> >      int rc;
> >          ^~
> >    cc1: some warnings being treated as errors
> > 
> > vim +840 drivers/crypto/inside-secure/safexcel.c
> > 
> >    826	
> >    827	static int safexcel_request_ring_irq(void *pdev, int irqid,
> >    828					     int is_pci_dev,
> >    829					     irq_handler_t handler,
> >    830					     irq_handler_t threaded_handler,
> >    831					     struct safexcel_ring_irq_data *ring_irq_priv)
> >    832	{
> >    833		int ret, irq;
> >    834		struct device *dev;
> >    835	
> >    836		if (IS_ENABLED(CONFIG_PCI) && is_pci_dev) {
> >    837			struct pci_dev *pci_pdev = pdev;
> >    838	
> >    839			dev = &pci_pdev->dev;
> >  > 840			irq = pci_irq_vector(pci_pdev, irqid);
> >    841			if (irq < 0) {
> >    842				dev_err(dev, "unable to get device MSI IRQ %d (err %d)\n",
> >    843					irqid, irq);
> >    844				return irq;
> >    845			}
> >    846		} else if (IS_ENABLED(CONFIG_OF)) {
> >    847			struct platform_device *plf_pdev = pdev;
> >    848			char irq_name[6] = {0}; /* "ringX\0" */
> >    849	
> >    850			snprintf(irq_name, 6, "ring%d", irqid);
> >    851			dev = &plf_pdev->dev;
> >    852			irq = platform_get_irq_byname(plf_pdev, irq_name);
> >    853	
> >    854			if (irq < 0) {
> >    855				dev_err(dev, "unable to get IRQ '%s' (err %d)\n",
> >    856					irq_name, irq);
> >    857				return irq;
> >    858			}
> >    859		}
> >    860	
> >    861		ret = devm_request_threaded_irq(dev, irq, handler,
> >    862						threaded_handler, IRQF_ONESHOT,
> >    863						dev_name(dev), ring_irq_priv);
> >    864		if (ret) {
> >    865			dev_err(dev, "unable to request IRQ %d\n", irq);
> >    866			return ret;
> >    867		}
> >    868	
> >    869		return irq;
> >    870	}
> >    871	
> > 
> > ---
> > 0-DAY kernel test infrastructure                Open Source Technology Center
> > https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
> > 

-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt