diff options
| author | Bryan O'Donoghue <bryan.odonoghue@linaro.org> | 2018-01-12 12:40:15 +0000 |
|---|---|---|
| committer | Stefano Babic <sbabic@denx.de> | 2018-01-14 17:26:30 +0100 |
| commit | 4467ae6c28aa7eef84d5784f56234472a015d5e2 (patch) | |
| tree | 49d0c926640e4441d88e72f5121e4d4cecb69e6d /arch/arm/mach-imx/hab.c | |
| parent | e5b30e4ac19b745ef5ed75813d48fa78d7c579f6 (diff) | |
arm: imx: hab: Make authenticate_image() return zero on open boards
The BootROM will not successfully process a HAB image passed by u-boot
unless the board has been set into locked mode. Some of the existing usages
of authenticate_image() expect and rely on unlocked boards doing the
following
1. Not calling into the BootROM authenticate_image() callback
2. Returning a pass status for authenticate_image() calls anyway
A previous patch removed the necessity to call into imx_hab_is_enabled()
twice. This patch ensures the reliance on authenticate_image() returning
zero is maintained.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Suggested-by: Breno Matheus Lima <brenomatheus@gmail.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Albert Aribaud <albert.u.boot@aribaud.net>
Cc: Sven Ebenfeld <sven.ebenfeld@gmail.com>
Cc: George McCollister <george.mccollister@gmail.com>
Tested-by: Breno Lima <breno.lima@nxp.com>
Diffstat (limited to 'arch/arm/mach-imx/hab.c')
| -rw-r--r-- | arch/arm/mach-imx/hab.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/mach-imx/hab.c b/arch/arm/mach-imx/hab.c index d917ac3af9a..3b19a7e5736 100644 --- a/arch/arm/mach-imx/hab.c +++ b/arch/arm/mach-imx/hab.c @@ -458,7 +458,7 @@ int imx_hab_authenticate_image(uint32_t ddr_start, uint32_t image_size, if (!imx_hab_is_enabled()) { puts("hab fuse not enabled\n"); - return result; + return 0; } printf("\nAuthenticate image from DDR location 0x%x...\n", |