[PATCH v2] tools: mkimage: Call verify_header after writing image to disk

Tom Rini trini at konsulko.com
Sat Jan 22 03:15:43 CET 2022


On Sat, Jan 22, 2022 at 02:44:22AM +0100, Pali Rohár wrote:
> On Friday 21 January 2022 16:21:33 Tom Rini wrote:
> > On Fri, Jan 14, 2022 at 06:34:43PM +0100, Pali Rohár wrote:
> > 
> > > If image backend provides verify_header callback then call it after writing
> > > image to disk. This ensures that written image is correct.
> > > 
> > > Signed-off-by: Pali Rohár <pali at kernel.org>
> > > Reviewed-by: Stefan Roese <sr at denx.de>
> > > Reviewed-by: Simon Glass <sjg at chromium.org>
> > > ---
> > >  tools/mkimage.c | 41 +++++++++++++++++++++++++++++++++++++++++
> > >  1 file changed, 41 insertions(+)
> > 
> > This breaks a number of platforms such as ls1021atwr_sdcard_qspi and
> > it's not clear to me why exactly.
> 
> Maybe they were already broken and this patch just detected it?
> Or verify_header callback for particular image type is reject valid
> image?
> 
> Do you have some pointers to failed build logs?

Try building for ls1021atwr_sdcard_qspi with your patch applied, the
only new thing that's shown in the logs is the error message.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220121/13052384/attachment.sig>


More information about the U-Boot mailing list