[PATCH u-boot-marvell v3 00/39] kwboot higher baudrate

[Stefan Roese]

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 at 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 at 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 at ryzen u-boot ((ab92b38a0161...)|BISECTING)]$ git bisect good
a2ac2b964bfbb20d6791ee94b9034a50cfadb5b0 is the first bad commit
commit a2ac2b964bfbb20d6791ee94b9034a50cfadb5b0
Author: Tom Rini <trini at 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 at 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