RE: [PATCH] tools/imx8mimage: Remove the usage of IVT reserved1 field

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.fan@oss.nxp.com wrote:

...

From: Peng Fan peng.fan@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.chelalau@nxp.com Cc: Dan Douglass dan.douglass@nxp.com Signed-off-by: Peng Fan peng.fan@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