[U-Boot] CONFIG_SYS_XIP_BOOT for when it's a choice?

Albert ARIBAUD albert.u.boot at aribaud.net
Mon Apr 7 18:31:41 CEST 2014


Hi Tom,

On Wed, 5 Mar 2014 12:28:06 -0500, Tom Rini <trini at 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.


More information about the U-Boot mailing list