Hi Jun

> Subject: [PATCH v3 3/9] imx: mx7: Add empty arch_cpu_init if skipped
> 
> Add empty arch_cpu_init if low level init is skipped. So that it does not 
> break
> spl compile though spl is not needed in the skipped case actually.
> 
> Signed-off-by: Jun Nie <jun....@linaro.org>
> ---
>  arch/arm/mach-imx/mx7/soc.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mx7/soc.c b/arch/arm/mach-imx/mx7/soc.c
> index 7cfdff0..9b04013 100644
> --- a/arch/arm/mach-imx/mx7/soc.c
> +++ b/arch/arm/mach-imx/mx7/soc.c
> @@ -286,6 +286,11 @@ int arch_cpu_init(void)
> 
>       return 0;
>  }
> +#else
> +int arch_cpu_init(void)
> +{
> +     return 0;
> +}
>  #endif

Please describe what ATF/OP-TEE initialization has done
when booting into uboot.

I think SKIP_LOWLEVEL_INIT is mostly for lowlevel_init,
not arch_cpu_init.

init_aips/init_csu/isolate_resources/init_snvs might be
done in your ATF, I am not sure, but imx_enet_mdio_fixup,
mxs_dma_init, imx_gpcv2_init, are these also done in ATF?

Regards,
Peng.

> 
>  #ifdef CONFIG_ARCH_MISC_INIT
> --
> 2.7.4

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to