[tip:irq/core 12/13] drivers/irqchip/irq-vic.c:295:2: error: implicit declaration of function 'set_handle_irq'; did you mean 'vic_handle_irq'?

From: kbuild test robot
Date: Mon Mar 19 2018 - 17:53:40 EST

Next message: Sasha Levin: "[PATCH AUTOSEL for 4.14 74/97] EDAC, mv64x60: Fix an error handling path"
Previous message: Sasha Levin: "[PATCH AUTOSEL for 4.14 95/97] xfs: account finobt blocks properly in perag reservation"
Next in thread: Thomas Gleixner: "Re: [tip:irq/core 12/13] drivers/irqchip/irq-vic.c:295:2: error: implicit declaration of function 'set_handle_irq'; did you mean 'vic_handle_irq'?"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Palmer,

FYI, the error/warning still remains.

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core
head: ba63053f018721a0ad60b083e21a15e324a4a8c4
commit: 886d70aac148f6015ae3d1379c81b98af3e70981 [12/13] ARM: irq: Convert to GENERIC_IRQ_MULTI_HANDLER
config: arm-netx_defconfig (attached as .config)
compiler: arm-linux-gnueabi-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 886d70aac148f6015ae3d1379c81b98af3e70981
# save the attached .config to linux build tree
make.cross ARCH=arm

All errors (new ones prefixed by >>):

drivers/irqchip/irq-vic.c: In function 'vic_register':
>> drivers/irqchip/irq-vic.c:295:2: error: implicit declaration of function 'set_handle_irq'; did you mean 'vic_handle_irq'? [-Werror=implicit-function-declaration]
set_handle_irq(vic_handle_irq);
^~~~~~~~~~~~~~
vic_handle_irq
cc1: some warnings being treated as errors

vim +295 drivers/irqchip/irq-vic.c

ce94df9c0 arch/arm/common/vic.c Linus Walleij 2012-04-20 262
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 263 /**
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 264 * vic_register() - Register a VIC.
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 265 * @base: The base address of the VIC.
e641b987c drivers/irqchip/irq-vic.c Linus Walleij 2013-11-21 266 * @parent_irq: The parent IRQ if cascaded, else 0.
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 267 * @irq: The base IRQ for the VIC.
fa943bed6 arch/arm/common/vic.c Linus Walleij 2012-04-20 268 * @valid_sources: bitmask of valid interrupts
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 269 * @resume_sources: bitmask of interrupts allowed for resume sources.
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 270 * @node: The device tree node associated with the VIC.
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 271 *
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 272 * Register the VIC with the system device tree so that it can be notified
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 273 * of suspend and resume requests and ensure that the correct actions are
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 274 * taken to re-instate the settings on resume.
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 275 *
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 276 * This also configures the IRQ domain for the VIC.
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 277 */
e641b987c drivers/irqchip/irq-vic.c Linus Walleij 2013-11-21 278 static void __init vic_register(void __iomem *base, unsigned int parent_irq,
e641b987c drivers/irqchip/irq-vic.c Linus Walleij 2013-11-21 279 unsigned int irq,
fa943bed6 arch/arm/common/vic.c Linus Walleij 2012-04-20 280 u32 valid_sources, u32 resume_sources,
fa943bed6 arch/arm/common/vic.c Linus Walleij 2012-04-20 281 struct device_node *node)
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 282 {
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 283 struct vic_device *v;
5ced33bc0 arch/arm/common/vic.c Linus Walleij 2012-12-26 284 int i;
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 285
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 286 if (vic_id >= ARRAY_SIZE(vic_devices)) {
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 287 printk(KERN_ERR "%s: too few VICs, increase CONFIG_ARM_VIC_NR\n", __func__);
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 288 return;
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 289 }
f9b28ccbc arch/arm/common/vic.c Jamie Iles 2011-09-27 290
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 291 v = &vic_devices[vic_id];
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 292 v->base = base;
ce94df9c0 arch/arm/common/vic.c Linus Walleij 2012-04-20 293 v->valid_sources = valid_sources;
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 294 v->resume_sources = resume_sources;
7fb7d8ae1 arch/arm/common/vic.c Rob Herring 2012-11-20 @295 set_handle_irq(vic_handle_irq);
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 296 vic_id++;
e641b987c drivers/irqchip/irq-vic.c Linus Walleij 2013-11-21 297
e641b987c drivers/irqchip/irq-vic.c Linus Walleij 2013-11-21 298 if (parent_irq) {
9f2135419 drivers/irqchip/irq-vic.c Thomas Gleixner 2015-06-21 299 irq_set_chained_handler_and_data(parent_irq,
9f2135419 drivers/irqchip/irq-vic.c Thomas Gleixner 2015-06-21 300 vic_handle_irq_cascaded, v);
e641b987c drivers/irqchip/irq-vic.c Linus Walleij 2013-11-21 301 }
e641b987c drivers/irqchip/irq-vic.c Linus Walleij 2013-11-21 302
07c9249f1 arch/arm/common/vic.c Linus Walleij 2012-10-16 303 v->domain = irq_domain_add_simple(node, fls(valid_sources), irq,
fa943bed6 arch/arm/common/vic.c Linus Walleij 2012-04-20 304 &vic_irqdomain_ops, v);
5ced33bc0 arch/arm/common/vic.c Linus Walleij 2012-12-26 305 /* create an IRQ mapping for each valid IRQ */
5ced33bc0 arch/arm/common/vic.c Linus Walleij 2012-12-26 306 for (i = 0; i < fls(valid_sources); i++)
5ced33bc0 arch/arm/common/vic.c Linus Walleij 2012-12-26 307 if (valid_sources & (1 << i))
5ced33bc0 arch/arm/common/vic.c Linus Walleij 2012-12-26 308 irq_create_mapping(v->domain, i);
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij 2013-11-24 309 /* If no base IRQ was passed, figure out our allocated base */
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij 2013-11-24 310 if (irq)
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij 2013-11-24 311 v->irq = irq;
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij 2013-11-24 312 else
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij 2013-11-24 313 v->irq = irq_find_mapping(v->domain, 0);
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 314 }
bb06b7371 arch/arm/common/vic.c Hartley Sweeten 2010-01-12 315

:::::: The code at line 295 was first introduced by commit
:::::: 7fb7d8ae1036f1002fe8b16c15cc26d32046df64 ARM: VIC: set handle_arch_irq in VIC initialization

:::::: TO: Rob Herring <rob.herring@xxxxxxxxxxx>
:::::: CC: Rob Herring <rob.herring@xxxxxxxxxxx>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip

Next message: Sasha Levin: "[PATCH AUTOSEL for 4.14 74/97] EDAC, mv64x60: Fix an error handling path"
Previous message: Sasha Levin: "[PATCH AUTOSEL for 4.14 95/97] xfs: account finobt blocks properly in perag reservation"
Next in thread: Thomas Gleixner: "Re: [tip:irq/core 12/13] drivers/irqchip/irq-vic.c:295:2: error: implicit declaration of function 'set_handle_irq'; did you mean 'vic_handle_irq'?"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]