All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
@ 2025-07-12  8:26 Martin Herren
  0 siblings, 0 replies; 12+ messages in thread
From: Martin Herren @ 2025-07-12  8:26 UTC (permalink / raw)
  To: u-boot
  Cc: Martin Herren, Andreas Schwab, Anup Patel, Atish Patra, Bin Meng,
	Chia-Wei, Wang, Conor Dooley, Cyril Jean, E Shattow, Green Wan,
	Hal Feng, Heinrich Schuchardt, Ilias Apalodimas, Jaehoon Chung,
	Jerome Forissier, Junhui Liu, Kongyang Liu, Leo Yu-Chi Liang,
	Maksim Kiselev, Marek Vasut, Martin Schwan, Mattijs Korpershoek,
	Michal Simek, Minda Chen, Padmarao Begari, Palmer Dabbelt,
	Paul Walmsley, Peter Robinson, Rick Chen, Sean Anderson,
	Simon Glass, Sughosh Ganu, Sumit Garg, Thomas Bonnefille,
	Tom Rini, Wei Fu, Yao Zi, Yixun Lan, Yuri Zaporozhets

[-- Attachment #1: Type: text/plain, Size: 2286 bytes --]


The most commonly used SYS_BOOTM_LEN for riscv is 0x4000000 which is
used in 25 of the current defconfigs.

The previous default config value of 0x800000 was only used in 8
defconfigs.

This patch sets the default to this value.

Command used to get the stats of the used values on RISCV defconfigs:

```
grep -l "CONFIG_RISCV=y" configs/* | \
 xargs -I {} sh -c "grep -H CONFIG_SYS_BOOTM_LEN {} || echo {}:DEFAULT" | \
 cut -d ':' -f 2 | sort | uniq -c
```


Martin Herren (3):
  riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
  riscv: Set SYS_BOOTM_LEN default to 0x4000000
  riscv: Remove default SYS_BOOTM_LEN from defconfig

 boot/Kconfig                            | 2 +-
 configs/ae350_rv32_defconfig            | 1 -
 configs/ae350_rv32_falcon_defconfig     | 1 -
 configs/ae350_rv32_falcon_xip_defconfig | 1 -
 configs/ae350_rv32_spl_defconfig        | 1 -
 configs/ae350_rv32_spl_xip_defconfig    | 1 -
 configs/ae350_rv32_xip_defconfig        | 1 -
 configs
/ae350_rv64_defconfig            | 1 -
 configs/ae350_rv64_falcon_defconfig     | 1 -
 configs/ae350_rv64_falcon_xip_defconfig | 1 -
 configs/ae350_rv64_spl_defconfig        | 1 -
 configs/ae350_rv64_spl_xip_defconfig    | 1 -
 configs/ae350_rv64_xip_defconfig        | 1 -
 configs/ibex-ast2700_defconfig          | 1 -
 configs/k230_canmv_defconfig            | 1 +
 configs/microchip_mpfs_icicle_defconfig | 1 +
 configs/milkv_duo_defconfig             | 1 -
 configs/qemu-riscv32_defconfig          | 1 -
 configs/qemu-riscv32_smode_defconfig    | 1 -
 configs/qemu-riscv32_spl_defconfig      | 1 -
 configs/qemu-riscv64_defconfig          | 1 -
 configs/qemu-riscv64_smode_defconfig    | 1 -
 configs/qemu-riscv64_spl_defconfig      | 1 -
 configs/sifive_unleashed_defconfig      | 1 -
 configs/sifive_unmatched_defconfig      | 1 -
 configs/sipeed_licheerv_nano_defconfig  | 1 -
 configs/sipeed_maix_bitm_defconfig      | 1 +
 configs/sipeed_maix_smode_defconfig     | 1 +

 configs/starfive_visionfive2_defconfig  | 1 -
 configs/th1520_lpi4a_defconfig          | 1 -
 configs/xilinx_mbv32_defconfig          | 1 +
 configs/xilinx_mbv32_smode_defconfig    | 1 +
 configs/xilinx_mbv64_defconfig          | 1 +
 configs/xilinx_mbv64_smode_defconfig    | 1 +
 34 files changed, 9 insertions(+), 26 deletions(-)


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 322 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
@ 2025-07-14 13:33 Martin Herren
  2025-07-14 13:33 ` [PATCH 1/3] riscv: Set SYS_BOOTM_LEN explicitly to 0x800000 Martin Herren
                   ` (3 more replies)
  0 siblings, 4 replies; 12+ messages in thread
From: Martin Herren @ 2025-07-14 13:33 UTC (permalink / raw)
  To: u-boot
  Cc: Martin Herren, Andreas Schwab, Anup Patel, Atish Patra, Bin Meng,
	Chia-Wei, Wang, Conor Dooley, Cyril Jean, E Shattow, Green Wan,
	Hal Feng, Heinrich Schuchardt, Ilias Apalodimas, Jerome Forissier,
	Junhui Liu, Kongyang Liu, Leo Yu-Chi Liang, Maksim Kiselev,
	Marek Vasut, Martin Schwan, Mattijs Korpershoek, Michal Simek,
	Minda Chen, Padmarao Begari, Palmer Dabbelt, Paul Walmsley,
	Peter Robinson, Rick Chen, Sean Anderson, Simon Glass, Sumit Garg,
	Thomas Bonnefille, Tom Rini, Wei Fu, Yao Zi, Yixun Lan,
	Yuri Zaporozhets


The most commonly used SYS_BOOTM_LEN for riscv is 0x4000000 which is
used in 25 of the current defconfigs.

The previous default config value of 0x800000 was only used in 8
defconfigs.

This patch sets the default to this value.

Command used to get the stats of the used values on RISCV defconfigs:

```
grep -l "CONFIG_RISCV=y" configs/* | \
 xargs -I {} sh -c "grep -H CONFIG_SYS_BOOTM_LEN {} || echo {}:DEFAULT" | \
 cut -d ':' -f 2 | sort | uniq -c
```

Changes in v2:
- Resubmit using another mail provider to prevent mangling and signature
  attachement.

Martin Herren (3):
  riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
  riscv: Set SYS_BOOTM_LEN default to 0x4000000
  riscv: Remove default SYS_BOOTM_LEN from defconfig

 boot/Kconfig                            | 2 +-
 configs/ae350_rv32_defconfig            | 1 -
 configs/ae350_rv32_falcon_defconfig     | 1 -
 configs/ae350_rv32_falcon_xip_defconfig | 1 -
 configs/ae350_rv32_spl_defconfig        | 1 -
 configs/ae350_rv32_spl_xip_defconfig    | 1 -
 configs/ae350_rv32_xip_defconfig        | 1 -
 configs/ae350_rv64_defconfig            | 1 -
 configs/ae350_rv64_falcon_defconfig     | 1 -
 configs/ae350_rv64_falcon_xip_defconfig | 1 -
 configs/ae350_rv64_spl_defconfig        | 1 -
 configs/ae350_rv64_spl_xip_defconfig    | 1 -
 configs/ae350_rv64_xip_defconfig        | 1 -
 configs/ibex-ast2700_defconfig          | 1 -
 configs/k230_canmv_defconfig            | 1 +
 configs/microchip_mpfs_icicle_defconfig | 1 +
 configs/milkv_duo_defconfig             | 1 -
 configs/qemu-riscv32_defconfig          | 1 -
 configs/qemu-riscv32_smode_defconfig    | 1 -
 configs/qemu-riscv32_spl_defconfig      | 1 -
 configs/qemu-riscv64_defconfig          | 1 -
 configs/qemu-riscv64_smode_defconfig    | 1 -
 configs/qemu-riscv64_spl_defconfig      | 1 -
 configs/sifive_unleashed_defconfig      | 1 -
 configs/sifive_unmatched_defconfig      | 1 -
 configs/sipeed_licheerv_nano_defconfig  | 1 -
 configs/sipeed_maix_bitm_defconfig      | 1 +
 configs/sipeed_maix_smode_defconfig     | 1 +
 configs/starfive_visionfive2_defconfig  | 1 -
 configs/th1520_lpi4a_defconfig          | 1 -
 configs/xilinx_mbv32_defconfig          | 1 +
 configs/xilinx_mbv32_smode_defconfig    | 1 +
 configs/xilinx_mbv64_defconfig          | 1 +
 configs/xilinx_mbv64_smode_defconfig    | 1 +
 34 files changed, 9 insertions(+), 26 deletions(-)


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/3] riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
  2025-07-14 13:33 [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Martin Herren
@ 2025-07-14 13:33 ` Martin Herren
  2025-07-16 10:33   ` Conor Dooley
  2025-07-14 13:33 ` [PATCH 2/3] riscv: Set SYS_BOOTM_LEN default to 0x4000000 Martin Herren
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 12+ messages in thread
From: Martin Herren @ 2025-07-14 13:33 UTC (permalink / raw)
  To: u-boot
  Cc: Martin Herren, Conor Dooley, Cyril Jean, Heinrich Schuchardt,
	Ilias Apalodimas, Jerome Forissier, Junhui Liu, Leo Yu-Chi Liang,
	Marek Vasut, Michal Simek, Padmarao Begari, Sean Anderson,
	Simon Glass, Sumit Garg, Tom Rini

For all riscv defconfigs that use the current default value.

This is done in provision of changing the default value to the most
common used value of 0x4000000.

Signed-off-by: Martin Herren <sputnik@on-the-web.ch>
---

(no changes since v1)

 configs/k230_canmv_defconfig            | 1 +
 configs/microchip_mpfs_icicle_defconfig | 1 +
 configs/sipeed_maix_bitm_defconfig      | 1 +
 configs/sipeed_maix_smode_defconfig     | 1 +
 configs/xilinx_mbv32_defconfig          | 1 +
 configs/xilinx_mbv32_smode_defconfig    | 1 +
 configs/xilinx_mbv64_defconfig          | 1 +
 configs/xilinx_mbv64_smode_defconfig    | 1 +
 8 files changed, 8 insertions(+)

diff --git a/configs/k230_canmv_defconfig b/configs/k230_canmv_defconfig
index 47fa1add2a9..ed49a42b6fc 100644
--- a/configs/k230_canmv_defconfig
+++ b/configs/k230_canmv_defconfig
@@ -4,6 +4,7 @@ CONFIG_NR_DRAM_BANKS=1
 CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80400000
 CONFIG_DEFAULT_DEVICE_TREE="k230-canmv"
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0xc000000
 CONFIG_TARGET_K230_CANMV=y
 CONFIG_ARCH_RV64I=y
diff --git a/configs/microchip_mpfs_icicle_defconfig b/configs/microchip_mpfs_icicle_defconfig
index bb110225488..8616780f2b4 100644
--- a/configs/microchip_mpfs_icicle_defconfig
+++ b/configs/microchip_mpfs_icicle_defconfig
@@ -5,6 +5,7 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
 CONFIG_ENV_SIZE=0x2000
 CONFIG_DEFAULT_DEVICE_TREE="microchip/mpfs-icicle-kit"
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SYS_MEM_TOP_HIDE=0x400000
 CONFIG_TARGET_MICROCHIP_ICICLE=y
diff --git a/configs/sipeed_maix_bitm_defconfig b/configs/sipeed_maix_bitm_defconfig
index b2e21c7d7ae..c21cdace328 100644
--- a/configs/sipeed_maix_bitm_defconfig
+++ b/configs/sipeed_maix_bitm_defconfig
@@ -5,6 +5,7 @@ CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x803fffff
 CONFIG_ENV_SIZE=0x1000
 CONFIG_ENV_OFFSET=0xfff000
 CONFIG_ENV_SECT_SIZE=0x1000
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0x80000000
 CONFIG_SF_DEFAULT_BUS=3
 CONFIG_TARGET_SIPEED_MAIX=y
diff --git a/configs/sipeed_maix_smode_defconfig b/configs/sipeed_maix_smode_defconfig
index d838b252d53..3c77e40e47e 100644
--- a/configs/sipeed_maix_smode_defconfig
+++ b/configs/sipeed_maix_smode_defconfig
@@ -6,6 +6,7 @@ CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x803fffff
 CONFIG_ENV_SIZE=0x1000
 CONFIG_ENV_OFFSET=0xfff000
 CONFIG_ENV_SECT_SIZE=0x1000
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0x80000000
 CONFIG_SF_DEFAULT_BUS=3
 CONFIG_TARGET_SIPEED_MAIX=y
diff --git a/configs/xilinx_mbv32_defconfig b/configs/xilinx_mbv32_defconfig
index 7dde2fc0a8f..dfcc51cb8e6 100644
--- a/configs/xilinx_mbv32_defconfig
+++ b/configs/xilinx_mbv32_defconfig
@@ -8,6 +8,7 @@ CONFIG_DEFAULT_DEVICE_TREE="xilinx-mbv32"
 CONFIG_SPL_STACK=0x80200000
 CONFIG_SPL_BSS_START_ADDR=0x84000000
 CONFIG_SPL_BSS_MAX_SIZE=0x80000
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL_SIZE_LIMIT=0x40000
 CONFIG_SPL=y
diff --git a/configs/xilinx_mbv32_smode_defconfig b/configs/xilinx_mbv32_smode_defconfig
index b96b02c6125..0b14c7ce4c0 100644
--- a/configs/xilinx_mbv32_smode_defconfig
+++ b/configs/xilinx_mbv32_smode_defconfig
@@ -8,6 +8,7 @@ CONFIG_DEFAULT_DEVICE_TREE="xilinx-mbv32"
 CONFIG_SPL_STACK=0x80200000
 CONFIG_SPL_BSS_START_ADDR=0x84000000
 CONFIG_SPL_BSS_MAX_SIZE=0x80000
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL_SIZE_LIMIT=0x40000
 CONFIG_SPL=y
diff --git a/configs/xilinx_mbv64_defconfig b/configs/xilinx_mbv64_defconfig
index 77fcf4d6865..a437895b5ab 100644
--- a/configs/xilinx_mbv64_defconfig
+++ b/configs/xilinx_mbv64_defconfig
@@ -8,6 +8,7 @@ CONFIG_DEFAULT_DEVICE_TREE="xilinx-mbv64"
 CONFIG_SPL_STACK=0x80200000
 CONFIG_SPL_BSS_START_ADDR=0x84000000
 CONFIG_SPL_BSS_MAX_SIZE=0x80000
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL_SIZE_LIMIT=0x40000
 CONFIG_SPL=y
diff --git a/configs/xilinx_mbv64_smode_defconfig b/configs/xilinx_mbv64_smode_defconfig
index e53c0771baf..87d1d17d5b9 100644
--- a/configs/xilinx_mbv64_smode_defconfig
+++ b/configs/xilinx_mbv64_smode_defconfig
@@ -8,6 +8,7 @@ CONFIG_DEFAULT_DEVICE_TREE="xilinx-mbv64"
 CONFIG_SPL_STACK=0x80200000
 CONFIG_SPL_BSS_START_ADDR=0x84000000
 CONFIG_SPL_BSS_MAX_SIZE=0x80000
+CONFIG_SYS_BOOTM_LEN=0x800000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL_SIZE_LIMIT=0x40000
 CONFIG_SPL=y

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/3] riscv: Set SYS_BOOTM_LEN default to 0x4000000
  2025-07-14 13:33 [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Martin Herren
  2025-07-14 13:33 ` [PATCH 1/3] riscv: Set SYS_BOOTM_LEN explicitly to 0x800000 Martin Herren
@ 2025-07-14 13:33 ` Martin Herren
  2025-07-14 13:33 ` [PATCH 3/3] riscv: Remove default SYS_BOOTM_LEN from defconfig Martin Herren
  2025-07-15 11:27 ` [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Michal Simek
  3 siblings, 0 replies; 12+ messages in thread
From: Martin Herren @ 2025-07-14 13:33 UTC (permalink / raw)
  To: u-boot
  Cc: Martin Herren, Heinrich Schuchardt, Ilias Apalodimas, Marek Vasut,
	Martin Schwan, Mattijs Korpershoek, Peter Robinson, Simon Glass,
	Tom Rini

This changes the default value to the most commonly used one among
existing defconfigs.

Signed-off-by: Martin Herren <sputnik@on-the-web.ch>
---

(no changes since v1)

 boot/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/boot/Kconfig b/boot/Kconfig
index 2ff6f003738..54ef7052c5c 100644
--- a/boot/Kconfig
+++ b/boot/Kconfig
@@ -1057,7 +1057,7 @@ config SYS_BOOTM_LEN
 	hex "Maximum size of a decompresed OS image"
 	depends on CMD_BOOTM || CMD_BOOTI || CMD_BOOTZ || \
 		LEGACY_IMAGE_FORMAT || SPL_LEGACY_IMAGE_FORMAT
-	default 0x4000000 if PPC || ARM64
+	default 0x4000000 if PPC || ARM64 || RISCV
 	default 0x1000000 if X86 || ARCH_MX6 || ARCH_MX7
 	default 0x800000
 	help

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/3] riscv: Remove default SYS_BOOTM_LEN from defconfig
  2025-07-14 13:33 [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Martin Herren
  2025-07-14 13:33 ` [PATCH 1/3] riscv: Set SYS_BOOTM_LEN explicitly to 0x800000 Martin Herren
  2025-07-14 13:33 ` [PATCH 2/3] riscv: Set SYS_BOOTM_LEN default to 0x4000000 Martin Herren
@ 2025-07-14 13:33 ` Martin Herren
  2025-07-15 11:27 ` [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Michal Simek
  3 siblings, 0 replies; 12+ messages in thread
From: Martin Herren @ 2025-07-14 13:33 UTC (permalink / raw)
  To: u-boot
  Cc: Martin Herren, Andreas Schwab, Anup Patel, Atish Patra, Bin Meng,
	Chia-Wei, Wang, E Shattow, Green Wan, Hal Feng,
	Heinrich Schuchardt, Ilias Apalodimas, Jerome Forissier,
	Kongyang Liu, Leo Yu-Chi Liang, Maksim Kiselev, Marek Vasut,
	Minda Chen, Palmer Dabbelt, Paul Walmsley, Rick Chen, Simon Glass,
	Sumit Garg, Thomas Bonnefille, Tom Rini, Wei Fu, Yao Zi,
	Yixun Lan, Yuri Zaporozhets

Remove CONFIG_SYS_BOOTM_LEN from all riscv defconfigs where the new
default value is used.

Signed-off-by: Martin Herren <sputnik@on-the-web.ch>
---

(no changes since v1)

 configs/ae350_rv32_defconfig            | 1 -
 configs/ae350_rv32_falcon_defconfig     | 1 -
 configs/ae350_rv32_falcon_xip_defconfig | 1 -
 configs/ae350_rv32_spl_defconfig        | 1 -
 configs/ae350_rv32_spl_xip_defconfig    | 1 -
 configs/ae350_rv32_xip_defconfig        | 1 -
 configs/ae350_rv64_defconfig            | 1 -
 configs/ae350_rv64_falcon_defconfig     | 1 -
 configs/ae350_rv64_falcon_xip_defconfig | 1 -
 configs/ae350_rv64_spl_defconfig        | 1 -
 configs/ae350_rv64_spl_xip_defconfig    | 1 -
 configs/ae350_rv64_xip_defconfig        | 1 -
 configs/ibex-ast2700_defconfig          | 1 -
 configs/milkv_duo_defconfig             | 1 -
 configs/qemu-riscv32_defconfig          | 1 -
 configs/qemu-riscv32_smode_defconfig    | 1 -
 configs/qemu-riscv32_spl_defconfig      | 1 -
 configs/qemu-riscv64_defconfig          | 1 -
 configs/qemu-riscv64_smode_defconfig    | 1 -
 configs/qemu-riscv64_spl_defconfig      | 1 -
 configs/sifive_unleashed_defconfig      | 1 -
 configs/sifive_unmatched_defconfig      | 1 -
 configs/sipeed_licheerv_nano_defconfig  | 1 -
 configs/starfive_visionfive2_defconfig  | 1 -
 configs/th1520_lpi4a_defconfig          | 1 -
 25 files changed, 25 deletions(-)

diff --git a/configs/ae350_rv32_defconfig b/configs/ae350_rv32_defconfig
index 467d61d73df..e87782005f1 100644
--- a/configs/ae350_rv32_defconfig
+++ b/configs/ae350_rv32_defconfig
@@ -7,7 +7,6 @@ CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xfffe80
 CONFIG_ENV_SECT_SIZE=0x1000
 CONFIG_DEFAULT_DEVICE_TREE="ae350_32"
 CONFIG_SYS_MONITOR_LEN=786432
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_TARGET_ANDES_AE350=y
 CONFIG_SYS_MONITOR_BASE=0x88000000
diff --git a/configs/ae350_rv32_falcon_defconfig b/configs/ae350_rv32_falcon_defconfig
index 66b809d2376..f1d88f8560f 100644
--- a/configs/ae350_rv32_falcon_defconfig
+++ b/configs/ae350_rv32_falcon_defconfig
@@ -9,7 +9,6 @@ CONFIG_DEFAULT_DEVICE_TREE="ae350_32"
 CONFIG_SYS_MONITOR_LEN=786432
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv32_falcon_xip_defconfig b/configs/ae350_rv32_falcon_xip_defconfig
index a2f8d4cd236..2355bac5019 100644
--- a/configs/ae350_rv32_falcon_xip_defconfig
+++ b/configs/ae350_rv32_falcon_xip_defconfig
@@ -10,7 +10,6 @@ CONFIG_SYS_MONITOR_LEN=786432
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_TEXT_BASE=0x80000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv32_spl_defconfig b/configs/ae350_rv32_spl_defconfig
index 098cf7686d6..6655cbd4a96 100644
--- a/configs/ae350_rv32_spl_defconfig
+++ b/configs/ae350_rv32_spl_defconfig
@@ -9,7 +9,6 @@ CONFIG_DEFAULT_DEVICE_TREE="ae350_32"
 CONFIG_SYS_MONITOR_LEN=786432
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv32_spl_xip_defconfig b/configs/ae350_rv32_spl_xip_defconfig
index 23927888c87..44a6b6534b8 100644
--- a/configs/ae350_rv32_spl_xip_defconfig
+++ b/configs/ae350_rv32_spl_xip_defconfig
@@ -10,7 +10,6 @@ CONFIG_SYS_MONITOR_LEN=786432
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_TEXT_BASE=0x80000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv32_xip_defconfig b/configs/ae350_rv32_xip_defconfig
index 75eb0618454..15f3b5c378b 100644
--- a/configs/ae350_rv32_xip_defconfig
+++ b/configs/ae350_rv32_xip_defconfig
@@ -7,7 +7,6 @@ CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xfffe80
 CONFIG_ENV_SECT_SIZE=0x1000
 CONFIG_DEFAULT_DEVICE_TREE="ae350_32"
 CONFIG_SYS_MONITOR_LEN=786432
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_TARGET_ANDES_AE350=y
 CONFIG_XIP=y
diff --git a/configs/ae350_rv64_defconfig b/configs/ae350_rv64_defconfig
index 932739e5dec..78b9fc439ac 100644
--- a/configs/ae350_rv64_defconfig
+++ b/configs/ae350_rv64_defconfig
@@ -6,7 +6,6 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xfffd70
 CONFIG_ENV_SECT_SIZE=0x1000
 CONFIG_DEFAULT_DEVICE_TREE="ae350_64"
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_TARGET_ANDES_AE350=y
 CONFIG_ARCH_RV64I=y
diff --git a/configs/ae350_rv64_falcon_defconfig b/configs/ae350_rv64_falcon_defconfig
index c9417175ce7..81bebb1a1d4 100644
--- a/configs/ae350_rv64_falcon_defconfig
+++ b/configs/ae350_rv64_falcon_defconfig
@@ -8,7 +8,6 @@ CONFIG_ENV_SECT_SIZE=0x1000
 CONFIG_DEFAULT_DEVICE_TREE="ae350_64"
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv64_falcon_xip_defconfig b/configs/ae350_rv64_falcon_xip_defconfig
index e072f7c2ae9..8e1b8e20052 100644
--- a/configs/ae350_rv64_falcon_xip_defconfig
+++ b/configs/ae350_rv64_falcon_xip_defconfig
@@ -9,7 +9,6 @@ CONFIG_DEFAULT_DEVICE_TREE="ae350_64"
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_TEXT_BASE=0x80000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv64_spl_defconfig b/configs/ae350_rv64_spl_defconfig
index 83ce28077f0..7950074642e 100644
--- a/configs/ae350_rv64_spl_defconfig
+++ b/configs/ae350_rv64_spl_defconfig
@@ -8,7 +8,6 @@ CONFIG_ENV_SECT_SIZE=0x1000
 CONFIG_DEFAULT_DEVICE_TREE="ae350_64"
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv64_spl_xip_defconfig b/configs/ae350_rv64_spl_xip_defconfig
index 5ad1751686c..d5882af1de1 100644
--- a/configs/ae350_rv64_spl_xip_defconfig
+++ b/configs/ae350_rv64_spl_xip_defconfig
@@ -9,7 +9,6 @@ CONFIG_DEFAULT_DEVICE_TREE="ae350_64"
 CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000000
 CONFIG_SPL_TEXT_BASE=0x80000000
 CONFIG_SPL_BSS_START_ADDR=0x400000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_SPL=y
 CONFIG_TARGET_ANDES_AE350=y
diff --git a/configs/ae350_rv64_xip_defconfig b/configs/ae350_rv64_xip_defconfig
index 5ec2cba670b..46ce063c484 100644
--- a/configs/ae350_rv64_xip_defconfig
+++ b/configs/ae350_rv64_xip_defconfig
@@ -6,7 +6,6 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xfffd70
 CONFIG_ENV_SECT_SIZE=0x1000
 CONFIG_DEFAULT_DEVICE_TREE="ae350_64"
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x100000
 CONFIG_TARGET_ANDES_AE350=y
 CONFIG_ARCH_RV64I=y
diff --git a/configs/ibex-ast2700_defconfig b/configs/ibex-ast2700_defconfig
index f088aec8716..8e8259f291d 100644
--- a/configs/ibex-ast2700_defconfig
+++ b/configs/ibex-ast2700_defconfig
@@ -15,7 +15,6 @@ CONFIG_DM_RESET=y
 CONFIG_SPL_TEXT_BASE=0x14bc0080
 CONFIG_SPL_BSS_START_ADDR=0x14bd7800
 CONFIG_SPL_BSS_MAX_SIZE=0x800
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x83000000
 CONFIG_SPL_SIZE_LIMIT=0x16000
 CONFIG_SPL=y
diff --git a/configs/milkv_duo_defconfig b/configs/milkv_duo_defconfig
index 70393de5736..f1f3930564b 100644
--- a/configs/milkv_duo_defconfig
+++ b/configs/milkv_duo_defconfig
@@ -5,7 +5,6 @@ CONFIG_NR_DRAM_BANKS=1
 CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x82300000
 CONFIG_DEFAULT_DEVICE_TREE="cv1800b-milkv-duo"
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80080000
 CONFIG_IDENT_STRING="milkv_duo"
 CONFIG_TARGET_MILKV_DUO=y
diff --git a/configs/qemu-riscv32_defconfig b/configs/qemu-riscv32_defconfig
index cdffda26281..d77054e5a0e 100644
--- a/configs/qemu-riscv32_defconfig
+++ b/configs/qemu-riscv32_defconfig
@@ -5,7 +5,6 @@ CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
 CONFIG_ENV_SIZE=0x20000
 CONFIG_DEFAULT_DEVICE_TREE="qemu-virt32"
 CONFIG_SYS_MONITOR_LEN=786432
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_TARGET_QEMU_VIRT=y
 CONFIG_FIT=y
diff --git a/configs/qemu-riscv32_smode_defconfig b/configs/qemu-riscv32_smode_defconfig
index 3d065b6a9fb..22d9462c022 100644
--- a/configs/qemu-riscv32_smode_defconfig
+++ b/configs/qemu-riscv32_smode_defconfig
@@ -5,7 +5,6 @@ CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
 CONFIG_ENV_SIZE=0x20000
 CONFIG_DEFAULT_DEVICE_TREE="qemu-virt32"
 CONFIG_SYS_MONITOR_LEN=786432
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_TARGET_QEMU_VIRT=y
 CONFIG_RISCV_SMODE=y
diff --git a/configs/qemu-riscv32_spl_defconfig b/configs/qemu-riscv32_spl_defconfig
index 15f1a5d973d..b43eda1e160 100644
--- a/configs/qemu-riscv32_spl_defconfig
+++ b/configs/qemu-riscv32_spl_defconfig
@@ -6,7 +6,6 @@ CONFIG_ENV_SIZE=0x20000
 CONFIG_DEFAULT_DEVICE_TREE="qemu-virt32"
 CONFIG_SYS_MONITOR_LEN=786432
 CONFIG_SPL_BSS_START_ADDR=0x84000000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL=y
 CONFIG_TARGET_QEMU_VIRT=y
diff --git a/configs/qemu-riscv64_defconfig b/configs/qemu-riscv64_defconfig
index bf9a0b07400..b5d65db6308 100644
--- a/configs/qemu-riscv64_defconfig
+++ b/configs/qemu-riscv64_defconfig
@@ -4,7 +4,6 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
 CONFIG_ENV_SIZE=0x20000
 CONFIG_DEFAULT_DEVICE_TREE="qemu-virt64"
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_TARGET_QEMU_VIRT=y
 CONFIG_ARCH_RV64I=y
diff --git a/configs/qemu-riscv64_smode_defconfig b/configs/qemu-riscv64_smode_defconfig
index 6cc42817970..d3f7465ff82 100644
--- a/configs/qemu-riscv64_smode_defconfig
+++ b/configs/qemu-riscv64_smode_defconfig
@@ -4,7 +4,6 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
 CONFIG_ENV_SIZE=0x20000
 CONFIG_DEFAULT_DEVICE_TREE="qemu-virt64"
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_TARGET_QEMU_VIRT=y
 CONFIG_ARCH_RV64I=y
diff --git a/configs/qemu-riscv64_spl_defconfig b/configs/qemu-riscv64_spl_defconfig
index 1c7cef056c4..eae94d8d161 100644
--- a/configs/qemu-riscv64_spl_defconfig
+++ b/configs/qemu-riscv64_spl_defconfig
@@ -5,7 +5,6 @@ CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
 CONFIG_ENV_SIZE=0x20000
 CONFIG_DEFAULT_DEVICE_TREE="qemu-virt64"
 CONFIG_SPL_BSS_START_ADDR=0x84000000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL=y
 CONFIG_TARGET_QEMU_VIRT=y
diff --git a/configs/sifive_unleashed_defconfig b/configs/sifive_unleashed_defconfig
index 01963a4e54e..f341e3e0735 100644
--- a/configs/sifive_unleashed_defconfig
+++ b/configs/sifive_unleashed_defconfig
@@ -11,7 +11,6 @@ CONFIG_DM_RESET=y
 CONFIG_SPL_MMC=y
 CONFIG_SPL_STACK=0x81cfe70
 CONFIG_SPL_BSS_START_ADDR=0x85000000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL=y
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/sifive_unmatched_defconfig b/configs/sifive_unmatched_defconfig
index acbea7fe1bb..0a736f2ba95 100644
--- a/configs/sifive_unmatched_defconfig
+++ b/configs/sifive_unmatched_defconfig
@@ -12,7 +12,6 @@ CONFIG_DM_RESET=y
 CONFIG_SPL_MMC=y
 CONFIG_SPL_STACK=0x81cfe60
 CONFIG_SPL_BSS_START_ADDR=0x85000000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL=y
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/sipeed_licheerv_nano_defconfig b/configs/sipeed_licheerv_nano_defconfig
index 14fefa968c6..ab29f327d7a 100644
--- a/configs/sipeed_licheerv_nano_defconfig
+++ b/configs/sipeed_licheerv_nano_defconfig
@@ -5,7 +5,6 @@ CONFIG_NR_DRAM_BANKS=1
 CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x82300000
 CONFIG_DEFAULT_DEVICE_TREE="sg2002-licheerv-nano-b"
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80080000
 CONFIG_IDENT_STRING="licheerv_nano"
 CONFIG_TARGET_LICHEERV_NANO=y
diff --git a/configs/starfive_visionfive2_defconfig b/configs/starfive_visionfive2_defconfig
index 3e34e4a87f8..544140c03f7 100644
--- a/configs/starfive_visionfive2_defconfig
+++ b/configs/starfive_visionfive2_defconfig
@@ -18,7 +18,6 @@ CONFIG_SPL_STACK=0x8180000
 CONFIG_SPL_TEXT_BASE=0x8000000
 CONFIG_SPL_BSS_START_ADDR=0x8040000
 CONFIG_SPL_BSS_MAX_SIZE=0x10000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x82000000
 CONFIG_SPL=y
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
diff --git a/configs/th1520_lpi4a_defconfig b/configs/th1520_lpi4a_defconfig
index 78e3b25ab82..55a8eaac4a9 100644
--- a/configs/th1520_lpi4a_defconfig
+++ b/configs/th1520_lpi4a_defconfig
@@ -10,7 +10,6 @@ CONFIG_DEFAULT_DEVICE_TREE="th1520-lichee-pi-4a"
 CONFIG_SPL_STACK=0xffe0170000
 CONFIG_SPL_BSS_START_ADDR=0xffe0160000
 CONFIG_SPL_BSS_MAX_SIZE=0x10000
-CONFIG_SYS_BOOTM_LEN=0x4000000
 CONFIG_SYS_LOAD_ADDR=0x80200000
 CONFIG_SPL=y
 CONFIG_TARGET_TH1520_LPI4A=y

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
  2025-07-14 13:33 [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Martin Herren
                   ` (2 preceding siblings ...)
  2025-07-14 13:33 ` [PATCH 3/3] riscv: Remove default SYS_BOOTM_LEN from defconfig Martin Herren
@ 2025-07-15 11:27 ` Michal Simek
  2025-07-16  1:35   ` E Shattow
  3 siblings, 1 reply; 12+ messages in thread
From: Michal Simek @ 2025-07-15 11:27 UTC (permalink / raw)
  To: Martin Herren, u-boot
  Cc: Andreas Schwab, Anup Patel, Atish Patra, Bin Meng, Chia-Wei, Wang,
	Conor Dooley, Cyril Jean, E Shattow, Green Wan, Hal Feng,
	Heinrich Schuchardt, Ilias Apalodimas, Jerome Forissier,
	Junhui Liu, Kongyang Liu, Leo Yu-Chi Liang, Maksim Kiselev,
	Marek Vasut, Martin Schwan, Mattijs Korpershoek, Minda Chen,
	Padmarao Begari, Palmer Dabbelt, Paul Walmsley, Peter Robinson,
	Rick Chen, Sean Anderson, Simon Glass, Sumit Garg,
	Thomas Bonnefille, Tom Rini, Wei Fu, Yao Zi, Yixun Lan,
	Yuri Zaporozhets



On 7/14/25 15:33, Martin Herren wrote:
> 
> The most commonly used SYS_BOOTM_LEN for riscv is 0x4000000 which is
> used in 25 of the current defconfigs.
> 
> The previous default config value of 0x800000 was only used in 8
> defconfigs.
> 
> This patch sets the default to this value.
> 
> Command used to get the stats of the used values on RISCV defconfigs:
> 
> ```
> grep -l "CONFIG_RISCV=y" configs/* | \
>   xargs -I {} sh -c "grep -H CONFIG_SYS_BOOTM_LEN {} || echo {}:DEFAULT" | \
>   cut -d ':' -f 2 | sort | uniq -c
> ```
> 
> Changes in v2:
> - Resubmit using another mail provider to prevent mangling and signature
>    attachement.
> 
> Martin Herren (3):
>    riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
>    riscv: Set SYS_BOOTM_LEN default to 0x4000000
>    riscv: Remove default SYS_BOOTM_LEN from defconfig
> 
>   boot/Kconfig                            | 2 +-
>   configs/ae350_rv32_defconfig            | 1 -
>   configs/ae350_rv32_falcon_defconfig     | 1 -
>   configs/ae350_rv32_falcon_xip_defconfig | 1 -
>   configs/ae350_rv32_spl_defconfig        | 1 -
>   configs/ae350_rv32_spl_xip_defconfig    | 1 -
>   configs/ae350_rv32_xip_defconfig        | 1 -
>   configs/ae350_rv64_defconfig            | 1 -
>   configs/ae350_rv64_falcon_defconfig     | 1 -
>   configs/ae350_rv64_falcon_xip_defconfig | 1 -
>   configs/ae350_rv64_spl_defconfig        | 1 -
>   configs/ae350_rv64_spl_xip_defconfig    | 1 -
>   configs/ae350_rv64_xip_defconfig        | 1 -
>   configs/ibex-ast2700_defconfig          | 1 -
>   configs/k230_canmv_defconfig            | 1 +
>   configs/microchip_mpfs_icicle_defconfig | 1 +
>   configs/milkv_duo_defconfig             | 1 -
>   configs/qemu-riscv32_defconfig          | 1 -
>   configs/qemu-riscv32_smode_defconfig    | 1 -
>   configs/qemu-riscv32_spl_defconfig      | 1 -
>   configs/qemu-riscv64_defconfig          | 1 -
>   configs/qemu-riscv64_smode_defconfig    | 1 -
>   configs/qemu-riscv64_spl_defconfig      | 1 -
>   configs/sifive_unleashed_defconfig      | 1 -
>   configs/sifive_unmatched_defconfig      | 1 -
>   configs/sipeed_licheerv_nano_defconfig  | 1 -
>   configs/sipeed_maix_bitm_defconfig      | 1 +
>   configs/sipeed_maix_smode_defconfig     | 1 +
>   configs/starfive_visionfive2_defconfig  | 1 -
>   configs/th1520_lpi4a_defconfig          | 1 -
>   configs/xilinx_mbv32_defconfig          | 1 +
>   configs/xilinx_mbv32_smode_defconfig    | 1 +
>   configs/xilinx_mbv64_defconfig          | 1 +
>   configs/xilinx_mbv64_smode_defconfig    | 1 +
>   34 files changed, 9 insertions(+), 26 deletions(-)
> 

I don't mind with these changes. It is just value which can be changed.

Acked-by; Michal Simek <michal.simek@amd.com> # xilinx_mbv

Thanks
Michal



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
  2025-07-15 11:27 ` [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Michal Simek
@ 2025-07-16  1:35   ` E Shattow
  2025-07-16 10:11     ` Martin Herren
  2025-07-16 17:00     ` Tom Rini
  0 siblings, 2 replies; 12+ messages in thread
From: E Shattow @ 2025-07-16  1:35 UTC (permalink / raw)
  To: Michal Simek, Martin Herren, u-boot
  Cc: Andreas Schwab, Anup Patel, Atish Patra, Bin Meng, Chia-Wei, Wang,
	Conor Dooley, Cyril Jean, Green Wan, Hal Feng,
	Heinrich Schuchardt, Ilias Apalodimas, Jerome Forissier,
	Junhui Liu, Kongyang Liu, Leo Yu-Chi Liang, Maksim Kiselev,
	Marek Vasut, Martin Schwan, Mattijs Korpershoek, Minda Chen,
	Padmarao Begari, Palmer Dabbelt, Paul Walmsley, Peter Robinson,
	Rick Chen, Sean Anderson, Simon Glass, Sumit Garg,
	Thomas Bonnefille, Tom Rini, Wei Fu, Yao Zi, Yixun Lan,
	Yuri Zaporozhets



On 7/15/25 04:27, Michal Simek wrote:
> 
> 
> On 7/14/25 15:33, Martin Herren wrote:
>>
>> The most commonly used SYS_BOOTM_LEN for riscv is 0x4000000 which is
>> used in 25 of the current defconfigs.
>>
>> The previous default config value of 0x800000 was only used in 8
>> defconfigs.
>>
>> This patch sets the default to this value.
>>
>> Command used to get the stats of the used values on RISCV defconfigs:
>>
>> ```
>> grep -l "CONFIG_RISCV=y" configs/* | \
>>   xargs -I {} sh -c "grep -H CONFIG_SYS_BOOTM_LEN {} || echo
>> {}:DEFAULT" | \
>>   cut -d ':' -f 2 | sort | uniq -c
>> ```
>>
>> Changes in v2:
>> - Resubmit using another mail provider to prevent mangling and signature
>>    attachement.
>>
>> Martin Herren (3):
>>    riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
>>    riscv: Set SYS_BOOTM_LEN default to 0x4000000
>>    riscv: Remove default SYS_BOOTM_LEN from defconfig
>>
>>   boot/Kconfig                            | 2 +-
>>   configs/ae350_rv32_defconfig            | 1 -
>>   configs/ae350_rv32_falcon_defconfig     | 1 -
>>   configs/ae350_rv32_falcon_xip_defconfig | 1 -
>>   configs/ae350_rv32_spl_defconfig        | 1 -
>>   configs/ae350_rv32_spl_xip_defconfig    | 1 -
>>   configs/ae350_rv32_xip_defconfig        | 1 -
>>   configs/ae350_rv64_defconfig            | 1 -
>>   configs/ae350_rv64_falcon_defconfig     | 1 -
>>   configs/ae350_rv64_falcon_xip_defconfig | 1 -
>>   configs/ae350_rv64_spl_defconfig        | 1 -
>>   configs/ae350_rv64_spl_xip_defconfig    | 1 -
>>   configs/ae350_rv64_xip_defconfig        | 1 -
>>   configs/ibex-ast2700_defconfig          | 1 -
>>   configs/k230_canmv_defconfig            | 1 +
>>   configs/microchip_mpfs_icicle_defconfig | 1 +
>>   configs/milkv_duo_defconfig             | 1 -
>>   configs/qemu-riscv32_defconfig          | 1 -
>>   configs/qemu-riscv32_smode_defconfig    | 1 -
>>   configs/qemu-riscv32_spl_defconfig      | 1 -
>>   configs/qemu-riscv64_defconfig          | 1 -
>>   configs/qemu-riscv64_smode_defconfig    | 1 -
>>   configs/qemu-riscv64_spl_defconfig      | 1 -
>>   configs/sifive_unleashed_defconfig      | 1 -
>>   configs/sifive_unmatched_defconfig      | 1 -
>>   configs/sipeed_licheerv_nano_defconfig  | 1 -
>>   configs/sipeed_maix_bitm_defconfig      | 1 +
>>   configs/sipeed_maix_smode_defconfig     | 1 +
>>   configs/starfive_visionfive2_defconfig  | 1 -
>>   configs/th1520_lpi4a_defconfig          | 1 -
>>   configs/xilinx_mbv32_defconfig          | 1 +
>>   configs/xilinx_mbv32_smode_defconfig    | 1 +
>>   configs/xilinx_mbv64_defconfig          | 1 +
>>   configs/xilinx_mbv64_smode_defconfig    | 1 +
>>   34 files changed, 9 insertions(+), 26 deletions(-)
>>
> 
> I don't mind with these changes. It is just value which can be changed.
> 
> Acked-by; Michal Simek <michal.simek@amd.com> # xilinx_mbv
> 
> Thanks
> Michal
> 
> 

Is there some reason for the value from documentation about the
architecture? Or is it arbitrary?

I don't really see the sense in a default value which is simply popular
but not founded in an architectural reasoning.

The values do not change often (ever?) so I miss why there is any
default at all.

-E

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
  2025-07-16  1:35   ` E Shattow
@ 2025-07-16 10:11     ` Martin Herren
  2025-07-16 16:23       ` E Shattow
  2025-07-16 17:00     ` Tom Rini
  1 sibling, 1 reply; 12+ messages in thread
From: Martin Herren @ 2025-07-16 10:11 UTC (permalink / raw)
  To: E Shattow
  Cc: Michal Simek, u-boot, Andreas Schwab, Anup Patel, Atish Patra,
	Bin Meng, Chia-Wei, Wang, Conor Dooley, Cyril Jean, Green Wan,
	Hal Feng, Heinrich Schuchardt, Ilias Apalodimas, Jerome Forissier,
	Junhui Liu, Kongyang Liu, Leo Yu-Chi Liang, Maksim Kiselev,
	Marek Vasut, Martin Schwan, Mattijs Korpershoek, Minda Chen,
	Padmarao Begari, Palmer Dabbelt, Paul Walmsley, Peter Robinson,
	Rick Chen, Sean Anderson, Simon Glass, Sumit Garg,
	Thomas Bonnefille, Tom Rini, Wei Fu, Yao Zi, Yixun Lan,
	Yuri Zaporozhets


[-- Attachment #1.1: Type: text/plain, Size: 1530 bytes --]

Hej all,

On Wednesday, July 16th, 2025 at 3:35 AM, E Shattow <e@freeshell.de> wrote:
> 

> 

> Is there some reason for the value from documentation about the
> architecture? Or is it arbitrary?
> 

> I don't really see the sense in a default value which is simply popular
> but not founded in an architectural reasoning.
> 

> The values do not change often (ever?) so I miss why there is any
> default at all.
> 

> -E

No, there is no documentation justifying this value. My reasoning was the reverse, there is no justification either that this default value should be almost an order of magnitude lower than ARM64 as typically generated RISCV code size is similar and thus the typical kernel size as well.

This is reflected in the most commonly used value in the existing riscv defconfigs as well as other boards not yet mainlined into u-boot.

With a lower value, typical kernels generated from Yocto or others are too big to be able to be booted.

In my idea the default value, when provided, should reflect the most common use case which can then be overridden if wanted.

I understand it was a very bold move on my side as a first submission to challenge a default value and touch over 30 boards, so i won't be offended if this is not accepted. The learning experience on just submitting a patch here and learning about patman and co is already a success for me.

Still I consider the proposed new default as sane and maybe more future oriented for new boards to come.

Cheers,

Martin

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 343 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/3] riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
  2025-07-14 13:33 ` [PATCH 1/3] riscv: Set SYS_BOOTM_LEN explicitly to 0x800000 Martin Herren
@ 2025-07-16 10:33   ` Conor Dooley
  0 siblings, 0 replies; 12+ messages in thread
From: Conor Dooley @ 2025-07-16 10:33 UTC (permalink / raw)
  To: Martin Herren
  Cc: u-boot, Conor Dooley, Cyril Jean, Heinrich Schuchardt,
	Ilias Apalodimas, Jerome Forissier, Junhui Liu, Leo Yu-Chi Liang,
	Marek Vasut, Michal Simek, Padmarao Begari, Sean Anderson,
	Simon Glass, Sumit Garg, Tom Rini

[-- Attachment #1: Type: text/plain, Size: 1148 bytes --]

On Mon, Jul 14, 2025 at 03:33:10PM +0200, Martin Herren wrote:
> For all riscv defconfigs that use the current default value.
> 
> This is done in provision of changing the default value to the most
> common used value of 0x4000000.
> 
> Signed-off-by: Martin Herren <sputnik@on-the-web.ch>
> ---
> 
> (no changes since v1)
> 
>  configs/k230_canmv_defconfig            | 1 +

>  configs/microchip_mpfs_icicle_defconfig | 1 +

tbh, I think this should get the new default value instead of the old
one. Pretty sure in our downstream users we've made it larger and this
is one of the first things I end up changing whenever I do some
debugging or w/e with upstream U-Boot since there's not much chance of
fitting some kernel with all the debug etc bells and whistles into
the current size.

>  configs/sipeed_maix_bitm_defconfig      | 1 +
>  configs/sipeed_maix_smode_defconfig     | 1 +
>  configs/xilinx_mbv32_defconfig          | 1 +
>  configs/xilinx_mbv32_smode_defconfig    | 1 +
>  configs/xilinx_mbv64_defconfig          | 1 +
>  configs/xilinx_mbv64_smode_defconfig    | 1 +
>  8 files changed, 8 insertions(+)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
  2025-07-16 10:11     ` Martin Herren
@ 2025-07-16 16:23       ` E Shattow
  0 siblings, 0 replies; 12+ messages in thread
From: E Shattow @ 2025-07-16 16:23 UTC (permalink / raw)
  To: Martin Herren
  Cc: Michal Simek, u-boot, Andreas Schwab, Anup Patel, Atish Patra,
	Bin Meng, Chia-Wei, Wang, Conor Dooley, Cyril Jean, Green Wan,
	Hal Feng, Heinrich Schuchardt, Ilias Apalodimas, Jerome Forissier,
	Junhui Liu, Kongyang Liu, Leo Yu-Chi Liang, Maksim Kiselev,
	Marek Vasut, Martin Schwan, Mattijs Korpershoek, Minda Chen,
	Padmarao Begari, Palmer Dabbelt, Paul Walmsley, Peter Robinson,
	Rick Chen, Sean Anderson, Simon Glass, Sumit Garg,
	Thomas Bonnefille, Tom Rini, Wei Fu, Yao Zi, Yixun Lan,
	Yuri Zaporozhets



On 7/16/25 03:11, Martin Herren wrote:
> Hej all,
> 
> On Wednesday, July 16th, 2025 at 3:35 AM, E Shattow <e@freeshell.de> wrote:
>>
> 
>>
> 
>> Is there some reason for the value from documentation about the
>> architecture? Or is it arbitrary?
>>
> 
>> I don't really see the sense in a default value which is simply popular
>> but not founded in an architectural reasoning.
>>
> 
>> The values do not change often (ever?) so I miss why there is any
>> default at all.
>>
> 
>> -E
> 
> No, there is no documentation justifying this value. My reasoning was the reverse, there is no justification either that this default value should be almost an order of magnitude lower than ARM64 as typically generated RISCV code size is similar and thus the typical kernel size as well.
> 
> This is reflected in the most commonly used value in the existing riscv defconfigs as well as other boards not yet mainlined into u-boot.
> 
> With a lower value, typical kernels generated from Yocto or others are too big to be able to be booted.
> 
> In my idea the default value, when provided, should reflect the most common use case which can then be overridden if wanted.
> 
> I understand it was a very bold move on my side as a first submission to challenge a default value and touch over 30 boards, so i won't be offended if this is not accepted. The learning experience on just submitting a patch here and learning about patman and co is already a success for me.
> 
> Still I consider the proposed new default as sane and maybe more future oriented for new boards to come.
> 
> Cheers,
> 
> Martin

My concern is to not-find what I am looking for where I am looking for
it and then hunt around to determine where this value comes from. A
default ceases to be a default when there are more than a few
exceptions, as it then becomes an obfuscation.

I do tend to complain when the code base becomes clever for the sake of
being clever, and defer to actual programmers however... all else being
the same (it compiles the same?) go with what you think is most
maintainable and readable.

-E

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
  2025-07-16  1:35   ` E Shattow
  2025-07-16 10:11     ` Martin Herren
@ 2025-07-16 17:00     ` Tom Rini
  2025-07-17  5:10       ` Leo Liang
  1 sibling, 1 reply; 12+ messages in thread
From: Tom Rini @ 2025-07-16 17:00 UTC (permalink / raw)
  To: E Shattow
  Cc: Michal Simek, Martin Herren, u-boot, Andreas Schwab, Anup Patel,
	Atish Patra, Bin Meng, Chia-Wei, Wang, Conor Dooley, Cyril Jean,
	Green Wan, Hal Feng, Heinrich Schuchardt, Ilias Apalodimas,
	Jerome Forissier, Junhui Liu, Kongyang Liu, Leo Yu-Chi Liang,
	Maksim Kiselev, Marek Vasut, Martin Schwan, Mattijs Korpershoek,
	Minda Chen, Padmarao Begari, Palmer Dabbelt, Paul Walmsley,
	Peter Robinson, Rick Chen, Sean Anderson, Simon Glass, Sumit Garg,
	Thomas Bonnefille, Wei Fu, Yao Zi, Yixun Lan, Yuri Zaporozhets

[-- Attachment #1: Type: text/plain, Size: 3790 bytes --]

On Tue, Jul 15, 2025 at 06:35:17PM -0700, E Shattow wrote:
> 
> 
> On 7/15/25 04:27, Michal Simek wrote:
> > 
> > 
> > On 7/14/25 15:33, Martin Herren wrote:
> >>
> >> The most commonly used SYS_BOOTM_LEN for riscv is 0x4000000 which is
> >> used in 25 of the current defconfigs.
> >>
> >> The previous default config value of 0x800000 was only used in 8
> >> defconfigs.
> >>
> >> This patch sets the default to this value.
> >>
> >> Command used to get the stats of the used values on RISCV defconfigs:
> >>
> >> ```
> >> grep -l "CONFIG_RISCV=y" configs/* | \
> >>   xargs -I {} sh -c "grep -H CONFIG_SYS_BOOTM_LEN {} || echo
> >> {}:DEFAULT" | \
> >>   cut -d ':' -f 2 | sort | uniq -c
> >> ```
> >>
> >> Changes in v2:
> >> - Resubmit using another mail provider to prevent mangling and signature
> >>    attachement.
> >>
> >> Martin Herren (3):
> >>    riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
> >>    riscv: Set SYS_BOOTM_LEN default to 0x4000000
> >>    riscv: Remove default SYS_BOOTM_LEN from defconfig
> >>
> >>   boot/Kconfig                            | 2 +-
> >>   configs/ae350_rv32_defconfig            | 1 -
> >>   configs/ae350_rv32_falcon_defconfig     | 1 -
> >>   configs/ae350_rv32_falcon_xip_defconfig | 1 -
> >>   configs/ae350_rv32_spl_defconfig        | 1 -
> >>   configs/ae350_rv32_spl_xip_defconfig    | 1 -
> >>   configs/ae350_rv32_xip_defconfig        | 1 -
> >>   configs/ae350_rv64_defconfig            | 1 -
> >>   configs/ae350_rv64_falcon_defconfig     | 1 -
> >>   configs/ae350_rv64_falcon_xip_defconfig | 1 -
> >>   configs/ae350_rv64_spl_defconfig        | 1 -
> >>   configs/ae350_rv64_spl_xip_defconfig    | 1 -
> >>   configs/ae350_rv64_xip_defconfig        | 1 -
> >>   configs/ibex-ast2700_defconfig          | 1 -
> >>   configs/k230_canmv_defconfig            | 1 +
> >>   configs/microchip_mpfs_icicle_defconfig | 1 +
> >>   configs/milkv_duo_defconfig             | 1 -
> >>   configs/qemu-riscv32_defconfig          | 1 -
> >>   configs/qemu-riscv32_smode_defconfig    | 1 -
> >>   configs/qemu-riscv32_spl_defconfig      | 1 -
> >>   configs/qemu-riscv64_defconfig          | 1 -
> >>   configs/qemu-riscv64_smode_defconfig    | 1 -
> >>   configs/qemu-riscv64_spl_defconfig      | 1 -
> >>   configs/sifive_unleashed_defconfig      | 1 -
> >>   configs/sifive_unmatched_defconfig      | 1 -
> >>   configs/sipeed_licheerv_nano_defconfig  | 1 -
> >>   configs/sipeed_maix_bitm_defconfig      | 1 +
> >>   configs/sipeed_maix_smode_defconfig     | 1 +
> >>   configs/starfive_visionfive2_defconfig  | 1 -
> >>   configs/th1520_lpi4a_defconfig          | 1 -
> >>   configs/xilinx_mbv32_defconfig          | 1 +
> >>   configs/xilinx_mbv32_smode_defconfig    | 1 +
> >>   configs/xilinx_mbv64_defconfig          | 1 +
> >>   configs/xilinx_mbv64_smode_defconfig    | 1 +
> >>   34 files changed, 9 insertions(+), 26 deletions(-)
> >>
> > 
> > I don't mind with these changes. It is just value which can be changed.
> > 
> > Acked-by; Michal Simek <michal.simek@amd.com> # xilinx_mbv
> > 
> > Thanks
> > Michal
> > 
> > 
> 
> Is there some reason for the value from documentation about the
> architecture? Or is it arbitrary?
> 
> I don't really see the sense in a default value which is simply popular
> but not founded in an architectural reasoning.
> 
> The values do not change often (ever?) so I miss why there is any
> default at all.

So, the default for SYS_BOOTM_LEN isn't so much arbitrary as just no
longer reflective of modern times. It's why there's larger defaults for
ARM64 for example. Likely RISCV should just also use that as the default
and platforms with very small amounts of RAM should stick with smaller.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value.
  2025-07-16 17:00     ` Tom Rini
@ 2025-07-17  5:10       ` Leo Liang
  0 siblings, 0 replies; 12+ messages in thread
From: Leo Liang @ 2025-07-17  5:10 UTC (permalink / raw)
  To: Tom Rini
  Cc: E Shattow, Michal Simek, Martin Herren, u-boot, Andreas Schwab,
	Anup Patel, Atish Patra, Bin Meng, Chia-Wei, Wang, Conor Dooley,
	Cyril Jean, Green Wan, Hal Feng, Heinrich Schuchardt,
	Ilias Apalodimas, Jerome Forissier, Junhui Liu, Kongyang Liu,
	Maksim Kiselev, Marek Vasut, Martin Schwan, Mattijs Korpershoek,
	Minda Chen, Padmarao Begari, Palmer Dabbelt, Paul Walmsley,
	Peter Robinson, Rick Chen, Sean Anderson, Simon Glass, Sumit Garg,
	Thomas Bonnefille, Wei Fu, Yao Zi, Yixun Lan, Yuri Zaporozhets

On Wed, Jul 16, 2025 at 11:00:15AM -0600, Tom Rini wrote:
> [EXTERNAL MAIL]

> Date: Wed, 16 Jul 2025 11:00:15 -0600
> From: Tom Rini <trini@konsulko.com>
> To: E Shattow <e@freeshell.de>
> Cc: Michal Simek <michal.simek@amd.com>, Martin Herren
>  <sputnik@on-the-web.ch>, u-boot@lists.denx.de, Andreas Schwab
>  <schwab@suse.de>, Anup Patel <anup@brainfault.org>, Atish Patra
>  <atishp@atishpatra.org>, Bin Meng <bmeng.cn@gmail.com>, "Chia-Wei, Wang"
>  <chiawei_wang@aspeedtech.com>, Conor Dooley <conor.dooley@microchip.com>,
>  Cyril Jean <cyril.jean@microchip.com>, Green Wan <green.wan@sifive.com>,
>  Hal Feng <hal.feng@starfivetech.com>, Heinrich Schuchardt
>  <xypron.glpk@gmx.de>, Ilias Apalodimas <ilias.apalodimas@linaro.org>,
>  Jerome Forissier <jerome.forissier@linaro.org>, Junhui Liu
>  <junhui.liu@pigmoral.tech>, Kongyang Liu <seashell11234455@gmail.com>, Leo
>  Yu-Chi Liang <ycliang@andestech.com>, Maksim Kiselev
>  <bigunclemax@gmail.com>, Marek Vasut <marek.vasut+renesas@mailbox.org>,
>  Martin Schwan <m.schwan@phytec.de>, Mattijs Korpershoek
>  <mkorpershoek@kernel.org>, Minda Chen <minda.chen@starfivetech.com>,
>  Padmarao Begari <padmarao.begari@amd.com>, Palmer Dabbelt
>  <palmer@dabbelt.com>, Paul Walmsley <paul.walmsley@sifive.com>, Peter
>  Robinson <pbrobinson@gmail.com>, Rick Chen <rick@andestech.com>, Sean
>  Anderson <seanga2@gmail.com>, Simon Glass <sjg@chromium.org>, Sumit Garg
>  <sumit.garg@kernel.org>, Thomas Bonnefille
>  <thomas.bonnefille@bootlin.com>, Wei Fu <wefu@redhat.com>, Yao Zi
>  <ziyao@disroot.org>, Yixun Lan <dlan@gentoo.org>, Yuri Zaporozhets
>  <yuriz@vodafonemail.de>
> Subject: Re: [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly
>  used value.
> 
> On Tue, Jul 15, 2025 at 06:35:17PM -0700, E Shattow wrote:
> > 
> > 
> > On 7/15/25 04:27, Michal Simek wrote:
> > > 
> > > 
> > > On 7/14/25 15:33, Martin Herren wrote:
> > >>
> > >> The most commonly used SYS_BOOTM_LEN for riscv is 0x4000000 which is
> > >> used in 25 of the current defconfigs.
> > >>
> > >> The previous default config value of 0x800000 was only used in 8
> > >> defconfigs.
> > >>
> > >> This patch sets the default to this value.
> > >>
> > >> Command used to get the stats of the used values on RISCV defconfigs:
> > >>
> > >> ```
> > >> grep -l "CONFIG_RISCV=y" configs/* | \
> > >>   xargs -I {} sh -c "grep -H CONFIG_SYS_BOOTM_LEN {} || echo
> > >> {}:DEFAULT" | \
> > >>   cut -d ':' -f 2 | sort | uniq -c
> > >> ```
> > >>
> > >> Changes in v2:
> > >> - Resubmit using another mail provider to prevent mangling and signature
> > >>    attachement.
> > >>
> > >> Martin Herren (3):
> > >>    riscv: Set SYS_BOOTM_LEN explicitly to 0x800000
> > >>    riscv: Set SYS_BOOTM_LEN default to 0x4000000
> > >>    riscv: Remove default SYS_BOOTM_LEN from defconfig
> > >>
> > >>   boot/Kconfig                            | 2 +-
> > >>   configs/ae350_rv32_defconfig            | 1 -
> > >>   configs/ae350_rv32_falcon_defconfig     | 1 -
> > >>   configs/ae350_rv32_falcon_xip_defconfig | 1 -
> > >>   configs/ae350_rv32_spl_defconfig        | 1 -
> > >>   configs/ae350_rv32_spl_xip_defconfig    | 1 -
> > >>   configs/ae350_rv32_xip_defconfig        | 1 -
> > >>   configs/ae350_rv64_defconfig            | 1 -
> > >>   configs/ae350_rv64_falcon_defconfig     | 1 -
> > >>   configs/ae350_rv64_falcon_xip_defconfig | 1 -
> > >>   configs/ae350_rv64_spl_defconfig        | 1 -
> > >>   configs/ae350_rv64_spl_xip_defconfig    | 1 -
> > >>   configs/ae350_rv64_xip_defconfig        | 1 -
> > >>   configs/ibex-ast2700_defconfig          | 1 -
> > >>   configs/k230_canmv_defconfig            | 1 +
> > >>   configs/microchip_mpfs_icicle_defconfig | 1 +
> > >>   configs/milkv_duo_defconfig             | 1 -
> > >>   configs/qemu-riscv32_defconfig          | 1 -
> > >>   configs/qemu-riscv32_smode_defconfig    | 1 -
> > >>   configs/qemu-riscv32_spl_defconfig      | 1 -
> > >>   configs/qemu-riscv64_defconfig          | 1 -
> > >>   configs/qemu-riscv64_smode_defconfig    | 1 -
> > >>   configs/qemu-riscv64_spl_defconfig      | 1 -
> > >>   configs/sifive_unleashed_defconfig      | 1 -
> > >>   configs/sifive_unmatched_defconfig      | 1 -
> > >>   configs/sipeed_licheerv_nano_defconfig  | 1 -
> > >>   configs/sipeed_maix_bitm_defconfig      | 1 +
> > >>   configs/sipeed_maix_smode_defconfig     | 1 +
> > >>   configs/starfive_visionfive2_defconfig  | 1 -
> > >>   configs/th1520_lpi4a_defconfig          | 1 -
> > >>   configs/xilinx_mbv32_defconfig          | 1 +
> > >>   configs/xilinx_mbv32_smode_defconfig    | 1 +
> > >>   configs/xilinx_mbv64_defconfig          | 1 +
> > >>   configs/xilinx_mbv64_smode_defconfig    | 1 +
> > >>   34 files changed, 9 insertions(+), 26 deletions(-)
> > >>
> > > 
> > > I don't mind with these changes. It is just value which can be changed.
> > > 
> > > Acked-by; Michal Simek <michal.simek@amd.com> # xilinx_mbv
> > > 
> > > Thanks
> > > Michal
> > > 
> > > 
> > 
> > Is there some reason for the value from documentation about the
> > architecture? Or is it arbitrary?
> > 
> > I don't really see the sense in a default value which is simply popular
> > but not founded in an architectural reasoning.
> > 
> > The values do not change often (ever?) so I miss why there is any
> > default at all.
> 
> So, the default for SYS_BOOTM_LEN isn't so much arbitrary as just no
> longer reflective of modern times. It's why there's larger defaults for
> ARM64 for example. Likely RISCV should just also use that as the default
> and platforms with very small amounts of RAM should stick with smaller.
> 
> -- 
> Tom

Hi Martin,

These changes seem reasonable, and I'm fine with them.
If you send a v2 PR incorporating Conor's comment on the first patch,
you may add my Reviewed-by tag.

Best regards,
Leo

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2025-07-17  5:25 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-14 13:33 [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Martin Herren
2025-07-14 13:33 ` [PATCH 1/3] riscv: Set SYS_BOOTM_LEN explicitly to 0x800000 Martin Herren
2025-07-16 10:33   ` Conor Dooley
2025-07-14 13:33 ` [PATCH 2/3] riscv: Set SYS_BOOTM_LEN default to 0x4000000 Martin Herren
2025-07-14 13:33 ` [PATCH 3/3] riscv: Remove default SYS_BOOTM_LEN from defconfig Martin Herren
2025-07-15 11:27 ` [PATCH 0/3] Update riscv's SYS_BOOTM_LEN to the most commonly used value Michal Simek
2025-07-16  1:35   ` E Shattow
2025-07-16 10:11     ` Martin Herren
2025-07-16 16:23       ` E Shattow
2025-07-16 17:00     ` Tom Rini
2025-07-17  5:10       ` Leo Liang
  -- strict thread matches above, loose matches on Subject: below --
2025-07-12  8:26 Martin Herren

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.