From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Tue, 01 Nov 2011 06:06:31 +0100 Subject: [U-Boot] [PATCH v5 05/11] arm, davinci: add support for new spl framework In-Reply-To: <4EAF1CA7.5010403@freescale.com> References: <1320036790-24283-1-git-send-email-hs@denx.de> <1320036790-24283-6-git-send-email-hs@denx.de> <4EAF1CA7.5010403@freescale.com> Message-ID: <4EAF7E57.2000104@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Scott, Scott Wood wrote: > On 10/30/2011 11:53 PM, Heiko Schocher wrote: >> diff --git a/arch/arm/cpu/arm926ejs/davinci/spl_nand.c b/arch/arm/cpu/arm926ejs/davinci/spl_nand.c >> new file mode 100644 >> index 0000000..efc0521 >> --- /dev/null >> +++ b/arch/arm/cpu/arm926ejs/davinci/spl_nand.c >> @@ -0,0 +1,57 @@ [...] >> +#include >> +#include >> +#include > > Does this file use anything from asm/io.h? removed. >> +/* >> + * The main entry for NAND booting. It's necessary that SDRAM is already >> + * configured and available since this code loads the main U-Boot image >> + * from NAND into SDRAM and starts it from there. >> + */ [...] >> + /* >> + * Jump to U-Boot image >> + */ >> + uboot = (void *)CONFIG_SYS_NAND_U_BOOT_START; >> + (*uboot)(); >> +} > > This is common to most NAND SPLs. It would be nice to stick this in a > common place (at least the loading part, maybe not the jump). Not > nand_spl_simple.c, as it's utility is not limited to the simple driver. > Maybe drivers/mtd/nand/nand_spl_load.c? moved to drivers/mtd/nand/spl_nand_load.c, also I want to let the jump in this file. I prefer the name "spl_nand_load.c as we have already a CONFIG_SPL_NAND_SIMPLE -> CONFIG_SPL_NAND_LOAD -> spl_nand_load.c Is this OK for you? Shouldn;t we rename nand_spl_simple.c to spl_nand_simple.c also? bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany