[U-Boot] [RFC PATCH 0/1] riscv: qemu-riscv environment size
Hello,
currently the environment size for the qemu-riscv platform is set to only 4kB. The default environment (with distro_bootcmd support) has already nearly 4kB, so defining additional environment variables very quickly exceeds the allocated space. Is there a specific reason to use only 4kB for the environment? If not, I would like to propose to increase the environment size to 128kB similar to what we have on a number of other platforms.
Regards, Karsten
Karsten Merker (1): riscv: increase the environment size for the qemu-riscv platform to 128kB
include/configs/qemu-riscv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
The existing default size of 4kB is too small as the default environment has already nearly that size and defining a single additional environment variable can exceed the available space.
Signed-off-by: Karsten Merker merker@debian.org --- include/configs/qemu-riscv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/qemu-riscv.h b/include/configs/qemu-riscv.h index 2588c5a0b2..3d9208bab2 100644 --- a/include/configs/qemu-riscv.h +++ b/include/configs/qemu-riscv.h @@ -20,7 +20,7 @@ #define CONFIG_STANDALONE_LOAD_ADDR 0x80200000
/* Environment options */ -#define CONFIG_ENV_SIZE SZ_4K +#define CONFIG_ENV_SIZE SZ_128K
#define BOOT_TARGET_DEVICES(func) \ func(QEMU, qemu, na) \
On Mon, May 6, 2019 at 5:49 AM Karsten Merker merker@debian.org wrote:
The existing default size of 4kB is too small as the default environment has already nearly that size and defining a single additional environment variable can exceed the available space.
Signed-off-by: Karsten Merker merker@debian.org
include/configs/qemu-riscv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Bin Meng bmeng.cn@gmail.com
participants (2)
-
Bin Meng -
Karsten Merker