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

Sergei Shtylyov sshtylyov at mvista.com
Sat Aug 21 09:12:51 EDT 2010


Hello.

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...

WBR, Sergei



More information about the linux-arm-kernel mailing list