On Fri, Mar 09, 2018 at 01:07:21PM +0000, Bryan O'Donoghue wrote: > commit 8c4037a09a5c ("imx: hab: Ensure the IVT DCD pointer is Null prior > to calling HAB authenticate function.") makes the DCD field being NULL a > dependency. > > This change though will break loading and executing of existing pre-signed > binaries on a u-boot update i.e. if this change is deployed on a board you > will be forced to redo all images on that board to NULL out the DCD. > > There is no prior guidance from NXP that the DCD must be NULL similarly > public guidance on usage of the HAB doesn't call out this NULL dependency > (see boundary devices link). > > Since later SoCs will reject a non-NULL DCD there's no reason to make a > NULL DCD a requirement, however if there is an actual dependency for later > SoCs the appropriate fix would be to do SoC version checking. > > Earlier SoCs are capable (and happy) to authenticate images with non-NULL > DCDs, we should not be forcing this change on downstream users - > particularly if it means those users now must rewrite their build systems > and/or redeploy signed images in the field. > > Fixes: 8c4037a09a5c ("imx: hab: Ensure the IVT DCD pointer is Null prior > to calling HAB authenticate function.") > > Signed-off-by: Bryan O'Donoghue <bryan.odonog...@linaro.org> > Cc: Utkarsh Gupta <utkarsh.gu...@nxp.com> > Cc: Breno Lima <breno.l...@nxp.com> > Cc: Fabio Estevam <fabio.este...@nxp.com> > Link: https://boundarydevices.com/high-assurance-boot-hab-dummies > Reviewed-by: Fabio Estevam <fabio.este...@nxp.com>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot