[PATCH u-boot-marvell 00/13] Yet another kwboot improvements

[Pali Rohár]

On Monday 25 October 2021 16:39:44 Stefan Roese wrote:
> Hi Marek,
> 
> On 25.10.21 15:12, Marek Behún wrote:
> > From: Marek Behún <marek.behun at nic.cz>
> > 
> > Hello Stefan,
> > 
> > these are another improvements for kwboot, please apply only after series
> >    arm: mvebu: nandpagesize support for kwbimage v1
> 
> I'm checking right now and have applied the 3 NAND patches on current
> master. But this patchset fails at this one:
> 
> tools: kwboot: Do not modify kwbimage header before increasing its size
> 
> [stefan at ryzen u-boot-marvell (kwboot-test1)]$ git am -3 ~/tmp/kwboot2/*
> Applying: tools: kwboot: Initialize rfds to zero
> Applying: tools: kwboot: Fix initialization of tty device
> Applying: tools: kwboot: Reserve enough space for patching kwbimage in
> memory
> Applying: tools: kwboot: Validate 4-byte image data checksum
> Applying: tools: kwboot: Inject baudrate change back code after data part
> Applying: tools: kwboot: Recalculate 4-byte data checksum after injecting
> baudrate code
> Applying: tools: kwboot: Correctly set configuration of UART for BootROM
> messages
> Applying: tools: kwboot: Show verbose message when waiting for baudrate
> change magic
> Applying: tools: kwboot: Simplify code for aligning image header
> Applying: tools: kwboot: Do not modify kwbimage header before increasing its
> size
> error: sha1 information is lacking or useless (tools/kwboot.c).
> error: could not build fake ancestor
> Patch failed at 0010 tools: kwboot: Do not modify kwbimage header before
> increasing its size
> hint: Use 'git am --show-current-patch=diff' to see the failed patch
> When you have resolved this problem, run "git am --continue".
> If you prefer to skip this patch, run "git am --skip" instead.
> To restore the original branch and stop patching, run "git am --abort".
> 
> Any idea what's missing here?

Hello! I'm using also this patch:
https://patchwork.ozlabs.org/project/uboot/patch/20211021144609.9319-2-pali@kernel.org/

> Thanks,
> Stefan
> 
> > The main improvement is in patch 5, which changes where we inject the code
> > for changing baudrate back to 115200 Baud after fast upload. Instead of
> > injecting it before the main data image, we now inject it after.
> > 
> > This is because there are some kwb images that upload at address 0, and
> > injecting the code before that doesn't work, since there is no RAM mapped
> > at 0xfffff000.
> > 
> > Marek & Pali
> > 
> > Pali Rohár (13):
> >    tools: kwboot: Initialize rfds to zero
> >    tools: kwboot: Fix initialization of tty device
> >    tools: kwboot: Reserve enough space for patching kwbimage in memory
> >    tools: kwboot: Validate 4-byte image data checksum
> >    tools: kwboot: Inject baudrate change back code after data part
> >    tools: kwboot: Recalculate 4-byte data checksum after injecting
> >      baudrate code
> >    tools: kwboot: Correctly set configuration of UART for BootROM
> >      messages
> >    tools: kwboot: Show verbose message when waiting for baudrate change
> >      magic
> >    tools: kwboot: Simplify code for aligning image header
> >    tools: kwboot: Do not modify kwbimage header before increasing its
> >      size
> >    tools: kwboot: Calculate real used space in kwbimage header when
> >      calling kwboot_img_grow_hdr()
> >    tools: kwboot: Change retry loop from decreasing to increasing
> >    tools: kwboot: Resend first 3 xmodem retry packets immediately
> > 
> >   tools/kwboot.c | 178 +++++++++++++++++++++++++++++++++----------------
> >   1 file changed, 120 insertions(+), 58 deletions(-)
> > 
> 
> 
> Viele Grüße,
> Stefan
> 
> -- 
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de