Re: [PATCH u-boot-marvell 00/29] kwboot higher baudrate

[Chris Packham <judge.packham@gmail.com>]

On Fri, Aug 27, 2021 at 1:16 PM Chris Packham <judge.packham@gmail.com> wrote:
>
> On Thu, Aug 26, 2021 at 1:46 AM Marek Behún <marek.behun@nic.cz> wrote:
> >
> > Hello Stefan and others,
> >
> > this series adds support for booting Marvell platforms via UART (those
> > bootable with kwboot) at higher baudrates.
> >
> > Tested on Turris Omnia up to 5.15 MBd, which is 44x faster than
> > 115200 Bd.
> >
> > The user can direct kwboot to use higher baudrate via the -B option.
> > (BTW this option was there before but did not work with the -b option.)
> >
> > Only the payload part of the KWB image is uploaded at this higher
> > baudrate. The header part is still uploaded at 115200 Bd, since the code
> > that changes baudrate is injected into header as a binary extension.
> > (The payload part makes up majority of the binary, though. On Turris
> >  Omnia the payload currently makes ~87%.)
> >
> > The series also contains various other fixes, refactors and improvements
> > of the code, upon which the main change is done.
> >
> > Marek & Pali
>
> What tree/sha is this series based on. I've tried to apply them out of
> patchwork and it fails at "tools: kwbimage: Simplify iteration over
> version 1 optional headers"
>

Ah it needs http://patchwork.ozlabs.org/project/uboot/list/?series=257577

> >
> > Marek Behún (13):
> >   tools: kwbimage: Fix printf format warning
> >   tools: kwboot: Fix buffer overflow in kwboot_terminal()
> >   tools: kwboot: Make the quit sequence buffer const
> >   tools: kwboot: Refactor and fix writing buffer
> >   tools: kwboot: Fix comparison of integers with different size
> >   tools: kwboot: Use a function to check whether received byte is a
> >     Xmodem reply
> >   tools: kwboot: Print new line after SPL output
> >   tools: kwboot: Allow greater timeout when executing header code
> >   tools: kwbimage: Simplify iteration over version 1 optional headers
> >   tools: kwbimage: Refactor image_version()
> >   tools: kwbimage: Refactor kwbimage header size determination
> >   doc/kwboot.1: Update man page
> >   MAINTAINERS: Add entry for kwbimage / kwboot tools
> >
> > Pali Rohár (16):
> >   tools: kwboot: Print version information header
> >   tools: kwboot: Fix kwboot_xm_sendblock() function when
> >     kwboot_tty_recv() fails
> >   tools: kwboot: Fix return type of kwboot_xm_makeblock() function
> >   tools: kwboot: Fix printing progress
> >   tools: kwboot: Print newline on error when progress was not completed
> >   tools: kwboot: Split sending image into header and data stages
> >   tools: kwboot: Allow non-xmodem text output from BootROM only in a
> >     specific case
> >   tools: kwboot: Properly finish xmodem transfer
> >   tools: kwboot: Always call kwboot_img_patch_hdr()
> >   tools: kwboot: Don't patch image header if signed
> >   tools: kwboot: Patch source address in image header
> >   tools: kwboot: Patch destination address to DDR area for SPI image
> >   tools: kwboot: Round up header size to 128 B when patching
> >   tools: kwboot: Support higher baudrates when booting via UART
> >   tools: kwboot: Allow any baudrate on Linux
> >   tools: kwboot: Add Pali and Marek as authors
> >
> >  MAINTAINERS           |   10 +
> >  doc/kwboot.1          |   58 ++-
> >  tools/kwbimage.c      |  127 ++---
> >  tools/kwbimage.h      |   93 +++-
> >  tools/kwboot.c        | 1103 +++++++++++++++++++++++++++++++++++------
> >  tools/termios_linux.h |  170 +++++++
> >  6 files changed, 1289 insertions(+), 272 deletions(-)
> >  create mode 100644 tools/termios_linux.h
> >
> > --
> > 2.31.1
> >