On Tue, Apr 30, 2013 at 11:33 AM, Nicolas Ferre <nicolas.fe...@atmel.com> wrote: > So, I am wondering if the best correction is to add the types.h header file > in the asm/mach/arch.h file, like this: > > --- a/arch/arm/include/asm/mach/arch.h > +++ b/arch/arm/include/asm/mach/arch.h > @@ -8,6 +8,8 @@ > * published by the Free Software Foundation. > */ > > +#include <linux/types.h> > + > #ifndef __ASSEMBLY__ > > struct tag; > > or if a better option would be to re-arrange the include directives in > various at91xxx_devices.c files?
Include files should be self-contained, so asm/mach/arch.h is the right place. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/