On Mon, Apr 07, 2014 at 06:31:41PM +0200, Albert ARIBAUD wrote: > 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?
Right. But what to call the option? -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
