Re: [PATCH v2 11/15] security: optee: add optee_handoff_overlay helper

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

Hi,

On 2/4/26 9:01 PM, Marco Felsch wrote:
> Add a helper to pass the OP-TEE provided overlay to barebox via the
> handoff mechanism. This can be useful to generalize the reserved-memory
> node handling since OP-TEE can provide this information via the overlay
> if configured in OP-TEE.
> 
> While on it sort the include headers in alphabetical order.

With below omission fixed:

Reviewed-by: Ahmad Fatoum <a.fatoum@pengutronix.de>

> 
> Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
> ---
>  include/tee/optee.h |  5 +++++
>  security/Kconfig    |  9 +++++++++
>  security/optee.c    | 15 ++++++++++++++-
>  3 files changed, 28 insertions(+), 1 deletion(-)
> 
> diff --git a/include/tee/optee.h b/include/tee/optee.h
> index 10e829c04f8832c4c04771800f1963e25f425482..7126e3c6e24bbf0f37aee7827dc3b4d9619d3bc3 100644
> --- a/include/tee/optee.h
> +++ b/include/tee/optee.h
> @@ -37,6 +37,7 @@ int optee_verify_header (const struct optee_header *hdr);
>  
>  void optee_set_membase(const struct optee_header *hdr);
>  int optee_get_membase(u64 *membase);
> +void optee_handoff_overlay(void *ovl, unsigned int ovl_sz);
>  
>  #else
>  
> @@ -49,6 +50,10 @@ static inline int optee_get_membase(u64 *membase)
>  	return -ENOSYS;
>  }
>  
> +void optee_handoff_overlay(void *ovl, unsigned int ovl_sz)
> +{
> +}

static inline missing

Cheers,
Ahmad

-- 
Pengutronix e.K.                  |                             |
Steuerwalder Str. 21              | http://www.pengutronix.de/  |
31137 Hildesheim, Germany         | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |