On Tue, Mar 14, 2006 at 01:45:35PM -0800, Andrew Morton wrote:
> "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.
Ugh. What is the file that is causing the problem? What is "include2"
in your directory path above?
Whatever .c file has it, should just include the internal pci.h file
that already has this prototype, like Andrew says...
thanks,
greg k-h
-
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]
