From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 7 Apr 2014 12:45:51 -0400 Subject: [U-Boot] CONFIG_SYS_XIP_BOOT for when it's a choice? In-Reply-To: References: <20140305172806.GY16805@bill-the-cat> Message-ID: <20140407164551.GF23803@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: