diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2023-03-07 10:34:23 +0100 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2023-03-10 11:20:08 +0100 |
commit | a60135d20d268a467bef8a2c15e52560a7825a9e (patch) | |
tree | b11278851ac745e625a92836636b2ece36d83d5c | |
parent | f913e2170a2ba1a1c9a1b833e9a4e8e00ffe24db (diff) | |
download | barebox-a60135d20d268a467bef8a2c15e52560a7825a9e.tar.gz |
ARM: i.MX8M: add option to install OP-TEE
While infrastructure for installing OP-TEE was already there, no
upstream boards made use of it. Let's have OP-TEE installation happen in
generic code, so it may be enabled for any upstream board.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Link: https://lore.barebox.org/20230307093426.1551193-1-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
-rw-r--r-- | arch/arm/mach-imx/atf.c | 50 | ||||
-rw-r--r-- | firmware/.gitignore | 1 | ||||
-rw-r--r-- | firmware/Kconfig | 12 | ||||
-rw-r--r-- | firmware/Makefile | 9 | ||||
-rw-r--r-- | include/mach/imx/xload.h | 16 |
5 files changed, 74 insertions, 14 deletions
diff --git a/arch/arm/mach-imx/atf.c b/arch/arm/mach-imx/atf.c index 335cc7713f..eaf436db07 100644 --- a/arch/arm/mach-imx/atf.c +++ b/arch/arm/mach-imx/atf.c @@ -7,6 +7,9 @@ #include <mach/imx/generic.h> #include <mach/imx/xload.h> #include <mach/imx/romapi.h> +#include <mach/imx/esdctl.h> +#include <asm-generic/memory_layout.h> +#include <asm/barebox-arm.h> #include <mach/imx/imx8m-regs.h> #include <soc/fsl/fsl_udc.h> #include <soc/fsl/caam.h> @@ -134,9 +137,17 @@ void imx8mm_load_bl33(void *bl33) __noreturn void imx8mm_load_and_start_image_via_tfa(void) { - imx8mm_save_bootrom_log(); - imx8mm_load_bl33((void *)MX8M_ATF_BL33_BASE_ADDR); - imx8mm_load_and_start_tfa(imx8mm_bl31_bin); + void *bl33 = (void *)MX8M_ATF_BL33_BASE_ADDR; + void *endmem = (void *)MX8M_DDR_CSD1_BASE_ADDR + + imx8m_barebox_earlymem_size(32); + + imx8m_save_bootrom_log(__arm_mem_scratch(endmem)); + imx8mm_load_bl33(bl33); + + if (IS_ENABLED(CONFIG_FIRMWARE_IMX8MM_OPTEE)) + imx8m_load_and_start_optee_via_tfa(imx8mm, endmem - OPTEE_SIZE, bl33); + else + imx8mm_load_and_start_tfa(imx8mm_bl31_bin); } void imx8mp_load_bl33(void *bl33) @@ -169,13 +180,22 @@ void imx8mp_load_bl33(void *bl33) memcpy(bl33, __image_start, barebox_pbl_size); } -void imx8mp_load_and_start_image_via_tfa(void) +__noreturn void imx8mp_load_and_start_image_via_tfa(void) { - imx8mp_save_bootrom_log(); - imx8mp_load_bl33((void *)MX8M_ATF_BL33_BASE_ADDR); - imx8mp_load_and_start_tfa(imx8mp_bl31_bin); + void *bl33 = (void *)MX8M_ATF_BL33_BASE_ADDR; + void *endmem = (void *)MX8M_DDR_CSD1_BASE_ADDR + + imx8m_barebox_earlymem_size(32); + + imx8m_save_bootrom_log(__arm_mem_scratch(endmem)); + imx8mp_load_bl33(bl33); + + if (IS_ENABLED(CONFIG_FIRMWARE_IMX8MP_OPTEE)) + imx8m_load_and_start_optee_via_tfa(imx8mp, endmem - OPTEE_SIZE, bl33); + else + imx8mp_load_and_start_tfa(imx8mp_bl31_bin); } + void imx8mn_load_bl33(void *bl33) { enum bootsource src; @@ -206,9 +226,17 @@ void imx8mn_load_bl33(void *bl33) memcpy(bl33, __image_start, barebox_pbl_size); } -void imx8mn_load_and_start_image_via_tfa(void) +__noreturn void imx8mn_load_and_start_image_via_tfa(void) { - imx8mn_save_bootrom_log(); - imx8mn_load_bl33((void *)MX8M_ATF_BL33_BASE_ADDR); - imx8mn_load_and_start_tfa(imx8mn_bl31_bin); + void *bl33 = (void *)MX8M_ATF_BL33_BASE_ADDR; + void *endmem = (void *)MX8M_DDR_CSD1_BASE_ADDR + + imx8m_barebox_earlymem_size(16); + + imx8m_save_bootrom_log(__arm_mem_scratch(endmem)); + imx8mn_load_bl33(bl33); + + if (IS_ENABLED(CONFIG_FIRMWARE_IMX8MN_OPTEE)) + imx8m_load_and_start_optee_via_tfa(imx8mn, endmem - OPTEE_SIZE, bl33); + else + imx8mn_load_and_start_tfa(imx8mn_bl31_bin); } diff --git a/firmware/.gitignore b/firmware/.gitignore index 4f3e56d8b7..8f9a956eb6 100644 --- a/firmware/.gitignore +++ b/firmware/.gitignore @@ -1,3 +1,4 @@ # SPDX-License-Identifier: GPL-2.0-only *.bin +*.bin-optee diff --git a/firmware/Kconfig b/firmware/Kconfig index 1a5e1bf514..a12a8f3044 100644 --- a/firmware/Kconfig +++ b/firmware/Kconfig @@ -24,6 +24,18 @@ config FIRMWARE_IMX8MP_ATF config FIRMWARE_IMX8MQ_ATF bool +config FIRMWARE_IMX8MM_OPTEE + bool "install OP-TEE on i.MX8MM boards" + depends on FIRMWARE_IMX8MM_ATF && PBL_OPTEE + +config FIRMWARE_IMX8MN_OPTEE + bool "install OP-TEE on i.MX8MN boards" + depends on FIRMWARE_IMX8MN_ATF && PBL_OPTEE + +config FIRMWARE_IMX8MP_OPTEE + bool "install OP-TEE on i.MX8MP boards" + depends on FIRMWARE_IMX8MP_ATF && PBL_OPTEE + config FIRMWARE_CCBV2_OPTEE bool depends on MACH_WEBASTO_CCBV2 && PBL_OPTEE diff --git a/firmware/Makefile b/firmware/Makefile index da341871d6..984192f030 100644 --- a/firmware/Makefile +++ b/firmware/Makefile @@ -12,10 +12,13 @@ firmware-$(CONFIG_FIRMWARE_IMX_DDR4_PMU_TRAIN) += \ ddr4_imem_1d.bin \ ddr4_imem_2d.bin -firmware-$(CONFIG_FIRMWARE_IMX8MM_ATF) += imx8mm-bl31.bin -firmware-$(CONFIG_FIRMWARE_IMX8MN_ATF) += imx8mn-bl31.bin -firmware-$(CONFIG_FIRMWARE_IMX8MP_ATF) += imx8mp-bl31.bin +firmware-$(CONFIG_FIRMWARE_IMX8MM_ATF) += imx8mm-bl31.bin$(if $(CONFIG_FIRMWARE_IMX8MM_OPTEE),-optee,) +firmware-$(CONFIG_FIRMWARE_IMX8MN_ATF) += imx8mn-bl31.bin$(if $(CONFIG_FIRMWARE_IMX8MN_OPTEE),-optee,) +firmware-$(CONFIG_FIRMWARE_IMX8MP_ATF) += imx8mp-bl31.bin$(if $(CONFIG_FIRMWARE_IMX8MP_OPTEE),-optee,) firmware-$(CONFIG_FIRMWARE_IMX8MQ_ATF) += imx8mq-bl31.bin +fw-external-$(CONFIG_FIRMWARE_IMX8MM_OPTEE) += imx8mm-bl32.bin +fw-external-$(CONFIG_FIRMWARE_IMX8MN_OPTEE) += imx8mn-bl32.bin +fw-external-$(CONFIG_FIRMWARE_IMX8MP_OPTEE) += imx8mp-bl32.bin firmware-$(CONFIG_ARCH_RK3568) += rk3568-bl31.bin firmware-$(CONFIG_ARCH_RK3568_OPTEE) += rk3568-op-tee.bin firmware-$(CONFIG_ARCH_RK3399) += rk3399-bl31.bin diff --git a/include/mach/imx/xload.h b/include/mach/imx/xload.h index 86270b03dc..88757c7ebf 100644 --- a/include/mach/imx/xload.h +++ b/include/mach/imx/xload.h @@ -5,6 +5,7 @@ #include <linux/compiler.h> #include <linux/types.h> +#include <mach/imx/tzasc.h> int imx53_nand_start_image(void); int imx6_spi_load_image(int instance, unsigned int flash_offset, void *buf, int len); @@ -42,4 +43,19 @@ struct imx_scratch_space *__imx8m_scratch_space(int ddr_buswidth); #define imx8mn_scratch_space() __imx8m_scratch_space(16) #define imx8mp_scratch_space() __imx8m_scratch_space(32) +#define imx8m_load_and_start_optee_via_tfa(soc, bl32, bl33) \ +do { \ + size_t bl32_size; \ + void *bl32_image; \ + \ + soc##_tzc380_init(); \ + get_builtin_firmware_ext(soc##_bl32_bin, \ + bl33, &bl32_image, \ + &bl32_size); \ + \ + memcpy(bl32, bl32_image, bl32_size); \ + \ + soc##_load_and_start_tfa(soc##_bl31_bin_optee); \ +} while (0) + #endif /* __MACH_IMX_XLOAD_H */ |