[Skiboot] [PATCH 3/3] core/init: move imc catalog preload init after the STB init.

[Vasant Hegde]

On 02/06/2018 07:46 AM, Pridhiviraj Paidipeddi wrote:
> As a safer side move the imc catalog preload after the STB init
> to make sure the imc catalog resource get's verified and measured
> properly during loading when both secure and trusted boot modes
> are on.
> 
> Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe at linux.vnet.ibm.com>
> ---
>   core/init.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/core/init.c b/core/init.c
> index ec9f329..6eb4d83 100644
> --- a/core/init.c
> +++ b/core/init.c
> @@ -997,9 +997,6 @@ void __noreturn __nomcount main_cpu_entry(const void *fdt)
>   	/* Read in NVRAM and set it up */
>   	nvram_init();
> 
> -	/* preload the IMC catalog dtb */
> -	imc_catalog_preload();
> -
>   	/* Set the console level */
>   	console_log_level();
> 
> @@ -1007,6 +1004,9 @@ void __noreturn __nomcount main_cpu_entry(const void *fdt)
>   	secureboot_init();
>   	trustedboot_init();

Can we initialize secureboot before platform init ? That would solve all the issues.

-Vasant