[U-Boot] CONFIG_DELAY_ENVIRONMENT usage

Allen Martin amartin at nvidia.com
Wed Jan 30 18:37:48 CET 2013


On Tue, Jan 29, 2013 at 10:08:39PM -0800, Lucas Stach wrote:
> Hi Allen,
> 
> Am Dienstag, den 29.01.2013, 19:05 -0800 schrieb Allen Martin:
> 
> > The problem I'm seeing is that this regressed environment loading on
> > all boards that use fdt, because none of them have
> > /config/load-environment defined.  If I read the commit message
> > correctly, I think the actual intention is:
> > 
> > static int should_load_env(void)
> > {
> > #ifdef CONFIG_DELAY_ENVIRONMENT
> > #ifdef CONFIG_OF_CONTROL
> >         return fdtdec_get_config_int(gd->fdt_blob, "load-environment",
> > 	0);
> > #else
> >         return 0;
> > #endif
> > #else
> >         return 1;
> > #endif
> > }
> > 
> There is already a patch for that on the mailing list. See
> http://www.mail-archive.com/u-boot@lists.denx.de/msg103841.html
> 
> We agreed that just swapping around the default value for the FDT case
> is the right thing to do.

Thanks for the pointer Lucas, I didn't notice the previous patch.

-Allen
-- 
nvpublic


More information about the U-Boot mailing list