[U-Boot] [PATCH] Redundant Environment: protect full sector size

Wolfgang Denk wd at denx.de
Thu Jun 4 00:16:44 CEST 2009


Hello,

In message <1242339363-28686-3-git-send-email-wd at denx.de> you wrote:
> Several boards used different ways to specify the size of the
> protected area when enabling flash write protection for the sectors
> holding the environment variables: some used CONFIG_ENV_SIZE and
> CONFIG_ENV_SIZE_REDUND, some used CONFIG_ENV_SECT_SIZE, and some even
> a mix of both for the "normal" and the "redundant" areas.
> 
> Normally, this makes no difference at all. However, things are
> different when you have to deal with boards that can come with
> different types of flash chips, which may have different sector
> sizes.
> 
> Here we may have to chose CONFIG_ENV_SECT_SIZE such that it fits the
> biggest sector size, which may include several sectors on boards using
> the smaller sector flash types. In such a case, using CONFIG_ENV_SIZE
> or CONFIG_ENV_SIZE_REDUND to enable the protection may lead to the
> case that only the first of these sectors get protected, while the
> following ones aren't.
> 
> This is no real problem, but it can be confusing for the user -
> especially on boards that use CONFIG_ENV_SECT_SIZE to protect the
> "normal" areas, while using CONFIG_ENV_SIZE_REDUND for the
> "redundant" area.
> 
> To avoid such inconsistencies, I changed all sucn boards that I found
> to consistently use CONFIG_ENV_SECT_SIZE for protection. This should
> not cause any functional changes to the code.
> 
> Signed-off-by: Wolfgang Denk <wd at denx.de>
> Cc: Paul Ruhland
> Cc: Pantelis Antoniou <panto at intracom.gr>
> Cc: Stefan Roese <sr at denx.de>
> Cc: Gary Jennejohn <garyj at denx.de>
> Cc: Dave Ellis <DGE at sixnetio.com>
> ---
>  board/lpd7a40x/flash.c         |    4 ++--
>  board/mcc200/mcc200.c          |    2 +-
>  board/netphone/flash.c         |    4 ++--
>  board/netta/flash.c            |    4 ++--
>  board/netta2/flash.c           |    4 ++--
>  board/netvia/flash.c           |    4 ++--
>  board/pleb2/flash.c            |    4 ++--
>  board/prodrive/pdnb3/flash.c   |    2 +-
>  board/rmu/flash.c              |    6 +++---
>  board/samsung/smdk2400/flash.c |    4 ++--
>  board/sixnet/flash.c           |    4 ++--
>  board/socrates/socrates.c      |    2 +-
>  board/tqc/tqm85xx/tqm85xx.c    |    2 +-
>  board/trab/flash.c             |    4 ++--
>  drivers/mtd/cfi_flash.c        |    2 +-
>  15 files changed, 26 insertions(+), 26 deletions(-)

Applied to master.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"If that makes any sense to you, you have a big problem."
                                  -- C. Durance, Computer Science 234


More information about the U-Boot mailing list