From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Fri, 20 Feb 2015 10:54:38 -0700 Subject: [U-Boot] [PATCH v3 6/7] kconfig: remove unneeded dependency on !SPL_BUILD In-Reply-To: References: <1424409901-22755-1-git-send-email-yamada.m@jp.panasonic.com> <1424409901-22755-7-git-send-email-yamada.m@jp.panasonic.com> Message-ID: <54E774DE.20405@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 02/20/2015 10:06 AM, Simon Glass wrote: > +Stephen > > Hi Masahiro, > > On 19 February 2015 at 22:25, Masahiro Yamada wrote: >> Now CONFIG_SPL_BUILD is not defined in Kconfig, so >> "!depends on SPL_BUILD" and "if !SPL_BUILD" are redundant. >> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig >> index 41f3220..700e2a8 100644 >> --- a/arch/arm/Kconfig >> +++ b/arch/arm/Kconfig >> @@ -739,9 +739,8 @@ config TEGRA >> bool "NVIDIA Tegra" >> select SUPPORT_SPL >> select SPL >> - select OF_CONTROL if !SPL_BUILD >> - select CPU_ARM720T if SPL_BUILD >> - select CPU_V7 if !SPL_BUILD >> + select OF_CONTROL >> + select CPU_V7 > > Sorry if I have missed something here. On Tegra most unfortunately the > SPL uses ARMv4t and U-Boot proper uses ARMv7. In fact that is the only > reason that Tegra has SPL. Doesn't this change with this commit? Yes, on Tegra SPL runs on an ARMv4t and main U-Boot on some ARMv7 CPU. The change above is actively incorrect. The OF_CONTROL change is probably incorrect too; we certainly don't intend to use OF_CONTROL in the SPL (there's really nothing to control in the SPL) - I just don't know if enabling that feature will cause any issue. Things to look out for would be bloat of the SPL binary so that it didn't fit into the space before the main binary's TEXT_BASE, since the two get concatenated together into a single binary that's loaded into RAM, and XIP'd.