On Fri, Jan 12, 2018 at 02:52:18PM +0000, Bryan O'Donoghue wrote:
> This patch makes OTPEE_TZDRAM_BASE a mandatory parameter.
> Subsequent patches will ensure that the region between
> OTPEE_TZDRAM_BASE and (OTPEE_TZDRAM_BASE +
> CONFIG_OPTEE_TZDRAM_SIZE) match the information given in the OPTEE header
> before handing off control to the OPTEE image when booting OPTEE directly
> via bootm.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonog...@linaro.org>
> Cc: Harinarayan Bhatta <harinara...@ti.com>
> Cc: Andrew F. Davis <a...@ti.com>
> Cc: Tom Rini <tr...@konsulko.com>
> Cc: Kever Yang <kever.y...@rock-chips.com>
> Cc: Philipp Tomsich <philipp.toms...@theobroma-systems.com>
> Cc: Peng Fan <peng....@nxp.com>
> ---
> lib/optee/optee.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/lib/optee/optee.c b/lib/optee/optee.c
> index a6c856a..6e55027 100644
> --- a/lib/optee/optee.c
> +++ b/lib/optee/optee.c
> @@ -5,9 +5,14 @@
> * SPDX-License-Identifier: GPL-2.0+
> */
>
> +#include <config.h>
> #include <common.h>
> #include <tee/optee.h>
>
> +#ifndef OPTEE_TZDRAM_BASE
> +#error "OPTEE_TZDRAM_BASE not defined"
> +#endif
> +
> int optee_verify_image(struct optee_header *hdr, unsigned long tzdram_start,
> unsigned long tzdram_len, unsigned long image_len)
> {I'm not a fan of adding #error's like this. Since this is a static value,it should be in Kconfig (with appropriate depends) or defined in a consistent location (asm/arch/optee.h ?) and it should be obvious from the code that uses the value that you need to have this value be provided and how to determine what the valid and correct value is. Thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
