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

[Stefan Roese]

Hi Pali,

On 22.10.21 10:48, Pali Rohár wrote:
> On Friday 22 October 2021 10:38:16 Stefan Roese wrote:
>> Hi Marek, Hi Pali,
>>
>> since now all is merged into master, I'm testing on my AXP target
>> again. Please see below.
>>
>> On 01.10.21 12:45, Marek Behún wrote:
>>> On Fri, 1 Oct 2021 12:40:29 +0200
>>> Stefan Roese <sr at denx.de> wrote:
>>>
>>>> U-Boot SPL 2021.10-rc5-00431-g6c96332441cf-dirty (Oct 01 2021 - 12:38:55
>>>> +0200)
>>>> High speed PHY - Version: 2.1.5 (COM-PHY-V20)
>>>> High speed PHY - Ended Successfully
>>>> DDR3 Training Sequence - Ver 5.7.4
>>>> DDR3 Training Sequence - Ended Successfully
>>>> Trying to boot from BOOTROM
>>>> Returning to BootROM (return address 0xffff0aa0)...
>>>> rc=-1 errno=110 allow_non_xm=1 *non_xm_print=1
>>>>
>>>>
>>>> xmodem: Connection timed out
>>>
>>> No ACK from BootROM. It is possible that return_from_bootrom() was not
>>> called, or failed, somehow in SPL.
>>
>> I'm not getting this when using master:
>>
>> [stefan at ryzen u-boot (master)]$ ./tools/kwboot -B 230400 -p -b
>> u-boot-spl.kwb -t
>> /dev/serial/by-id/usb-FTDI_FT232R_USB_UART_A1019EGY-if00-port0
>> Patching image boot signature to UART
>> Injecting binary header code for changing baudrate to 230400 Bd
>> Injecting code for changing baudrate back
>> Aligning image header to Xmodem block size
>> Sending boot message. Please reboot the target.../
>> Waiting 2s and flushing tty
>> Sending boot image header (90496 bytes)...
>>    0 %
>> [......................................................................]
>>   10 %
>> [......................................................................]
>>   19 %
>> [......................................................................]
>>   29 %
>> [......................................................................]
>>   39 %
>> [......................................................................]
>>   49 %
>> [......................................................................]
>>   59 %
>> [......................................................................]
>>   69 %
>> [......................................................................]
>>   79 %
>> [......................................................................]
>>   89 %
>> [......................................................................]
>>   99 % [.......       ]
>> Done
>>
>> U-Boot SPL 2021.10-00815-gf200a4bcecf1-dirty (Oct 22 2021 - 10:33:19 +0200)
>> High speed PHY - Version: 2.1.5 (COM-PHY-V20)
>> High speed PHY - Ended Successfully
>> DDR3 Training Sequence - Ver 5.7.4
>> DDR3 Training Sequence - Ended Successfully
>> Trying to boot from BOOTROM
>> Returning to BootROM (return address 0xffff0aa0)...
>>
>> Changing baudrate to 230400 Bd
>> Baudrate was not changed
>>
>>
>> xmodem: Protocol error
>>
>> It works with the standard 115k. Any ideas why switching baudrate fails
>> on AXP?
> 
> When 115k baudrate is specified (or no baudrate is specified) then
> kwboot does not inject any code for changing baudrate (as 115k is
> default). So it looks like that issue is with switching.
> 
> I have already some more "fixup" patches for kwboot and kwbimage, which
> are fixing some issues which I found. So it is possible that you hit
> some of that issue. I will pass patches to Marek and after some testing
> we will send them to mailing list for review.

Okay, sounds like a plan. I'll re-test after you (or Marek) sent these
new patches.

> I do not have any AXP board, so I cannot test or debug it manually.

Unlucky but no big deal. I'll do the testing.

Thanks,
Stefan