From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Fri, 21 Jun 2013 10:23:37 +0200 (CEST) Received: from multi.imgtec.com ([194.200.65.239]:4153 "EHLO multi.imgtec.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S6816022Ab3FUIXdRFIRb (ORCPT ); Fri, 21 Jun 2013 10:23:33 +0200 Message-ID: <51C40D78.4040603@imgtec.com> Date: Fri, 21 Jun 2013 09:23:20 +0100 From: James Hogan User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 MIME-Version: 1.0 To: Santosh Shilimkar CC: Vineet Gupta , , Russell King , Catalin Marinas , Will Deacon , Mark Salter , Aurelien Jacquiot , Michal Simek , Ralf Baechle , Jonas Bonn , Benjamin Herrenschmidt , Paul Mackerras , , , Chris Zankel , Max Filippov , "Grant Likely" , Rob Herring , , , Nicolas Pitre , , , , , , Subject: Re: [PATCH] of: Specify initrd location using 64-bit References: <1371775956-16453-1-git-send-email-santosh.shilimkar@ti.com> <51C3D8E7.5030601@synopsys.com> In-Reply-To: <51C3D8E7.5030601@synopsys.com> X-Enigmail-Version: 1.5.1 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit X-Originating-IP: [192.168.154.65] X-SEF-Processed: 7_3_0_01192__2013_06_21_09_23_22 Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 37078 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: james.hogan@imgtec.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips 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. Cheers James From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from multi.imgtec.com ([194.200.65.239]:4153 "EHLO multi.imgtec.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S6816022Ab3FUIXdRFIRb (ORCPT ); Fri, 21 Jun 2013 10:23:33 +0200 Message-ID: <51C40D78.4040603@imgtec.com> Date: Fri, 21 Jun 2013 09:23:20 +0100 From: James Hogan MIME-Version: 1.0 Subject: Re: [PATCH] of: Specify initrd location using 64-bit References: <1371775956-16453-1-git-send-email-santosh.shilimkar@ti.com> <51C3D8E7.5030601@synopsys.com> In-Reply-To: <51C3D8E7.5030601@synopsys.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-Path: 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: Santosh Shilimkar Cc: Vineet Gupta , 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.org, linux-mips@linux-mips.org, linuxppc-dev@lists.ozlabs.org, linux-xtensa@linux-xtensa.org, devicetree-discuss@lists.ozlabs.org Message-ID: <20130621082320.tgjrI4h3ujtNFwoMckiLg8K8s7jtayioDZvsVe2L8ds@z> 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. Cheers James From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <51C40D78.4040603@imgtec.com> Date: Fri, 21 Jun 2013 09:23:20 +0100 From: James Hogan MIME-Version: 1.0 To: Santosh Shilimkar Subject: Re: [PATCH] of: Specify initrd location using 64-bit References: <1371775956-16453-1-git-send-email-santosh.shilimkar@ti.com> <51C3D8E7.5030601@synopsys.com> In-Reply-To: <51C3D8E7.5030601@synopsys.com> Content-Type: text/plain; charset="ISO-8859-1" Cc: Nicolas Pitre , linux-mips@linux-mips.org, Aurelien Jacquiot , Catalin Marinas , bigeasy@linutronix.de, Will Deacon , Max Filippov , Paul Mackerras , Jonas Bonn , Russell King , linux-c6x-dev@linux-c6x.org, x86@kernel.org, arm@kernel.org, robherring2@gmail.com, Grant Likely , devicetree-discuss@lists.ozlabs.org, linux-xtensa@linux-xtensa.org, Vineet Gupta , Mark Salter , Rob Herring , linux-arm-kernel@lists.infradead.org, Chris Zankel , Michal Simek , linux-kernel@vger.kernel.org, Ralf Baechle , linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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. Cheers James From mboxrd@z Thu Jan 1 00:00:00 1970 From: james.hogan@imgtec.com (James Hogan) Date: Fri, 21 Jun 2013 09:23:20 +0100 Subject: [PATCH] of: Specify initrd location using 64-bit In-Reply-To: <51C3D8E7.5030601@synopsys.com> References: <1371775956-16453-1-git-send-email-santosh.shilimkar@ti.com> <51C3D8E7.5030601@synopsys.com> Message-ID: <51C40D78.4040603@imgtec.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 at kernel.org >> Cc: arm at kernel.org >> Cc: Chris Zankel >> Cc: Max Filippov >> Cc: Grant Likely >> Cc: Rob Herring >> Cc: bigeasy at linutronix.de >> Cc: robherring2 at gmail.com >> Cc: Nicolas Pitre >> >> Cc: linux-arm-kernel at lists.infradead.org >> Cc: linux-c6x-dev at linux-c6x.org >> Cc: linux-mips at linux-mips.org >> Cc: linuxppc-dev at lists.ozlabs.org >> Cc: linux-xtensa at linux-xtensa.org >> Cc: devicetree-discuss at 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. Cheers James From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Hogan Subject: Re: [PATCH] of: Specify initrd location using 64-bit Date: Fri, 21 Jun 2013 09:23:20 +0100 Message-ID: <51C40D78.4040603@imgtec.com> References: <1371775956-16453-1-git-send-email-santosh.shilimkar@ti.com> <51C3D8E7.5030601@synopsys.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51C3D8E7.5030601@synopsys.com> Sender: linux-kernel-owner@vger.kernel.org To: Santosh Shilimkar Cc: Vineet Gupta , 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.org, linux-mips@linux-mips.org, linuxppc-dev@lists.ozlabs.org, linux-xtensa@ List-Id: devicetree@vger.kernel.org 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. Cheers James