public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/1] mx51evk: savenv or env save command does not work
Date: Tue, 16 Nov 2010 18:28:03 +0100	[thread overview]
Message-ID: <4CE2BF23.20507@denx.de> (raw)
In-Reply-To: <1289896880-15792-1-git-send-email-r64343@freescale.com>

On 11/16/2010 09:41 AM, Jason Liu wrote:

Hi Jason,

> fix saveenv or env save command not work on mx51evk board.
> with this patch, we can use savenv or env save to
> store enviroments to mmc card slot 0

>  
> -#define CONFIG_ENV_SECT_SIZE    (128 * 1024)
> -#define CONFIG_ENV_SIZE		CONFIG_ENV_SECT_SIZE
> -#define CONFIG_ENV_IS_NOWHERE
> +#define CONFIG_ENV_OFFSET      (6 * 64 * 1024)
> +#define CONFIG_ENV_SIZE        (2 * 64 * 1024)
> +#define CONFIG_ENV_IS_IN_MMC
> +#define CONFIG_SYS_MMC_ENV_DEV 0

Why do we need 128KB to save the environment in case of MMC ? It seems
to me too much, because we are not constrained to the sector size as for
flash devices.

Can we save some time by saving or reading the environment reducing its
size ? Or do you plan to save something more in this area ?

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  reply	other threads:[~2010-11-16 17:28 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-16  8:41 [U-Boot] [PATCH 1/1] mx51evk: savenv or env save command does not work Jason Liu
2010-11-16 17:28 ` Stefano Babic [this message]
2010-11-17  1:44   ` Jason Liu
2010-11-17  7:03     ` Wolfgang Denk
2010-11-17  7:29       ` Jason Liu
2010-11-17  7:42         ` Wolfgang Denk
2010-11-17 21:28           ` Scott Wood
2010-11-17 21:50             ` Wolfgang Denk
2010-11-17  7:54         ` Stefano Babic
2010-11-17  8:03           ` Jason Liu
2010-11-17 13:00             ` Wolfgang Denk
2010-11-17  7:44     ` Stefano Babic

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=4CE2BF23.20507@denx.de \
    --to=sbabic@denx.de \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox