Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 7 Sep 2022 12:11:29 +0800
From:   kernel test robot <lkp@intel.com>
To:     Dmitry Torokhov <dmitry.torokhov@gmail.com>,
        Shawn Guo <shawn.guo@linaro.org>,
        Lorenzo Pieralisi <lpieralisi@kernel.org>,
        Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
        Pali =?iso-8859-1?Q?Roh=E1r?= <pali@kernel.org>,
        Bjorn Helgaas <helgaas@kernel.org>
Cc:     kbuild-all@lists.01.org, Rob Herring <robh@kernel.org>,
        Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= <kw@linux.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        Bartosz Golaszewski <brgl@bgdev.pl>, linux-pci@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2] PCI: mvebu: switch to using gpiod API
Message-ID: <202209071255.xlxq6qnE-lkp@intel.com>
References: <20220906204301.3736813-2-dmitry.torokhov@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20220906204301.3736813-2-dmitry.torokhov@gmail.com>
Precedence: bulk

Hi Dmitry,

I love your patch! Yet something to improve:

[auto build test ERROR on helgaas-pci/next]
[also build test ERROR on linus/master v6.0-rc4 next-20220906]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Dmitry-Torokhov/PCI-histb-switch-to-using-gpiod-API/20220907-044417
base:   https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git next
config: arm-defconfig (https://download.01.org/0day-ci/archive/20220907/202209071255.xlxq6qnE-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/90d5c7ec598d196395d3a5934099b56d1c8e071a
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Dmitry-Torokhov/PCI-histb-switch-to-using-gpiod-API/20220907-044417
        git checkout 90d5c7ec598d196395d3a5934099b56d1c8e071a
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   drivers/pci/controller/pci-mvebu.c: In function 'mvebu_pcie_irq_handler':
>> drivers/pci/controller/pci-mvebu.c:1100:9: error: implicit declaration of function 'chained_irq_enter'; did you mean 'ct_irq_enter'? [-Werror=implicit-function-declaration]
    1100 |         chained_irq_enter(chip, desc);
         |         ^~~~~~~~~~~~~~~~~
         |         ct_irq_enter
>> drivers/pci/controller/pci-mvebu.c:1115:9: error: implicit declaration of function 'chained_irq_exit'; did you mean 'ct_irq_exit'? [-Werror=implicit-function-declaration]
    1115 |         chained_irq_exit(chip, desc);
         |         ^~~~~~~~~~~~~~~~
         |         ct_irq_exit
   cc1: some warnings being treated as errors


vim +1100 drivers/pci/controller/pci-mvebu.c

ec075262648f39 Pali Roh?r 2022-02-22  1091  
ec075262648f39 Pali Roh?r 2022-02-22  1092  static void mvebu_pcie_irq_handler(struct irq_desc *desc)
ec075262648f39 Pali Roh?r 2022-02-22  1093  {
ec075262648f39 Pali Roh?r 2022-02-22  1094  	struct mvebu_pcie_port *port = irq_desc_get_handler_data(desc);
ec075262648f39 Pali Roh?r 2022-02-22  1095  	struct irq_chip *chip = irq_desc_get_chip(desc);
ec075262648f39 Pali Roh?r 2022-02-22  1096  	struct device *dev = &port->pcie->pdev->dev;
ec075262648f39 Pali Roh?r 2022-02-22  1097  	u32 cause, unmask, status;
ec075262648f39 Pali Roh?r 2022-02-22  1098  	int i;
ec075262648f39 Pali Roh?r 2022-02-22  1099  
ec075262648f39 Pali Roh?r 2022-02-22 @1100  	chained_irq_enter(chip, desc);
ec075262648f39 Pali Roh?r 2022-02-22  1101  
ec075262648f39 Pali Roh?r 2022-02-22  1102  	cause = mvebu_readl(port, PCIE_INT_CAUSE_OFF);
ec075262648f39 Pali Roh?r 2022-02-22  1103  	unmask = mvebu_readl(port, PCIE_INT_UNMASK_OFF);
ec075262648f39 Pali Roh?r 2022-02-22  1104  	status = cause & unmask;
ec075262648f39 Pali Roh?r 2022-02-22  1105  
ec075262648f39 Pali Roh?r 2022-02-22  1106  	/* Process legacy INTx interrupts */
ec075262648f39 Pali Roh?r 2022-02-22  1107  	for (i = 0; i < PCI_NUM_INTX; i++) {
ec075262648f39 Pali Roh?r 2022-02-22  1108  		if (!(status & PCIE_INT_INTX(i)))
ec075262648f39 Pali Roh?r 2022-02-22  1109  			continue;
ec075262648f39 Pali Roh?r 2022-02-22  1110  
ec075262648f39 Pali Roh?r 2022-02-22  1111  		if (generic_handle_domain_irq(port->intx_irq_domain, i) == -EINVAL)
ec075262648f39 Pali Roh?r 2022-02-22  1112  			dev_err_ratelimited(dev, "unexpected INT%c IRQ\n", (char)i+'A');
ec075262648f39 Pali Roh?r 2022-02-22  1113  	}
ec075262648f39 Pali Roh?r 2022-02-22  1114  
ec075262648f39 Pali Roh?r 2022-02-22 @1115  	chained_irq_exit(chip, desc);
ec075262648f39 Pali Roh?r 2022-02-22  1116  }
ec075262648f39 Pali Roh?r 2022-02-22  1117  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp