Added to Tom to Cc
On 01.10.21 12:43, Stefan Roese wrote:
On 01.10.21 12:39, Pali Rohár wrote:
Hello!
On Friday 01 October 2021 12:28:34 Stefan Roese wrote:
First the test on "next" without the 39 patches:
[stefan@ryzen u-boot (next)]$ ./tools/kwboot -p -b u-boot-spl.kwb -t
...
DDR3 Training Sequence - Ended Successfully Trying to boot from BOOTROM Returning to BootROM (return address 0xffff0aa0)... +xmodem: Protocol error
Not working!
And now test on "master" without the 39 patches:
[stefan@ryzen u-boot (master)]$ ./tools/kwboot -p -b u-boot-spl.kwb -t
...
Seems to okay.
Thanks, Stefan
So something is broken in next branch. I have looked at kwboot and kwbimage code and there is no change in next branch which is not in master branch.
So it looks like that something not related to kwboot broke it in next branch. And seems that git bisect on AXP hw would be required. But I do not own AXP board, so I cannot do it.
I can try to do this here later, if I find the time.
git bisect leads to this problematic commit:
[stefan@ryzen u-boot ((ab92b38a0161...)|BISECTING)]$ git bisect good a2ac2b964bfbb20d6791ee94b9034a50cfadb5b0 is the first bad commit commit a2ac2b964bfbb20d6791ee94b9034a50cfadb5b0 Author: Tom Rini trini@konsulko.com Date: Fri Aug 27 21:18:30 2021 -0400
Convert CONFIG_SKIP_LOWLEVEL_INIT et al to Kconfig
This converts the following to Kconfig: CONFIG_SKIP_LOWLEVEL_INIT CONFIG_SKIP_LOWLEVEL_INIT_ONLY
In order to do this, we need to introduce SPL and TPL variants of these options so that we can clearly disable these options only in SPL in some cases, and both instances in other cases.
Signed-off-by: Tom Rini trini@konsulko.com
Frankly, I did not analyse this change fully. But it seems to responsible for this breakage on Armada XP in "next".
Thanks, Stefan