Re: [PATCH] kontron-sl-mx8mm: change environment address variables

[Michael Walle]

Am 2022-02-16 09:12, schrieb Heiko Thiery:

Currently the space between kernel_addr_r and the fdt_addr_r is only 32MB.

To have enought space to load kernel images bigger than 32MB change the
varialbes to a feasable value.

variables, feasible

The new environment variables layout is based on the scheme from
"include/configs/ti_armv7_common.h".

The CONFIG_SYS_LOAD_ADDR value is set to 0x42000000.

reason is missing though, i can already tell from the patch :)

Signed-off-by: Heiko Thiery <heiko.thi...@gmail.com>
---
 configs/kontron-sl-mx8mm_defconfig |  2 +-
 include/configs/kontron-sl-mx8mm.h | 15 ++++++++++-----
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/configs/kontron-sl-mx8mm_defconfig
b/configs/kontron-sl-mx8mm_defconfig
index 940bf07a92..030e2251e7 100644
--- a/configs/kontron-sl-mx8mm_defconfig
+++ b/configs/kontron-sl-mx8mm_defconfig
@@ -20,7 +20,7 @@ CONFIG_SPL_SERIAL=y
 CONFIG_BOOTCOUNT_BOOTLIMIT=3
 CONFIG_SPL=y
 CONFIG_DISTRO_DEFAULTS=y
-CONFIG_SYS_LOAD_ADDR=0x40480000
+CONFIG_SYS_LOAD_ADDR=0x42000000
 CONFIG_FIT=y
 CONFIG_FIT_EXTERNAL_OFFSET=0x3000
 CONFIG_SPL_LOAD_FIT=y
diff --git a/include/configs/kontron-sl-mx8mm.h
b/include/configs/kontron-sl-mx8mm.h
index d1e87f97d6..0af6a9ab30 100644
--- a/include/configs/kontron-sl-mx8mm.h
+++ b/include/configs/kontron-sl-mx8mm.h
@@ -67,15 +67,20 @@

 #define FEC_QUIRK_ENET_MAC

-#define CONFIG_EXTRA_ENV_SETTINGS \
+#define ENV_MEM_LAYOUT_SETTINGS \
+       "loadaddr=" __stringify(CONFIG_SYS_LOAD_ADDR) "\0" \
        "kernel_addr_r=0x42000000\0" \
-       "fdt_addr_r=0x44000000\0" \
-       "ramdisk_addr_r=0x46400000\0" \
-       "pxefile_addr_r=0x46000000\0" \
-       "scriptaddr=0x46000000\0" \
+       "fdt_addr_r=0x48000000\0" \
+       "fdtoverlay_addr_r=0x49000000\0" \
+       "ramdisk_addr_r=0x48080000\0" \
+       "scriptaddr=0x40000000\0"\
+       "pxefile_addr_r=0x40100000\0" \

drop the \ at the end of the line.

+
+#define CONFIG_EXTRA_ENV_SETTINGS \
        "dfu_alt_info=sf 0:0=flash-bin raw 0x400 0x1f0000\0" \
        "bootdelay=3\0" \
        "hostname=" CONFIG_HOSTNAME "\0" \
+       ENV_MEM_LAYOUT_SETTINGS \
        BOOTENV

 #endif /* __KONTRON_MX8MM_CONFIG_H */

With that fixed:
Reviewed-by: Michael Walle <mich...@walle.cc>

--
-michael