Re: [PATCH] of: Specify initrd location using 64-bit

From: James Hogan
Date: Fri Jun 21 2013 - 04:23:42 EST


On 21/06/13 05:39, Vineet Gupta wrote:
> Hi Santosh,
>
> On 06/21/2013 06:22 AM, Santosh Shilimkar wrote:
>> Cc: Vineet Gupta <vgupta@xxxxxxxxxxxx>
>> Cc: Russell King <linux@xxxxxxxxxxxxxxxx>
>> Cc: Catalin Marinas <catalin.marinas@xxxxxxx>
>> Cc: Will Deacon <will.deacon@xxxxxxx>
>> Cc: Mark Salter <msalter@xxxxxxxxxx>
>> Cc: Aurelien Jacquiot <a-jacquiot@xxxxxx>
>> Cc: James Hogan <james.hogan@xxxxxxxxxx>
>> Cc: Michal Simek <monstr@xxxxxxxxx>
>> Cc: Ralf Baechle <ralf@xxxxxxxxxxxxxx>
>> Cc: Jonas Bonn <jonas@xxxxxxxxxxxx>
>> Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
>> Cc: Paul Mackerras <paulus@xxxxxxxxx>
>> Cc: x86@xxxxxxxxxx
>> Cc: arm@xxxxxxxxxx
>> Cc: Chris Zankel <chris@xxxxxxxxxx>
>> Cc: Max Filippov <jcmvbkbc@xxxxxxxxx>
>> Cc: Grant Likely <grant.likely@xxxxxxxxxx>
>> Cc: Rob Herring <rob.herring@xxxxxxxxxxx>
>> Cc: bigeasy@xxxxxxxxxxxxx
>> Cc: robherring2@xxxxxxxxx
>> Cc: Nicolas Pitre <nicolas.pitre@xxxxxxxxxx>
>>
>> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
>> Cc: linux-c6x-dev@xxxxxxxxxxxxx
>> Cc: linux-mips@xxxxxxxxxxxxxx
>> Cc: linuxppc-dev@xxxxxxxxxxxxxxxx
>> Cc: linux-xtensa@xxxxxxxxxxxxxxxx
>> Cc: devicetree-discuss@xxxxxxxxxxxxxxxx
>>
>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
>> ---
>> arch/arc/mm/init.c | 3 +--
>> arch/arm/mm/init.c | 2 +-
>> arch/arm64/mm/init.c | 3 +--
>> arch/c6x/kernel/devicetree.c | 3 +--
>> arch/metag/mm/init.c | 3 +--
>> arch/microblaze/kernel/prom.c | 3 +--
>> arch/mips/kernel/prom.c | 3 +--
>> arch/openrisc/kernel/prom.c | 3 +--
>> arch/powerpc/kernel/prom.c | 3 +--
>> arch/x86/kernel/devicetree.c | 3 +--
>> arch/xtensa/kernel/setup.c | 3 +--
>> drivers/of/fdt.c | 10 ++++++----
>> include/linux/of_fdt.h | 3 +--
>> 13 files changed, 18 insertions(+), 27 deletions(-)
>>
>> diff --git a/arch/arc/mm/init.c b/arch/arc/mm/init.c
>> index 4a17736..3640c74 100644
>> --- a/arch/arc/mm/init.c
>> +++ b/arch/arc/mm/init.c
>> @@ -157,8 +157,7 @@ void __init free_initrd_mem(unsigned long start, unsigned long end)
>> #endif
>>
>> #ifdef CONFIG_OF_FLATTREE
>> -void __init early_init_dt_setup_initrd_arch(unsigned long start,
>> - unsigned long end)
>> +void __init early_init_dt_setup_initrd_arch(u64 start, u64 end)
>> {
>> pr_err("%s(%lx, %lx)\n", __func__, start, end);
>> }
>
> To avoid gcc warnings, you need to fix the print format specifiers too.

The same thing goes for arch/metag too.

Cheers
James

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/