lkml.org 
[lkml]   [2015]   [Oct]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v10 3/6] pci:host: Add Altera PCIe host controller driver
Hi Ley,

[auto build test WARNING on pci/next -- if it's inappropriate base, please suggest rules for selecting the more suitable base]

url: https://github.com/0day-ci/linux/commits/Ley-Foon-Tan/arm-add-msi-h-to-Kbuild/20151019-162417
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

>> drivers/pci/host/pcie-altera.c:117:6: sparse: symbol 'altera_pcie_fixups' was not declared. Should it be static?
drivers/pci/host/pcie-altera.c:496:53: sparse: incorrect type in argument 2 (incompatible argument 1 (different base types))
drivers/pci/host/pcie-altera.c:496:53: expected void ( *[usertype] handle )( ... )
drivers/pci/host/pcie-altera.c:496:53: got void ( static [toplevel] *<noident> )( ... )
drivers/pci/host/pcie-altera.c: In function 'altera_pcie_parse_dt':
drivers/pci/host/pcie-altera.c:496:46: warning: passing argument 2 of 'irq_set_chained_handler_and_data' from incompatible pointer type [-Wincompatible-pointer-types]
irq_set_chained_handler_and_data(pcie->irq, altera_pcie_isr, pcie);
^
In file included from arch/x86/include/asm/hardirq.h:5:0,
from include/linux/hardirq.h:8,
from include/linux/interrupt.h:12,
from drivers/pci/host/pcie-altera.c:17:
include/linux/irq.h:545:1: note: expected 'irq_flow_handler_t {aka void (*)(unsigned int, struct irq_desc *)}' but argument is of type 'void (*)(struct irq_desc *)'
irq_set_chained_handler_and_data(unsigned int irq, irq_flow_handler_t handle,
^

Please review and possibly fold the followup patch.

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


\
 
 \ /
  Last update: 2015-10-19 11:41    [W:0.074 / U:6.656 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site