Hi Tom, On Wed, 5 Mar 2014 12:28:06 -0500, Tom Rini <tr...@ti.com> wrote:
> Hey all, > > I'm looking over some patches for am43xx to enable the case of non-SPL > XIP booting and this means we start getting code that's shared with > am335x that looks like: > #if defined(CONFIG_SPL_BUILD) || defined(CONFIG_NOR_BOOT) || \ > defined(CONFIG_QSPI_BOOT) > ... > #endif > > Because we need to do certain init either in SPL or early U-Boot. While > I don't see another XIP boot case being added anytime soon, it would be > nice to instead write: > #if defined(CONFIG_SPL_BUILD) || defined(CONFIG_SYS_XIP_BOOT) > or whatever makes the most sense for a name. Thoughts on the name? > Thanks! Or maybe name the config option not after the boot mode but after what it actually does, and then in the board config, enable this option if the boot mode wants it? Amicalement, -- Albert. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
