[PATCH] ARM: SAMSUNG: Fix on build warning regarding VMALLOC_END type

Kukjin Kim kgene.kim at samsung.com
Sun Aug 22 20:44:27 EDT 2010


Sergei Shtylyov wrote:
> 
> Hello.
> 
Hi ;-)

> Kukjin Kim wrote:
> 
> > Fix this warning:
> 
> > arch/arm/mm/init.c: In function 'mem_init':
> > arch/arm/mm/init.c:644: warning: format '%08lx' expects type
> > 'long unsigned int', but argument 12 has type 'unsigned int'
> 
> > Reported-by: Kyungmin Park <kyungmin.park at samsung.com>
> > Signed-off-by: Kukjin Kim <kgene.kim at samsung.com>
> > Cc: Ben Dooks <ben-linux at fluff.org>
> 
> [...]
> 
> > diff --git a/arch/arm/mach-s3c2410/include/mach/vmalloc.h
b/arch/arm/mach-
> s3c2410/include/mach/vmalloc.h
> > index 315b007..dc3b34c 100644
> > --- a/arch/arm/mach-s3c2410/include/mach/vmalloc.h
> > +++ b/arch/arm/mach-s3c2410/include/mach/vmalloc.h
> > @@ -15,6 +15,6 @@
> >  #ifndef __ASM_ARCH_VMALLOC_H
> >  #define __ASM_ARCH_VMALLOC_H
> >
> > -#define VMALLOC_END	  (0xE0000000)
> > +#define VMALLOC_END	(0xE0000000UL)
> >
> >  #endif /* __ASM_ARCH_VMALLOC_H */
> > diff --git a/arch/arm/mach-s3c64xx/include/mach/vmalloc.h
b/arch/arm/mach-
> s3c64xx/include/mach/vmalloc.h
> > index 7411ef3..f0a0fa2 100644
> > --- a/arch/arm/mach-s3c64xx/include/mach/vmalloc.h
> > +++ b/arch/arm/mach-s3c64xx/include/mach/vmalloc.h
> > @@ -15,6 +15,6 @@
> >  #ifndef __ASM_ARCH_VMALLOC_H
> >  #define __ASM_ARCH_VMALLOC_H
> >
> > -#define VMALLOC_END	  (0xE0000000)
> > +#define VMALLOC_END	(0xE0000000UL)
> >
> >  #endif /* __ASM_ARCH_VMALLOC_H */
> > diff --git a/arch/arm/mach-s5p6440/include/mach/vmalloc.h
b/arch/arm/mach-
> s5p6440/include/mach/vmalloc.h
> > index 16df257..8217b99 100644
> > --- a/arch/arm/mach-s5p6440/include/mach/vmalloc.h
> > +++ b/arch/arm/mach-s5p6440/include/mach/vmalloc.h
> > @@ -12,6 +12,6 @@
> >  #ifndef __ASM_ARCH_VMALLOC_H
> >  #define __ASM_ARCH_VMALLOC_H
> >
> > -#define VMALLOC_END	  (0xE0000000)
> > +#define VMALLOC_END	(0xE0000000UL)
> >
> >  #endif /* __ASM_ARCH_VMALLOC_H */
> > diff --git a/arch/arm/mach-s5p6442/include/mach/vmalloc.h
b/arch/arm/mach-
> s5p6442/include/mach/vmalloc.h
> > index be33336..b9fd9aa 100644
> > --- a/arch/arm/mach-s5p6442/include/mach/vmalloc.h
> > +++ b/arch/arm/mach-s5p6442/include/mach/vmalloc.h
> > @@ -12,6 +12,6 @@
> >  #ifndef __ASM_ARCH_VMALLOC_H
> >  #define __ASM_ARCH_VMALLOC_H
> >
> > -#define VMALLOC_END	  (0xE0000000)
> > +#define VMALLOC_END	(0xE0000000UL)
> >
> >  #endif /* __ASM_ARCH_VMALLOC_H */
> 
>     Could eliminate the useless parens while at it...
> 
Sure, it's no problem.
Will re-send soon with fix it.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




More information about the linux-arm-kernel mailing list