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