On Wed, Aug 31, 2016 at 11:24 PM, Robert P. J. Day rpjday@crashcourse.ca wrote:
just tripped across this in master branch:
$ grep -rw CONFIG_RAMBOOT * doc/README.ramboot-ppc85xx: #ifdef CONFIG_RAMBOOT include/configs/at91rm9200ek.h:#ifdef CONFIG_RAMBOOT README: CONFIG_RAMBOOT and CONFIG_NFSBOOT $
i have looked no further than this, so it might be perfectly valid, but it seems odd that there are so few references to that config option -- is it possible those should be CONFIG_SYS_RAMBOOT?
The was added in commit 3a4ff8b3cd71 ("at91rm9200ek: add configure target for RAM boot"). At the time there was a at91rm9200ek_ram target that would enable this option. This appears to have survived through to at91rm9200ek_ram_defconfig which has CONFIG_SYS_EXTRA_OPTIONS="RAMBOOT" so yes CONFIG_RAMBOOT seems to be perfectly valid for this board. Some of the other Freescale/NXP boards also use RAMBOOT_PBL.
I think CONFIG_SYS_EXTRA_OPTIONS is on the list to be removed. Not sure if anyone is actually working on it.
rday
--
======================================================================== Robert P. J. Day Ottawa, Ontario, CANADA http://crashcourse.ca
Twitter: http://twitter.com/rpjday LinkedIn: http://ca.linkedin.com/in/rpjday ========================================================================
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot