-----Original Message----- From: Wood Scott-B07421 Sent: Saturday, June 08, 2013 1:21 AM To: Zhang Ying-B40530 Cc: u-boot@lists.denx.de; aflem...@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.zh...@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. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
