On 07/16/2014 01:08 AM, Mike Qiu wrote: > build log: > When CONFIG_PCIEAER is not enabled: > In file included from include/ras/ras_event.h:11:0, > from drivers/ras/ras.c:13: > include/linux/aer.h:42:129: warning: ‘struct pci_dev’ > declared inside parameter list [enabled by default] > > include/linux/aer.h:42:129: warning: its scope is only > this definition or declaration, which is probably not > what you want [enabled by default] > > include/linux/aer.h:46:130: warning: ‘struct pci_dev’ > declared inside parameter list [enabled by default] > > include/linux/aer.h:50:136: warning: ‘struct pci_dev’ > declared inside parameter list [enabled by default] > > include/linux/aer.h:57:14: warning: ‘struct pci_dev’ > declared inside parameter list [enabled by default] > > Signed-off-by: Mike Qiu <qiudayu@xxxxxxxxxxxxxxxxxx> Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> Tested-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> > --- > include/linux/aer.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/linux/aer.h b/include/linux/aer.h > index 4dbaa70..c826d1c 100644 > --- a/include/linux/aer.h > +++ b/include/linux/aer.h > @@ -11,6 +11,8 @@ > #define AER_FATAL 1 > #define AER_CORRECTABLE 2 > > +struct pci_dev; > + > struct aer_header_log_regs { > unsigned int dw0; > unsigned int dw1; > -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html