Re: [PATCH v1] arm: mach-k3: common: EFI loader map memory below ram top

2024-04-11 Thread Tom Rini
On Thu, 28 Mar 2024 10:05:48 +, Vitor Soares wrote:

> During the boot, the EFI loader maps the memory from ram_top to ram_end
> as EFI_BOOT_SERVICES_DATA. When LMB does boot_fdt_add_mem_rsv_regions()
> to OPTEE, TFA, R5, and M4F DMA/memory "no-map" for the kernel it produces
> the following error message:
> 
> ERROR: reserving fdt memory region failed (addr=9cb0 size=10 flags=4)
> ERROR: reserving fdt memory region failed (addr=9cc0 size=e0 flags=4)
> ERROR: reserving fdt memory region failed (addr=9da0 size=10 flags=4)
> ERROR: reserving fdt memory region failed (addr=9db0 size=c0 flags=4)
> ERROR: reserving fdt memory region failed (addr=9e78 size=8 flags=4)
> ERROR: reserving fdt memory region failed (addr=9e80 size=180 flags=4)
> 
> [...]

Applied to u-boot/master, thanks!

-- 
Tom




[PATCH v1] arm: mach-k3: common: EFI loader map memory below ram top

2024-03-28 Thread Vitor Soares
From: Vitor Soares 

During the boot, the EFI loader maps the memory from ram_top to ram_end
as EFI_BOOT_SERVICES_DATA. When LMB does boot_fdt_add_mem_rsv_regions()
to OPTEE, TFA, R5, and M4F DMA/memory "no-map" for the kernel it produces
the following error message:

ERROR: reserving fdt memory region failed (addr=9cb0 size=10 flags=4)
ERROR: reserving fdt memory region failed (addr=9cc0 size=e0 flags=4)
ERROR: reserving fdt memory region failed (addr=9da0 size=10 flags=4)
ERROR: reserving fdt memory region failed (addr=9db0 size=c0 flags=4)
ERROR: reserving fdt memory region failed (addr=9e78 size=8 flags=4)
ERROR: reserving fdt memory region failed (addr=9e80 size=180 flags=4)

To avoid this, don't flag with EFI_BOOT_SERVICES_DATA the memory from
ram_top to ram_end by the EFI loader.

Signed-off-by: Vitor Soares 
---

Similar fix for the same issue can be found at
https://lore.kernel.org/all/20240308101230.2595220-1-patrice.chot...@foss.st.com/

 arch/arm/mach-k3/common.c | 12 
 1 file changed, 12 insertions(+)

diff --git a/arch/arm/mach-k3/common.c b/arch/arm/mach-k3/common.c
index b54f2ddc47..80e89b2a16 100644
--- a/arch/arm/mach-k3/common.c
+++ b/arch/arm/mach-k3/common.c
@@ -24,6 +24,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -607,3 +608,14 @@ void setup_qos(void)
writel(qos_data[i].val, (uintptr_t)qos_data[i].reg);
 }
 #endif
+
+void efi_add_known_memory(void)
+{
+   if (IS_ENABLED(CONFIG_EFI_LOADER))
+   /*
+* Memory over ram_top can be used by various firmware
+* Declare to EFI only memory area below ram_top
+*/
+   efi_add_memory_map(gd->ram_base, gd->ram_top - gd->ram_base,
+  EFI_CONVENTIONAL_MEMORY);
+}
-- 
2.34.1