Hi Marek, Pali, On Mon, Nov 8, 2021 at 6:14 PM Marek Behún <[email protected]> wrote: > Fill the real header size without padding into the main header > > This allows to reduce final image when converting image to another format > which does not need additional padding. > > Signed-off-by: Pali Rohár <[email protected]> > Signed-off-by: Marek Behún <[email protected]>
This patch seems to cause mkimage to generate v1 images with invalid checksums (which fail to verify with kwboot or mkimage -l). Could you double check whether you can reproduce on the latest u-boot master? I don't really understand how this patch is supposed to work (headersz_lsb/headersz_msb get updated *after* csum8 has already been computed!). $ tools/mkimage -n ./board/qnap/qsw-98dx3236/kwbimage.cfg -T kwbimage -a 0x00800000 -e 0x00800000 -d u-boot.bin u-boot.kwb Image Type: MVEBU Boot from nand Image Image version:1 BIN Hdr Size: 76224 Bytes = 74.44 KiB = 0.07 MiB Data Size: 735764 Bytes = 718.52 KiB = 0.70 MiB Load Address: 00800000 Entry Point: 00800000 $ sudo tools/kwboot -a -b u-boot.kwb -t -B 115200 /dev/ttyUSB0 kwboot version 2022.01-rc4-00075-g3bd6c62cf4-dirty u-boot.kwb: Invalid image. (The specific kwbimage.cfg/board is still a WIP, I can try to provide some more repro if somehow there is something specific to my setup, but I doubt it.) Best, -- Pierre Bourdon <[email protected]> Software Engineer @ Zürich, Switzerland https://delroth.net/
