For Apollo Lake SPL is run from CAR (cache-as-RAM) which is in a different
location from where SPL must be placed in ROM. In other words, although
SPL runs before SDRAM is set up, it is not execute-in-place (XIP).

Add a Kconfig option for the ROM position.

Signed-off-by: Simon Glass <s...@chromium.org>
Reviewed-by: Bin Meng <bmeng...@gmail.com>
---

Changes in v5: None
Changes in v4:
- apollolake -> Apollo Lake

Changes in v3:
- Add SPL condition to the option

Changes in v2: None

 arch/x86/Kconfig         | 5 +++++
 arch/x86/dts/u-boot.dtsi | 4 ++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index e105fda2f2..ae96e69f86 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -904,4 +904,9 @@ config X86_OFFSET_U_BOOT
        depends on HAVE_SYS_TEXT_BASE
        default SYS_TEXT_BASE
 
+config X86_OFFSET_SPL
+       hex "Offset of SPL in ROM image"
+       depends on SPL && X86
+       default SPL_TEXT_BASE
+
 endmenu
diff --git a/arch/x86/dts/u-boot.dtsi b/arch/x86/dts/u-boot.dtsi
index d84c64880a..fad3e7c951 100644
--- a/arch/x86/dts/u-boot.dtsi
+++ b/arch/x86/dts/u-boot.dtsi
@@ -45,7 +45,7 @@
        };
 #endif
        u-boot-spl {
-               offset = <CONFIG_SPL_TEXT_BASE>;
+               offset = <CONFIG_X86_OFFSET_SPL>;
        };
        u-boot-spl-dtb {
        };
@@ -54,7 +54,7 @@
        };
 #elif defined(CONFIG_SPL)
        u-boot-spl-with-ucode-ptr {
-               offset = <CONFIG_SPL_TEXT_BASE>;
+               offset = <CONFIG_X86_OFFSET_SPL>;
        };
        u-boot-dtb-with-ucode2 {
                type = "u-boot-dtb-with-ucode";
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to