Merge tag 'u-boot-amlogic-20200618' of https://gitlab.denx.de/u-boot/custodians/u...

[oweals/u-boot.git] / common / spl / spl_opensbi.c

diff --git a/common/spl/spl_opensbi.c b/common/spl/spl_opensbi.c
index 58bf2468ce3c515637d496d6754181ee2ec040c3..e88136e6f39ed8528bb5a2460624f178f06092b9 100644 (file)
--- a/common/spl/spl_opensbi.c
+++ b/common/spl/spl_opensbi.c
@@ -8,9 +8,12 @@
 #include <common.h>
 #include <cpu_func.h>
 #include <errno.h>
+#include <hang.h>
+#include <image.h>
 #include <spl.h>
 #include <asm/smp.h>
 #include <opensbi.h>
+#include <linux/libfdt.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -75,10 +78,20 @@ void spl_invoke_opensbi(struct spl_image_info *spl_image)
        opensbi_entry = (void (*)(ulong, ulong, ulong))spl_image->entry_point;
        invalidate_icache_all();
 
-#ifdef CONFIG_SMP
+#ifdef CONFIG_SPL_SMP
+       /*
+        * Start OpenSBI on all secondary harts and wait for acknowledgment.
+        *
+        * OpenSBI first relocates itself to its link address. This is done by
+        * the main hart. To make sure no hart is still running U-Boot SPL
+        * during relocation, we wait for all secondary harts to acknowledge
+        * the call-function request before entering OpenSBI on the main hart.
+        * Otherwise, code corruption can occur if the link address ranges of
+        * U-Boot SPL and OpenSBI overlap.
+        */
        ret = smp_call_function((ulong)spl_image->entry_point,
                                (ulong)spl_image->fdt_addr,
-                               (ulong)&opensbi_info, 0);
+                               (ulong)&opensbi_info, 1);
        if (ret)
                hang();
 #endif