On Fri, May 10, 2013 at 07:28:46PM -0500, Scott Wood wrote: > On 05/10/2013 07:03:07 PM, Wolfgang Denk wrote: > >Dear ying.zh...@freescale.com, > > > >In message > ><1368182421-859-1-git-send-email-ying.zh...@freescale.com> you > >wrote: > >> From: Ying Zhang <b40...@freescale.com> > >> > >> Add a new symbol CONFIG_SPL_ENV_SUPPORT to replace > >CONFIG_SPL_NET_SUPPORT. > > > >Any CONFIG_* stuff must be documented in the README. Please add this > >documentation. > > > >> In common/Makefile, the environment-related files in SPL is > >difined by > >> CONFIG_SPL_NET_SUPPORT, it is wrong. > > > >s/difined/defined/ > > > >Could you please also explain, what exactly is wrong, and how your new > >code fixes this? > > We will need the environment in SPL for reasons other than network > support (in particular, hwconfig contains info for how to set up > DDR). Yes, this information should go in the changelog.
Note that a3m071 already does SPL + Environment and should be updated, CONFIG-symbol wise here. It currently gets away with env_common/cmd_nvedit being pulled in always on LIBCOMMON and then discarded when not needed. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
