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

[Zhang Ying-B40530]

-----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. 

Just as you said: it would be nice to limit SPL symbols to only be defined for the SPL part of the build, so
we don't have to add checks for CONFIG_SPL_BUILD all over the place.  Currently this won't work for symbols
that makefiles look at, if "SPL: Makefile: Build a separate autoconf.mk for SPL" gets merged, we could just
define CONFIG_SPL_MINIMAL in the SPL build case.

2. No longer adds the new symbol CONFIG_SPL_MINIMAL:
We can use CONFIG_SPL_BUILD && CONFIG_SPL_INIT_MINIMAL to replace it in the C files.