Hi Peng
On Sun, Jul 3, 2022 at 8:46 AM Peng Fan (OSS) <peng....@oss.nxp.com> wrote:
>
> > 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.
>
Can you add your review on the Marek patch?
Michael
> Thanks,
> Peng.
>
> >
> > Michael
--
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
mich...@amarulasolutions.com
__________________________________
Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
i...@amarulasolutions.com
www.amarulasolutions.com
