[U-Boot] [PATCH v4 2/4] apf27: add support for the armadeus APF27 board

[Stefano Babic]

Hi Philippe,

On 01/08/2013 19:12, trem wrote:
>> Ok, this must be done in assembly - normally is part of lowelevel_init,
>> while board_init_f() is already written in C.
>>
>> Really I was waiting that general code is called, that is
>> spl_nand_load_image() and then the driver function. You are pushing a
>> parallel way that is, at the end, still the old way to boot (nand_spl).
> 
> This board boot from nand, so the SPL should be lower than 2KB.
> The SPL start from NFC (nand) buffer, and It has to :
> - initialize the ram
> - copy itself to another location (ram)
> - jump to this another location
> - copy u-boot from nand to ram
> - jump to ram
> 
> We have tried to use SPL framework (with nand driver), but the result
> is a SPL bigger than 2 KB, so we can't use it.
> 
> To have a SPL smaller than 2 KB, we have written everything in assembler.

Ok, sorry - I am now used with current i.MX where there is not this
constraint. I reread the boot part of i.MX27 manual and I got the point.
I understand there is no other way for the i.MX27.

I would like you add your useful explanation why you add an assembly
NAND driver (so what you have written here in the lines above) to the
commit message, so that is clear why you were forced to use this way
instead of the SPL framework.

> 
> Are you agree with this solution for the SPL ?
> 

I understand there is no other solution ;-)

Best regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================