________________________________________ From: Wood Scott-B07421 Sent: Monday, June 10, 2013 6:32 PM To: Zhang Ying-B40530 Cc: Wood Scott-B07421; u-boot@lists.denx.de; aflem...@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@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. 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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
