"Jun'ichi Nomura" <[email protected]> wrote:
>
> Declare msi_register() in msi.h.
>
> The patch is especially necessary for ia64 on which most of files
> emit compiler warnings like below:
> include2/asm/msi.h: In function `ia64_msi_init':
> include2/asm/msi.h:23: warning: implicit declaration of function `msi_register'
> In file included from include2/asm/machvec.h:408,
> from include2/asm/io.h:70,
> from include2/asm/smp.h:20,
> from /build/rc6/source/include/linux/smp.h:22,
I wonder why I didn't get that. Need a better ia64 config I guess.
> Signed-off-by: Jun'ichi Nomura <[email protected]>
>
> --- linux-2.6.16-rc6-mm1.orig/include/asm-ia64/msi.h 2006-03-14 13:54:11.000000000 -0500
> +++ linux-2.6.16-rc6-mm1/include/asm-ia64/msi.h 2006-03-14 14:05:26.000000000 -0500
> @@ -15,6 +15,7 @@ static inline void set_intr_gate (int nr
> #define MSI_TARGET_CPU_SHIFT 4
>
> extern struct msi_ops msi_apic_ops;
> +extern int msi_register(struct msi_ops *);
>
> /* default ia64 msi init routine */
> static inline int ia64_msi_init(void)
The offending patch is gregkh-pci-msi-vector-targeting-abstractions.patch.
That patch already adds a declaration for msi_register(), in
drivers/pci/pci.h. We don't want to add a duplicated declaration like
this.
One option might be to create inclued/linux/msi.h, put this declaration in
there then include <asm/msi.h>. Possibly some other declarations should be
moved into linux/msi.h as well.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
