Re: [PATCH 2/8] spl: atf: move storage for bl31_params into function

[Michal Simek]

On 16. 11. 20 22:47, Michael Walle wrote:
> There is no need to have the storage available globally. This is also a
> preparation for LOAD_IMAGE_V2 support. That will introduce a similar
> generator function which also has its own storage.
> 
> Signed-off-by: Michael Walle <mich...@walle.cc>
> ---
>  common/spl/spl_atf.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/common/spl/spl_atf.c b/common/spl/spl_atf.c
> index 9bd25f6b32..df0a198d55 100644
> --- a/common/spl/spl_atf.c
> +++ b/common/spl/spl_atf.c
> @@ -18,13 +18,12 @@
>  #include <spl.h>
>  #include <asm/cache.h>
>  
> -static struct bl2_to_bl31_params_mem bl31_params_mem;
> -static struct bl31_params *bl2_to_bl31_params;
> -
>  __weak struct bl31_params *bl2_plat_get_bl31_params(uintptr_t bl32_entry,
>                                                   uintptr_t bl33_entry,
>                                                   uintptr_t fdt_addr)
>  {
> +     static struct bl2_to_bl31_params_mem bl31_params_mem;
> +     struct bl31_params *bl2_to_bl31_params;
>       struct entry_point_info *bl32_ep_info;
>       struct entry_point_info *bl33_ep_info;
>  
> 

Acked-by: Michal Simek <michal.si...@xilinx.com>

M