[U-Boot] [PATCH v2] add nand spl boot for qi_lb60 board

[Scott Wood]

On 01/10/2012 09:13 AM, Xiangfu Liu wrote:
> Hi Scott
> 
> thanks for your comments. PATCH v3 have sent out for you to review.
> 
> On 01/07/2012 06:36 AM, Scott Wood wrote:
>> We are transitioning from nand_spl/ to spl/.  Please try to work within
>> the new infrastructure.  You may run into problems with size in spl/,
>> due to gc-sections not removing strings on components you don't need --
>> I plan to fix those issues (for NAND), hopefully this merge window, by
>> requiring all files to be explicitly asked for.  Feel free to fix them
>> first if you'd like. :-)
>>
> 
> I'd like to move the new spl/. I have one question about the new spl/
> I didn't see the new spl/ link to nand_spl/nand_boot.c. so how the new
> spl/ using nand_boot()?

It's been moved to drivers/mtd/nand/nand_spl_simple.c and
drivers/mtd/nand/nand_spl_load.c.

> can we add a new define 'CONFIG_SPL_START_S_NAME', since qi_lb60 using
> start_spl.S for SPL, not start.S.

Heh, that didn't take long:
http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/108574

Or, could we just leave start.o up to the arch Makefile, and just make
sure the arch "library" is linked in first?

> I think better I make the nand_spl/ working. then move to the new spl/ :)
> 
>>> +include $(TOPDIR)/config.mk
>>> +
>>> +LDSCRIPT= $(TOPDIR)/nand_spl/board/$(BOARDDIR)/u-boot.lds
>>> +LDFLAGS    = -Bstatic -T $(LDSCRIPT) -Ttext
>>> $(CONFIG_NAND_SPL_TEXT_BASE)
>>> +AFLAGS    += -DCONFIG_NAND_SPL
>>> +CFLAGS    += -DCONFIG_NAND_SPL -O2
>>
>> Why -O2 and not the -Os that you should already be inheriting from
>> config.mk?
> 
> when I remove the -O2. it give me error:
>  jz4740.c:(.text.sdram_init+0x14): undefined reference to `memcpy'
> so I keep the -O2 there. please anyone give me some advice on how to fix
> this.
> that will be great.

GCC requires memcpy even in freestanding environments.  Can you fit a
minimal memcpy in, such as what's currently bcopy in lib/string.c?

-Scott