From mboxrd@z Thu Jan  1 00:00:00 1970
From: Breno Matheus Lima <brenomatheus@gmail.com>
Date: Thu, 8 Mar 2018 17:53:03 -0300
Subject: [U-Boot] [PATCH] imximage: Check the IVT offset in the correct
	location
In-Reply-To: <1520541633-29312-1-git-send-email-festevam@gmail.com>
References: <1520541633-29312-1-git-send-email-festevam@gmail.com>
Message-ID: <CAC4tdFU6nwSm0VBaQ_1pvYCU+AbDz29wZa-_TBa-eHHoxo_vDQ@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Fabio,

2018-03-08 17:40 GMT-03:00 Fabio Estevam <festevam@gmail.com>:
> From: Fabio Estevam <fabio.estevam@nxp.com>
>
> Sometimes imximage throws the following error:
>
>   CFGS    board/freescale/vf610twr/imximage.cfg.cfgtmp
>   CFGS    board/freescale/vf610twr/imximage.cfg.cfgtmp
>   MKIMAGE u-boot-dtb.imx
> Error: No BOOT_FROM tag in board/freescale/vf610twr/imximage.cfg.cfgtmp
> arch/arm/mach-imx/Makefile:100: recipe for target 'u-boot-dtb.imx' failed
>
> This problem happens because imximage_ivt_offset is being checked
> at un unsafe point, and in some cases it can be checked prior to
> its assignment.
>
> Fix this issue by only checking imximage_ivt_offset after its
> assignment has really occurred.
>
> Introduce a check_ivt_offset() function to help on this task.
>
> Reported-by: Breno Lima <breno.lima@nxp.com>
> Reported-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>

Tested-by: Breno Lima <breno.lima@nxp.com>

Thanks,
Breno Lima