From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: Re: [PATCH] of: Specify initrd location using 64-bit Date: Fri, 21 Jun 2013 13:12:38 -0400 Message-ID: <51C48986.6080602@ti.com> References: <1371775956-16453-1-git-send-email-santosh.shilimkar@ti.com> <51C3D8E7.5030601@synopsys.com> <51C40D78.4040603@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51C40D78.4040603@imgtec.com> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: James Hogan , Vineet Gupta Cc: linux-kernel@vger.kernel.org, Russell King , Catalin Marinas , Will Deacon , Mark Salter , Aurelien Jacquiot , Michal Simek , Ralf Baechle , Jonas Bonn , Benjamin Herrenschmidt , Paul Mackerras , x86@kernel.org, arm@kernel.org, Chris Zankel , Max Filippov , Grant Likely , Rob Herring , bigeasy@linutronix.de, robherring2@gmail.com, Nicolas Pitre , linux-arm-kernel@lists.infradead.org, linux-c6x-dev@linux-c6x.orglin List-Id: devicetree@vger.kernel.org Vineet, James, On Friday 21 June 2013 04:23 AM, James Hogan wrote: > On 21/06/13 05:39, Vineet Gupta wrote: >> Hi Santosh, >> >> On 06/21/2013 06:22 AM, Santosh Shilimkar wrote: >>> Cc: Vineet Gupta >>> Cc: Russell King >>> Cc: Catalin Marinas >>> Cc: Will Deacon >>> Cc: Mark Salter >>> Cc: Aurelien Jacquiot >>> Cc: James Hogan >>> Cc: Michal Simek >>> Cc: Ralf Baechle >>> Cc: Jonas Bonn >>> Cc: Benjamin Herrenschmidt >>> Cc: Paul Mackerras >>> Cc: x86@kernel.org >>> Cc: arm@kernel.org >>> Cc: Chris Zankel >>> Cc: Max Filippov >>> Cc: Grant Likely >>> Cc: Rob Herring >>> Cc: bigeasy@linutronix.de >>> Cc: robherring2@gmail.com >>> Cc: Nicolas Pitre >>> >>> Cc: linux-arm-kernel@lists.infradead.org >>> Cc: linux-c6x-dev@linux-c6x.org >>> Cc: linux-mips@linux-mips.org >>> Cc: linuxppc-dev@lists.ozlabs.org >>> Cc: linux-xtensa@linux-xtensa.org >>> Cc: devicetree-discuss@lists.ozlabs.org >>> >>> Signed-off-by: Santosh Shilimkar >>> --- >>> 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. > Will update that in next version based on what we will end after the dicussion. i.e u64 or phys_addr_t Thanks for comments. Regards, Santosh