On Fri, Jan 15, 2016 at 07:18:37PM +0000, Geoff Levand wrote: > From: James Morse <james.morse at arm.com> > > From: James Morse <james.morse at arm.com> Not sure why this appears multiple times. > page.h uses '_AC' in the definition of PAGE_SIZE, but doesn't include > linux/const.h where this is defined. This produces build warnings when only > asm/page.h is included by asm code. > > Signed-off-by: James Morse <james.morse at arm.com> > Acked-by: Pavel Machek <pavel at ucw.cz> > Signed-off-by: Geoff Levand <geoff at infradead.org> This is sensible even in isolation, so FWIW: Acked-by: Mark Rutland <mark.rutland at arm.com> I note that for the !__ASSEMBLY__ portion we use current, READ_IMPLIES_EXEC, and some VM_* flags, without including the headers those are defined in. It might be worth fixing those up also. Mark. > --- > arch/arm64/include/asm/page.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/include/asm/page.h b/arch/arm64/include/asm/page.h > index 9b2f5a9..fbafd0a 100644 > --- a/arch/arm64/include/asm/page.h > +++ b/arch/arm64/include/asm/page.h > @@ -19,6 +19,8 @@ > #ifndef __ASM_PAGE_H > #define __ASM_PAGE_H > > +#include <linux/const.h> > + > /* PAGE_SHIFT determines the page size */ > /* CONT_SHIFT determines the number of pages which can be tracked together */ > #ifdef CONFIG_ARM64_64K_PAGES > -- > 2.5.0 > >
