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

[Zhang Ying-B40530]

________________________________________
From: Wood Scott-B07421
Sent: Monday, June 10, 2013 6:32 PM
To: Zhang Ying-B40530
Cc: Wood Scott-B07421; u-boot at lists.denx.de; afleming at gmail.com; Xie Xiaobo-R63061
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 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.

OK. 

-ZhangYing