[U-Boot] [PATCH v5 05/11] arm, davinci: add support for new spl framework

Scott Wood scottwood at freescale.com
Tue Nov 1 16:39:35 CET 2011


On 11/01/2011 12:06 AM, Heiko Schocher wrote:
> Hello Scott,
> 
> Scott Wood wrote:
>> On 10/30/2011 11:53 PM, Heiko Schocher wrote:
>> 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?

I suggested nand_spl_load.c because we already have a nand_spl_simple.c. :-)

I don't really care which way it goes, as long as it's consistent.

-Scott



More information about the U-Boot mailing list