From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Fri, 24 May 2013 12:07:53 -0400 Subject: [U-Boot] [PATCH 5/6] spl: Make CONFIG_SPL_BUILD contain more functionality In-Reply-To: <1369165322.1374.11@scott-Lenovo-G560> References: <1369030048-26130-1-git-send-email-ying.zhang@freescale.com> <1369030048-26130-5-git-send-email-ying.zhang@freescale.com> <1369165322.1374.11@scott-Lenovo-G560> Message-ID: <20130524160753.GR17119@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, May 21, 2013 at 02:42:02PM -0500, Scott Wood wrote: > Please change the title and the rest of the changelog to describe > what functionality you're adding and why. > > On 05/20/2013 01:07:27 AM, ying.zhang at freescale.com wrote: > >diff --git a/common/env_common.c b/common/env_common.c > >index 906b41f..8cb81e9 100644 > >--- a/common/env_common.c > >+++ b/common/env_common.c > >@@ -156,7 +156,6 @@ int set_default_vars(int nvars, char * const > >vars[]) > > H_NOCLEAR | H_INTERACTIVE, nvars, vars); > > } > > > >-#ifndef CONFIG_SPL_BUILD > > /* > > * Check if CRC is valid and (if yes) import the environment. > > * Note that "buf" may or may not be aligned. > >@@ -188,7 +187,6 @@ int env_import(const char *buf, int check) > > > > return 0; > > } > >-#endif > > This ifndef was introduced by Ilya Yanok in commit > 7ac2fe2da21d292aeaf3af74e5c80de9ce9dab56. > > Ilya, what are the consequences of removing this? Is there some > other symbol we can use here? This was likely done so that we could fit the combination of USB RNDIS networking in SPL into the size constraint we have and the need for more than just unused section discarding to get rid of codepaths we know won't be taken (but couldn't compile-time determine). -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: