[PATCH] ARM: i.MX8M: bootrom: access OCRAM directly if running in EL3
Ahmad Fatoum
a.fatoum at pengutronix.de
Tue Jan 10 23:51:14 PST 2023
It's straight-forward to hack barebox PBL to skip ATF installation
and to start barebox proper in EL3. This can be useful for debugging
and may in future be just a Kconfig option like we now have with
Rockchip.
In such a configuration, we don't need to copy the ROM log out of OCRAM,
because there's no ATF installed there that might overwrite it.
Therefore, just directly access the event log pointer in BootROM if
running in EL3. As 0x9e0 is in the first zero page, we use the function
in zero_page.h to temporarily disable traps in the zero page. We don't
need to do that in PBL though as even early MMU will be enabled later.
Signed-off-by: Ahmad Fatoum <a.fatoum at pengutronix.de>
---
I wanted to send this along with an option to disable ATF via Kconfig,
but that will need us to touch all boards and maybe replace current_el()
checks with comparison whether we run in DRAM to avoid renewed setup.
I may look later into this, but this patch may still be useful to have.
---
arch/arm/mach-imx/Makefile | 4 ++--
arch/arm/mach-imx/bootrom-cmd.c | 14 ++++++++++++--
arch/arm/mach-imx/include/mach/romapi.h | 4 ++++
arch/arm/mach-imx/romapi.c | 24 ++++++++++++++++--------
include/zero_page.h | 2 +-
5 files changed, 35 insertions(+), 13 deletions(-)
diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile
index cc834fed7be7..5d70e79b57f1 100644
--- a/arch/arm/mach-imx/Makefile
+++ b/arch/arm/mach-imx/Makefile
@@ -18,8 +18,8 @@ lwl-$(CONFIG_ARCH_IMX6) += imx6-mmdc.o
obj-$(CONFIG_ARCH_IMX7) += imx7.o
obj-$(CONFIG_ARCH_VF610) += vf610.o
obj-pbl-$(CONFIG_ARCH_IMX8M) += imx8m.o
-lwl-$(CONFIG_ARCH_IMX8M) += atf.o romapi.o
-obj-pbl-$(CONFIG_ARCH_IMX8M) += tzasc.o
+lwl-$(CONFIG_ARCH_IMX8M) += atf.o
+obj-pbl-$(CONFIG_ARCH_IMX8M) += romapi.o tzasc.o
obj-$(CONFIG_IMX_IIM) += iim.o
obj-$(CONFIG_NAND_IMX) += nand.o
lwl-$(CONFIG_ARCH_IMX_EXTERNAL_BOOT_NAND) += external-nand-boot.o
diff --git a/arch/arm/mach-imx/bootrom-cmd.c b/arch/arm/mach-imx/bootrom-cmd.c
index 0238d09b169f..c891fd2e0640 100644
--- a/arch/arm/mach-imx/bootrom-cmd.c
+++ b/arch/arm/mach-imx/bootrom-cmd.c
@@ -8,6 +8,7 @@
#include <linux/bitfield.h>
#include <mach/imx8m-regs.h>
#include <mach/xload.h>
+#include <mach/romapi.h>
#include <asm/barebox-arm.h>
/* i.MX7 and later ID field is swapped compared to i.MX6 */
@@ -51,6 +52,9 @@ static int imx8m_bootrom_decode_log(const u32 *rom_log)
{
int i;
+ if (!rom_log)
+ return -ENODATA;
+
for (i = 0; i < 128; i++) {
u8 event_id = FIELD_GET(ROM_EVENT_FORMAT_V1_ID, rom_log[i]);
u8 event_id_idx = FIELD_GET(ROM_EVENT_FORMAT_V1_ID_IDX, rom_log[i]);
@@ -178,11 +182,17 @@ static int imx8m_bootrom_decode_log(const u32 *rom_log)
static int do_bootrom(int argc, char *argv[])
{
- const struct imx_scratch_space *scratch = arm_mem_scratch_get();
- const u32 *rom_log_addr = scratch->bootrom_log;
+ const u32 *rom_log_addr;
bool log = false;
int ret, opt;
+ if (current_el() == 3) {
+ rom_log_addr = __imx8m_get_bootrom_log();
+ } else {
+ const struct imx_scratch_space *scratch = arm_mem_scratch_get();
+ rom_log_addr = scratch->bootrom_log;
+ }
+
while((opt = getopt(argc, argv, "la:")) > 0) {
switch(opt) {
case 'a':
diff --git a/arch/arm/mach-imx/include/mach/romapi.h b/arch/arm/mach-imx/include/mach/romapi.h
index d22ba7259dd0..0aecfa7f5c15 100644
--- a/arch/arm/mach-imx/include/mach/romapi.h
+++ b/arch/arm/mach-imx/include/mach/romapi.h
@@ -3,6 +3,7 @@
#define __MACH_IMX_ROMAPI_H
#include <mach/xload.h>
+#include <linux/types.h>
struct rom_api {
u16 ver;
@@ -40,6 +41,9 @@ int imx8mn_bootrom_load_image(void);
/* only call after DRAM has been configured */
void imx8m_save_bootrom_log(void *dst);
+/* only usable in EL3 */
+const u32 *__imx8m_get_bootrom_log(void);
+
#define imx8mq_save_bootrom_log() imx8m_save_bootrom_log(imx8mq_scratch_space())
#define imx8mm_save_bootrom_log() imx8m_save_bootrom_log(imx8mm_scratch_space())
#define imx8mn_save_bootrom_log() imx8m_save_bootrom_log(imx8mn_scratch_space())
diff --git a/arch/arm/mach-imx/romapi.c b/arch/arm/mach-imx/romapi.c
index 5885fb698ecd..74cc0119afdb 100644
--- a/arch/arm/mach-imx/romapi.c
+++ b/arch/arm/mach-imx/romapi.c
@@ -7,6 +7,7 @@
#include <mach/romapi.h>
#include <mach/atf.h>
#include <mach/imx8m-regs.h>
+#include <zero_page.h>
static int imx8m_bootrom_load(struct rom_api *rom_api, void *adr, size_t size)
{
@@ -50,23 +51,30 @@ int imx8mn_bootrom_load_image(void)
return imx8mp_bootrom_load_image();
}
-void imx8m_save_bootrom_log(void *dest)
+const u32 *__imx8m_get_bootrom_log(void)
{
ulong rom_log_addr;
- if (!IS_ENABLED(CONFIG_IMX_SAVE_BOOTROM_LOG)) {
- pr_debug("skipping bootrom log saving\n");
- return;
- }
-
- rom_log_addr = *(u32 *)0x9e0;
+ zero_page_access();
+ rom_log_addr = readl(IOMEM(0x9e0));
+ zero_page_faulting();
if (rom_log_addr < MX8M_OCRAM_BASE_ADDR ||
rom_log_addr >= MX8M_OCRAM_BASE_ADDR + MX8M_OCRAM_MAX_SIZE ||
rom_log_addr & 0x3) {
pr_warn("No BootROM log found at address 0x%08lx\n", rom_log_addr);
+ return NULL;
+ }
+
+ return (u32 *)rom_log_addr;
+}
+
+void imx8m_save_bootrom_log(void *dest)
+{
+ if (!IS_ENABLED(CONFIG_IMX_SAVE_BOOTROM_LOG)) {
+ pr_debug("skipping bootrom log saving\n");
return;
}
- memcpy(dest, (u32 *)rom_log_addr, 128 * sizeof(u32));
+ memcpy(dest, __imx8m_get_bootrom_log(), 128 * sizeof(u32));
}
diff --git a/include/zero_page.h b/include/zero_page.h
index 519e65be7628..26a12246f155 100644
--- a/include/zero_page.h
+++ b/include/zero_page.h
@@ -4,7 +4,7 @@
#include <common.h>
-#if defined CONFIG_ARCH_HAS_ZERO_PAGE && defined CONFIG_MMU
+#if defined CONFIG_ARCH_HAS_ZERO_PAGE && defined CONFIG_MMU && !defined __PBL__
/*
* zero_page_faulting - fault when accessing the zero page
--
2.30.2
More information about the barebox
mailing list