From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 16 May 2013 10:49:05 -0400 Subject: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR. In-Reply-To: <87y5bfm1pp.fsf@dell.be.48ers.dk> References: <1368469719-22406-1-git-send-email-trini@ti.com> <1368469719-22406-7-git-send-email-trini@ti.com> <87y5bfm1pp.fsf@dell.be.48ers.dk> Message-ID: <20130516144905.GI32163@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 Thu, May 16, 2013 at 04:36:02PM +0200, Peter Korsgaard wrote: > >>>>> "Tom" == Tom Rini writes: > > Tom> From: Steve Kipisz > Tom> NOR requires that s_init be within the first 4KiB of the image so that > Tom> we can perform the rest of the required pinmuxing to talk with the rest > Tom> of NOR that we are found on. When NOR_BOOT is set we save our > Tom> environment in NOR at 512KiB and a redundant copy at 768KiB. We avoid > Tom> using SPL for this case and u-boot.bin is written directly to the start > Tom> of NOR. [snip] > Tom> -#ifdef CONFIG_SPL_BUILD > Tom> +#if defined(CONFIG_SPL_BUILD) || defined(CONFIG_NOR_BOOT) > Tom> +#ifdef CONFIG_TI81XX > Tom> static struct dmm_lisa_map_regs *hw_lisa_map_regs = > Tom> (struct dmm_lisa_map_regs *)DMM_BASE; > Tom> +#endif > Tom> static struct vtp_reg *vtpreg[2] = { > Tom> (struct vtp_reg *)VTP0_CTRL_ADDR, > Tom> (struct vtp_reg *)VTP1_CTRL_ADDR}; > Tom> @@ -53,6 +55,7 @@ static struct vtp_reg *vtpreg[2] = { > Tom> static struct ddr_ctrl *ddrctrl = (struct ddr_ctrl *)DDR_CTRL_ADDR; > Tom> #endif > > Tom> +#ifdef CONFIG_TI81XX > > Why are you adding the TI81XX dependency here? That doesn't have > anything to do with nor boot, does it? dmm is TI81XX-only (for this file, it's also omap4+) and since main U-Boot doesn't use --ffunction-sections/--fdata-sections/--gc-sections we end up with a link error about not having enable_dmm_clocks(). This is OK on SPL since we do use --gc-sections and the link error is avoided as we discard config_dmm on am33xx builds. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: