Herbert,

This has been fixed in below patch, but I can't find it in linux-next.

https://patchwork.kernel.org/patch/11129983/

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 <[email protected]>
> 
> 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
> 

Reply via email to