On 24-01-11, Marco Felsch wrote:
> An size of 4M seems quite large and upstream OP-TEE code mostly uses 2M
> for all i.MX platforms. However keep the 4M as default but make it
> configurable for integrators which want to use the autom. OF fixup
> logic.
> 
> Signed-off-by: Marco Felsch <m.fel...@pengutronix.de>
> ---
>  arch/arm/mach-imx/imx8m.c | 2 +-
>  common/Kconfig            | 9 +++++++++
>  2 files changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/mach-imx/imx8m.c b/arch/arm/mach-imx/imx8m.c
> index c7c799c64bc4..90dc12ca18cc 100644
> --- a/arch/arm/mach-imx/imx8m.c
> +++ b/arch/arm/mach-imx/imx8m.c
> @@ -68,7 +68,7 @@ static int imx8m_init(const char *cputypestr)
>       if (IS_ENABLED(CONFIG_PBL_OPTEE) && tzc380_is_enabled() &&
>           !of_find_node_by_path_from(NULL, "/firmware/optee")) {
>               static struct of_optee_fixup_data optee_fixup_data = {
> -                     .shm_size = SZ_4M,
> +                     .shm_size = CONFIG_OPTEE_SHM_SIZE,

This fails if not defined via Kconfig. I prepare a v2 which uses the
same mechanism as we use for OPTEE_SIZE.

Regards,
  Marco

>                       .method = "smc",
>               };
>  
> diff --git a/common/Kconfig b/common/Kconfig
> index 1de76651fddf..5b5da6224752 100644
> --- a/common/Kconfig
> +++ b/common/Kconfig
> @@ -1136,6 +1136,15 @@ config OPTEE_SIZE
>         Can be smaller than the actual size used by OP-TEE, this is used to 
> prevent
>         barebox from allocating memory in this area.
>  
> +config OPTEE_SHM_SIZE
> +     hex
> +     default 0x400000
> +     prompt "OP-TEE Shared Memory Size"
> +     depends on HAVE_OPTEE
> +     help
> +       Size to reserve in main memory for OP-TEE shared memory communication.
> +       Can be used for fixing up the OP-TEE OF node.
> +
>  config BOOTM_OPTEE
>       bool
>       prompt "support booting OP-TEE"
> -- 
> 2.39.2
> 
> 
> 

Reply via email to