> Subject: Re: [PATCH] tools/imx8mimage: Remove the usage of IVT reserved1 > field > > Hi Peng > > On Sun, Jul 3, 2022 at 1:15 AM Peng Fan (OSS) <peng....@oss.nxp.com> > wrote: > > > > From: Peng Fan <peng....@nxp.com> > > > > Commit log from NXP imx-mkimage: > > On iMX8MQ B2, ROM will check IVT header and requires the reserved > > fields in IVT to be 0. > > However, in imx-mkimage we set the reserved1 field for the offset from > > second boot image to SPL boot image, which was used for mfgtool but > > has deprecated. So remove it to fix the boot failure on iMX8MQ B2. > > > > Update imx8mimage to address same issue > > > > Cc: Mihai Chelalau <mihai.chela...@nxp.com> > > Cc: Dan Douglass <dan.dougl...@nxp.com> > > Signed-off-by: Peng Fan <peng....@nxp.com> > > --- > > tools/imx8mimage.c | 8 -------- > > 1 file changed, 8 deletions(-) > > > > diff --git a/tools/imx8mimage.c b/tools/imx8mimage.c index > > 4eed683396f..52baf4c9903 100644 > > --- a/tools/imx8mimage.c > > +++ b/tools/imx8mimage.c > > @@ -505,14 +505,6 @@ void build_image(int ofd) > > exit(EXIT_FAILURE); > > } else { > > sld_header_off = sld_src_off - rom_image_offset; > > - /* > > - * Record the second bootloader relative offset in > > - * image's IVT reserved1 > > - */ > > - if (rom_version == ROM_V1) { > > - imx_header[IMAGE_IVT_ID].fhdr.reserved1 = > > - sld_header_off - header_image_off; > > - } > > sld_fd = open(sld_img, O_RDONLY | O_BINARY); > > if (sld_fd < 0) { > > fprintf(stderr, "%s: Can't open: > > %s\n", > > -- > > 2.36.0 > > > > I have seen a similar change from Marek last week > > tools: imx8mimage: Keep IVT reserved1 field zero always [Peng Fan]
Oh, yes. Please ignore this patch. Thanks, Peng. > > Michael