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 [email protected], > > > >In message > ><[email protected]> you > >wrote: > >> From: Ying Zhang <[email protected]> > >> > >> 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 [email protected] http://lists.denx.de/mailman/listinfo/u-boot
