[U-Boot] [PATCH v2 3/4] ARM: atmel: at91sam9x5ek: enable spl support

[Stefan Roese]

Hi Eugene,

On 19.02.19 15:56, Stefan Roese wrote:

<snip>

>> I did not test myself the SPL for any 9x5 board. The DDR2 init code
>> should be executed inside the SPL. Once you build your board with a
>> CONFIG_SPL enabled, you should get the spl binary built, then you can
>> add both SPL + U-boot proper on the boot media and have SPL boot
>> (execute DDR init code and load U-boot proper). I would suggest to try
>> on Sd-card first because it's easier (filesystem), but the commit says
>> only SPI and NAND tested.
> 
> On this board I only have NAND, so there is no option to first test
> some other boot device unfortunately.
> 
>> You can also look at the U-boot version which first included this patch,
>> some things may have changed since...
> 
> Sure, I can and will check this. But I first wanted to check with
> you guys about the status, since you implemented and most likely
> also tested this stuff.
>    
>> Let me know of your findings. I will let you know if I have the chance
>> to test this myself.
> 
> Okay. I'll try to test this stuff (even though I don't have JTAG on
> this board) and will get back to you with findings and most likely
> also some further questions.

So I did make some progress here but building and testing the SPL
version by loading it to 0x30.0000 and starting it there via the
"go" command. I'm not brave enough to flash it right now, since
de-bricking might be tricky here, as I don't have JTAG. With the
loaded version I'm able to fully load the main U-Boot image from
NAND and run it from there to the prompt.

But then I looked at the latest AT91Bootstrap code and noticed the
many changes in the lowlevel boot code (clocking, DDR2, etc). I'm
a bit hesitant to port all this code to mainline U-Boot SPL here.
One reason being that you at Microchip / Atmel will most likely
continue to work on the AT91Bootstrap code and not on the
potentially ported SPL code version. So what's you opinion here on
this? Why do you not integrate all this code into U-Boot SPL and
drop this bootstrap version completely? What is your suggestion for
me on how to handle this? Stay with AT91Bootstrap or move to SPL?

Thanks,
Stefan