Albert (& Axel),
-----Original Message----- From: Albert ARIBAUD [mailto:albert.u.boot@aribaud.net] Sent: Saturday, May 18, 2013 11:58 AM To: Tom Warren Cc: Stephen Warren; Axel Lin; Wolfgang Denk; Heiko Schocher; u- boot@lists.denx.de Subject: Re: [U-Boot] [PATCH] ARM: arm720t: Add missing CONFIG_SKIP_LOWLEVEL_INIT guard for cpu_init_crit
Hi Tom,
On Fri, 17 May 2013 15:11:26 -0700, Tom Warren TWarren@nvidia.com wrote:
Axel (& Albert)
-----Original Message----- From: Stephen Warren [mailto:swarren@wwwdotorg.org] Sent: Wednesday, May 15, 2013 9:29 PM To: Axel Lin Cc: Wolfgang Denk; Heiko Schocher; Tom Warren; u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH] ARM: arm720t: Add missing CONFIG_SKIP_LOWLEVEL_INIT guard for cpu_init_crit
On 05/15/2013 07:00 PM, Axel Lin wrote:
2013/5/15 Stephen Warren swarren@wwwdotorg.org:
On 05/14/2013 09:04 PM, Axel Lin wrote:
cpu_init_crit() can be skipped, but the code is still enabled requiring a platform to supply lowlevel_init().
diff --git a/arch/arm/cpu/arm720t/start.S b/arch/arm/cpu/arm720t/start.S
+#ifndef CONFIG_SKIP_LOWLEVEL_INIT cpu_init_crit:
#if !defined(CONFIG_TEGRA) @@ -258,6 +259,7 @@ cpu_init_crit: #endif
mov pc, lr+#endif /* CONFIG_SKIP_LOWLEVEL_INIT */
If you're going to make changes here, you should probably ensure that Tegra is setting CONFIG_SKIP_LOWLEVEL_INIT, and then remove the Tegra-specific ifdef from the body of that function. That's assuming that setting CONFIG_SKIP_LOWLEVEL_INIT doesn't have any other side-
effects.
Tegra exists in both arm720t and armv7:
./arch/arm/cpu/arm720t/tegra-common ./arch/arm/cpu/arm720t/tegra30 ./arch/arm/cpu/arm720t/tegra20 ./arch/arm/cpu/arm720t/tegra114
./arch/arm/cpu/armv7/tegra-common ./arch/arm/cpu/armv7/tegra30 ./arch/arm/cpu/armv7/tegra20 ./arch/arm/cpu/armv7/tegra114
I'm wondering if we set CONFIG_SKIP_LOWLEVEL_INIT for tegra will also impact the code path in arch/arm/cpu/armv7/start.S ?
Yes, you certainly only want to set CONFIG_SKIP_LOWLEVEL_INIT for the SPL build. There's a whole section of include/configs/tegra-common-post.h which is ifdef CONFIG_SPL_BUILD, so it might make sense to #define CONFIG_SKIP_LOWLEVEL_INIT there.
I've applied your two patches (this one and the RFT for Tegra) to u-boot-
tegra/next and I'm running a MAKEALL right now, then I'll do a test on Cardhu and Dalmore.
Both of these are assigned to me in PatchWork - Albert - do you want me
to take them both in via the Tegra tree, or do you want to take the ARM patch and I'll take the Tegra patch? Let me know.
Please take both in Tegra tree.
Will do. Axel - please resend these 2 patches w/o the RFT tag as a v2 and I'll take them in to the Tegra repo.
Tom
Axel - I'll let you know how testing goes, and then you can resubmit the RFT
patch as a real patch against Tegra.
Thanks,
Tom
nvpublic
Amicalement,
Albert.
-- nvpublic