ARM: fix compile warning in mm/init.c
Kyungmin Park
kmpark at infradead.org
Fri Nov 12 03:22:04 EST 2010
2010/11/12 Uwe Kleine-König <u.kleine-koenig at pengutronix.de>:
> On Fri, Nov 12, 2010 at 11:49:13AM +0530, Anand Gadiyar wrote:
>> Commit 7c63984b86 (ARM: do not define VMALLOC_END relative to PAGE_OFFSET)
>> changed VMALLOC_END to be an explicit value. Before this, it was
>> relative to PAGE_OFFSET and therefore automatically upgraded to
>> unsigned long as PAGE_OFFSET was an unsigned long. This introduced
>> the following build warning. Fix this.
>>
>> CC arch/arm/mm/init.o
>> arch/arm/mm/init.c: In function 'mem_init':
>> arch/arm/mm/init.c:606: warning: format '%08lx' expects type 'long unsigned int', but argument 12 has type 'unsigned int'
>>
>> Signed-off-by: Anand Gadiyar <gadiyar at ti.com>
>> Cc: Nicolas Pitre <nico at fluxnic.net>
>> ---
>> arch/arm/mm/init.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> Index: ehci/arch/arm/mm/init.c
>> ===================================================================
>> --- ehci.orig/arch/arm/mm/init.c
>> +++ ehci/arch/arm/mm/init.c
>> @@ -593,7 +593,7 @@ void __init mem_init(void)
>> #ifdef CONFIG_MMU
>> MLM(CONSISTENT_BASE, CONSISTENT_END),
>> #endif
>> - MLM(VMALLOC_START, VMALLOC_END),
>> + MLM(VMALLOC_START, (unsigned long)VMALLOC_END),
> I wonder if this is the only place where this is a problem. Maybe it's
> better to make the defined value of all VMALLOC_END an unsigned long as
> some already are?
Right, Samsung SoCs change it to UL to fix this issue.
Thank you,
Kyungmin Park
>
> Best regards
> Uwe
>
> --
> Pengutronix e.K. | Uwe Kleine-König |
> Industrial Linux Solutions | http://www.pengutronix.de/ |
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
More information about the linux-arm-kernel
mailing list