[U-Boot] [PATCH 5/8] powerpc: spl: deleted unused symbol CONFIG_SPL_NAND_MINIMAL and introduced new symbol CONFIG_SPL_MINIMAL

[Scott Wood]

On 06/07/2013 09:14:56 PM, Zhang Ying-B40530 wrote:
> 
> 
> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Saturday, June 08, 2013 1:21 AM
> To: Zhang Ying-B40530
> Cc: u-boot at lists.denx.de; afleming at gmail.com; Xie Xiaobo-R63061;  
> Zhang Ying-B40530
> Subject: Re: [PATCH 5/8] powerpc: spl: deleted unused symbol  
> CONFIG_SPL_NAND_MINIMAL and introduced new symbol CONFIG_SPL_MINIMAL
> 
> On 06/07/2013 04:25:17 AM, ying.zhang at freescale.com wrote:
> > diff --git a/README b/README
> > index e30e787..fbb50fb 100644
> > --- a/README
> > +++ b/README
> > @@ -2911,6 +2911,11 @@ FIT uImage format:
> >  		CONFIG_SPL_INIT_MINIMAL
> >  		Arch init code should be built for a very small image
> >
> > +		CONFIG_SPL_MINIMAL
> > +		It is different from common SPL. if set, the SPL image
> > +		as small as possible, only a tiny part of the SPL code
> > +		is built.
> 
> How is this different from CONFIG_SPL_INIT_MINIMAL?
> [Zhang Ying]
> 
> It is really some confusion. So, we have two options:
> 
> 1.  CONFIG_SPL_INIT_MINIMAL coexist with CONFIG_SPL_MINIMAL, I can  
> increase the description for the difference
> between both:
> CONFIG_SPL_INIT_MINIMAL expressed support for minimal SPL. But it  
> cannot be used independently, it must be
> applied with CONFIG_SPL_BUILD. It mainly used in Makefiles.
> 
> CONFIG_SPL_MINIMAL equals to CONFIG_SPL_BUILD &&  
> CONFIG_SPL_INIT_MINIMAL. It is applied to C files.

For now just use

#if !defined(CONFIG_SPL_INIT_MINIMAL) || !defined(CONFIG_SPL_BUILD)

...and once the patch for the separate SPL autoconf.mk gets merged, we  
can change all the SPL symbols to be only defined with CONFIG_SPL_BUILD  
(unless there's a specific reason not to for a particular symbol), and  
clean up the ifdefs.

-Scott