[U-Boot] [PATCH v1] colibri_imx7: boot kernel in secure mode

Stefan Agner
2019-07-24T11:32:42Z

On 2019-07-03 15:50, Igor Opaniuk wrote:
> From: Igor Opaniuk <igor.opaniuk@toradex.com>
>
> NXP downstream kernel uses legacy method to enable other cores,
> which requires kernel to run in a security mode (althought upstream kernel
> uses PSCI for this).
>
> As we're using NXP kernel in our BSPs, lets enable this by default.

I think I mentioned it internally, but just to be clear: I do not agree
with this patch.

Upstream U-Boot should be able to boot upstream Linux. I'd rather prefer
to carry that patch in our downstream fork (or set the bootm_boot_mode
environment in our boot script).

--
Stefan

>
> Signed-off-by: Igor Opaniuk <igor.opaniuk@toradex.com>
> ---
>
> include/configs/colibri_imx7.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/include/configs/colibri_imx7.h b/include/configs/colibri_imx7.h
> index 40173b18fa..5408577661 100644
> --- a/include/configs/colibri_imx7.h
> +++ b/include/configs/colibri_imx7.h
> @@ -164,7 +164,8 @@
> "${board}/flash_blk.img && source ${loadaddr}\0" \
> "setup=setenv setupargs " \
> "console=tty1 console=${console}" \
> - ",${baudrate}n8 ${memargs} consoleblank=0\0" \
> + ",${baudrate}n8 ${memargs} consoleblank=0; " \
> + "setenv bootm_boot_mode sec\0" \
> "setupdate=run setsdupdate || run setusbupdate || run setethupdate\0" \
> "setusbupdate=usb start && setenv interface usb && " \
> "fatload ${interface} 0:1 ${loadaddr} " \