[U-Boot] [RFC PATCH 6/8] omap3_beagle: add nand_spl support

[John Rigby]

On Tue, Dec 28, 2010 at 12:49 AM, Dirk Behme <dirk.behme at googlemail.com> wrote:
> On 28.12.2010 07:50, Aneesh V wrote:
>> Hello John,
>>
>> On Tuesday 28 December 2010 06:17 AM, John Rigby wrote:
>> <  snip>
>>
>>> +LDSCRIPT= $(TOPDIR)/nand_spl/board/$(BOARDDIR)/u-boot.lds
>>> +LDFLAGS     = -Bstatic -T $(nandobj)u-boot.lds -Ttext $(CONFIG_SYS_NAND_SPL_TEXT_BASE) $(PLATFORM_LDFLAGS)
>>> +AFLAGS      += -DCONFIG_PRELOADER -DCONFIG_NAND_SPL
>>> +CFLAGS      += -DCONFIG_PRELOADER -DCONFIG_NAND_SPL
>>
>> How big is the spl now? For the OMAP4 spl, using '-ffunction-sections'
>> and '--gc-sections' reduced the image size by 40% and helped it fit
>> into the SRAM bugdet. I am sure your nand_spl is already fitting in the
>> SRAM bugdet. But these flags may help reduce the size further. Do you
>> want to try it?
>
> Applying this patch series and building with gcc version 4.3.3
> (Sourcery G++ Lite 2009q1-203) I get
>
>  > ./MAKEALL omap3_beagle_nand
>
> nand_boot.c: In function 'nand_boot':
>
> nand_boot.c:356: warning: 'noreturn' function does return
>
> mkimage.c: In function ‘main’:
>
> mkimage.c:159: warning: implicit declaration of function
> ‘init_omap_image_type’
>
>  > ll u-boot-spl.ift
> 12580 u-boot-spl.ift
>
> Using gcc version 4.5.1 (Sourcery G++ Lite 2010.09-50) results in the
> same warnings and
>
Thanks, Dirk.  I will fix these in the next version.