diff mbox series

[U-Boot,RFC,1/1] riscv: increase the environment size for the qemu-riscv platform to 128kB

Message ID 20190505213629.22208-2-merker@debian.org
State Accepted
Commit eacf07e631dc97cef0ae903cea7a3655355d9941
Delegated to: Andes
Headers show
Series riscv: qemu-riscv environment size | expand

Commit Message

Karsten Merker May 5, 2019, 9:36 p.m. UTC
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(-)

Comments

Bin Meng May 8, 2019, 1:39 p.m. UTC | #1
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>
Rick Chen June 4, 2019, 5:43 a.m. UTC | #2
> From: U-Boot [mailto:u-boot-bounces@lists.denx.de] On Behalf Of Bin Meng
> Sent: Wednesday, May 08, 2019 9:39 PM
> To: Karsten Merker
> Cc: U-Boot Mailing List
> Subject: Re: [U-Boot] [RFC PATCH 1/1] riscv: increase the environment size for
> the qemu-riscv platform to 128kB
>
> 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>

Applied to u-boot-riscv/master, thanks!

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

Patch

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) \