This is easier to read than the #ifdef staircase, provides better
visibility into the memory map (alongside the other Kconfig
definitions), and allows these addresses to be reused from code.

Signed-off-by: Samuel Holland <sam...@sholland.org>
---

Changes in v2:
 - New patch for v2, split from the .dtsi changes

 arch/arm/dts/sunxi-u-boot.dtsi | 24 +++++++-----------------
 arch/arm/mach-sunxi/Kconfig    | 17 +++++++++++++++++
 2 files changed, 24 insertions(+), 17 deletions(-)

diff --git a/arch/arm/dts/sunxi-u-boot.dtsi b/arch/arm/dts/sunxi-u-boot.dtsi
index 8a6c9e901a..f38359fd42 100644
--- a/arch/arm/dts/sunxi-u-boot.dtsi
+++ b/arch/arm/dts/sunxi-u-boot.dtsi
@@ -1,15 +1,5 @@
 #include <config.h>
 
-#ifdef CONFIG_MACH_SUN50I_H6
-#define BL31_ADDR 0x104000
-#define  SCP_ADDR 0x114000
-#elif defined(CONFIG_MACH_SUN50I_H616)
-#define BL31_ADDR 0x40000000
-#else
-#define BL31_ADDR  0x44000
-#define  SCP_ADDR  0x50000
-#endif
-
 / {
        aliases {
 #ifndef CONFIG_MACH_SUNIV
@@ -64,8 +54,8 @@
                                        os = "arm-trusted-firmware";
                                        arch = "arm64";
                                        compression = "none";
-                                       load = <BL31_ADDR>;
-                                       entry = <BL31_ADDR>;
+                                       load = <CONFIG_SUNXI_BL31_BASE>;
+                                       entry = <CONFIG_SUNXI_BL31_BASE>;
 
                                        atf-bl31 {
                                                filename = "bl31.bin";
@@ -73,13 +63,13 @@
                                        };
                                };
 
-#ifdef SCP_ADDR
+#if CONFIG_SUNXI_SCP_BASE
                                scp {
                                        description = "SCP firmware";
                                        type = "firmware";
                                        arch = "or1k";
                                        compression = "none";
-                                       load = <SCP_ADDR>;
+                                       load = <CONFIG_SUNXI_SCP_BASE>;
 
                                        scp {
                                                filename = "scp.bin";
@@ -101,10 +91,10 @@
                                @config-SEQ {
                                        description = "NAME";
                                        firmware = "atf";
-#ifndef SCP_ADDR
-                                       loadables = "uboot";
-#else
+#if CONFIG_SUNXI_SCP_BASE
                                        loadables = "scp", "uboot";
+#else
+                                       loadables = "uboot";
 #endif
                                        fdt = "fdt-SEQ";
                                };
diff --git a/arch/arm/mach-sunxi/Kconfig b/arch/arm/mach-sunxi/Kconfig
index dbe6005daa..8e67f7e91d 100644
--- a/arch/arm/mach-sunxi/Kconfig
+++ b/arch/arm/mach-sunxi/Kconfig
@@ -110,6 +110,23 @@ config SUNXI_SRAM_ADDRESS
        Some newer SoCs map the boot ROM at address 0 instead and move the
        SRAM to a different address.
 
+config SUNXI_BL31_BASE
+       hex
+       default 0x00044000 if MACH_SUN50I || MACH_SUN50I_H5
+       default 0x00104000 if MACH_SUN50I_H6
+       default 0x40000000 if MACH_SUN50I_H616
+       default 0x0
+       help
+         Address where BL31 (TF-A) is loaded, or zero if BL31 is not used.
+
+config SUNXI_SCP_BASE
+       hex
+       default 0x00050000 if MACH_SUN50I || MACH_SUN50I_H5
+       default 0x00114000 if MACH_SUN50I_H6
+       default 0x0
+       help
+         Address where SCP firmware is loaded, or zero if it is not used.
+
 config SUNXI_A64_TIMER_ERRATUM
        bool
 
-- 
2.37.4

Reply via email to