[U-Boot] [PATCH v2] spl: Make CONFIG_SPL_BUILD contain more functionality

[Scott Wood]

On 05/17/2013 07:34:25 AM, Tom Rini wrote:
> On Fri, May 17, 2013 at 05:12:19PM +0800, ying.zhang at freescale.com  
> wrote:
> 
> > From: Ying Zhang <b40530 at freescale.com>
> >
> > There was some functionality will be used in the SPL. They
> > had been excluded by ifndef CONFIG_SPL_BUILD. Now, put it
> > into the SPL.
> >
> > Signed-off-by: Ying Zhang <b40530 at freescale.com>
> > ---
> > Compared with the previous version, give up new symbol and delete  
> the line
> > ifndef CONFIG_SPL_BUILD in common/env_common.c
> 
> What the heck is going on?  First, you seem to be changing a number of
> checks from !CONFIG_SPL_BUILD to !CONFIG_SPL_NAND_MINIMAL, and then  
> stop
> defining CONFIG_SPL_NAND_MINIMAL always and only define it for
> CONFIG_SPL_BUILD.  Next, powerpc uses
> -ffunction-sections/-fdata-sections/--gc-sections so outside of
> assembler files, we shouldn't need to be using CONFIG_SPL_BUILD to not
> build something that's a static function.

gc-sections does not work on anonymous strings.  It also doesn't work  
to avoid code that doesn't compile due to some missing dependency (e.g.  
missing #define).  I don't know off the top of my head if this code  
falls into one of these categories (I don't even know what code we're  
talking about from the context here, without digging back for the  
original patch).

I'm not sure why being a static function matters.

-Scott