Re: [PATCH -next] k8.h: add struct bootnode

From: David Rientjes
Date: Thu Oct 29 2009 - 05:31:55 EST


On Wed, 28 Oct 2009, Randy Dunlap wrote:

> From: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
>
> k8.h uses struct bootnode but does not #include a header file
> for it, so provide a simple declaration for it.
>
> arch/x86/include/asm/k8.h:13: warning: 'struct bootnode' declared inside parameter list
> arch/x86/include/asm/k8.h:13: warning: its scope is only this definition or declaration, which is probably not what you want
>
> Signed-off-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>

Acked-by: David Rientjes <rientjes@xxxxxxxxxx>

This is probably my fault originating from my patchset in Ingo's x86/mm
tree with some unexpected config option combination. Thanks Randy!

> ---
> arch/x86/include/asm/k8.h | 1 +
> 1 file changed, 1 insertion(+)
>
> --- linux-next-20091028.orig/arch/x86/include/asm/k8.h
> +++ linux-next-20091028/arch/x86/include/asm/k8.h
> @@ -4,6 +4,7 @@
> #include <linux/pci.h>
>
> extern struct pci_device_id k8_nb_ids[];
> +struct bootnode;
>
> extern int early_is_k8_nb(u32 value);
> extern struct pci_dev **k8_northbridges;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/