From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 08/11] arm/pxa: increase CONFIG_SYS_MALLOC_LEN to (196Kb + space for environment)
Date: Thu, 26 Aug 2010 13:48:07 +0200 [thread overview]
Message-ID: <201008261348.08623.marek.vasut@gmail.com> (raw)
In-Reply-To: <1282819683-655-8-git-send-email-mikhail.kshevetskiy@gmail.com>
Dne ?t 26. srpna 2010 12:48:03 Mikhail Kshevetskiy napsal(a):
> it looks like u-boot allocates a buffer of CONFIG_ENV_SECT_SIZE
> size when making "saveenv" command. In our case CONFIG_ENV_SECT_SIZE
> is 128Kb, so having CONFIG_SYS_MALLOC_LEN = (128Kb + space for
> environment) is not sufficient.
>
> Increase CONFIG_SYS_MALLOC_LEN to (196Kb + space for environment) to
> resolve a problem.
You probably hit this because you changed the ENV size. Wolfgang, can you please
comment on this one too?
Thanks
>
> Signed-off-by: Mikhail Kshevetskiy <mikhail.kshevetskiy@gmail.com>
> ---
> include/configs/vpac270.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/include/configs/vpac270.h b/include/configs/vpac270.h
> index 98faf9a..ddb3e0e 100644
> --- a/include/configs/vpac270.h
> +++ b/include/configs/vpac270.h
> @@ -37,7 +37,7 @@
> * Environment settings
> */
> #define CONFIG_ENV_SIZE (4 * 1024)
> -#define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 128*1024)
> +#define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 196*1024)
> #define CONFIG_SYS_GBL_DATA_SIZE 128
>
> #define CONFIG_BOOTCOMMAND
\
next prev parent reply other threads:[~2010-08-26 11:48 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-26 10:47 [U-Boot] [PATCH 01/11] arm/pxa: remove unused u-boot.lds from board/vpac270 Mikhail Kshevetskiy
2010-08-26 10:47 ` [U-Boot] [PATCH 02/11] arm/pxa: fix board/vpac270/vpac270.c compilation if usb support is disabled Mikhail Kshevetskiy
2010-08-26 10:47 ` [U-Boot] [PATCH 03/11] arm/pxa: enable stuart/btuart for vpac270 Mikhail Kshevetskiy
2010-08-26 10:47 ` [U-Boot] [PATCH 04/11] arm/pxa: cleanup vpac270 config and set CONFIG_SYS_HZ to 1000 Mikhail Kshevetskiy
2010-08-26 10:48 ` [U-Boot] [PATCH 05/11] arm/pxa: fix environment location/length for vpac270 and related bugs Mikhail Kshevetskiy
2010-08-26 11:41 ` Marek Vasut
2010-08-26 10:48 ` [U-Boot] [PATCH 06/11] arm/pxa: remove CONFIG_ENV_OVERWRITE from vpac270 config Mikhail Kshevetskiy
2010-08-26 11:43 ` Marek Vasut
2010-08-26 10:48 ` [U-Boot] [PATCH 07/11] arm/pxa: decrease environment size and stack size for vpac270 Mikhail Kshevetskiy
2010-08-26 11:45 ` Marek Vasut
2010-09-18 20:11 ` Wolfgang Denk
2010-08-26 10:48 ` [U-Boot] [PATCH 08/11] arm/pxa: increase CONFIG_SYS_MALLOC_LEN to (196Kb + space for environment) Mikhail Kshevetskiy
2010-08-26 11:48 ` Marek Vasut [this message]
2010-09-18 20:14 ` Wolfgang Denk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201008261348.08623.marek.vasut@gmail.com \
--to=marek.vasut@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.