Re: [U-Boot] [PATCHv2 3/4] env_nand.c: clarify log messages when env reading fails

On Thu, Feb 21, 2013 at 06:21:55PM +0100, Phil Sutter wrote:

The single message is misleading, since there is no equivalent success note when reading the other copy succeeds. Instead, warn if one of the redundant copies could not be loaded and emphasise on the error when reading both fails.

Signed-off-by: Phil Sutter phil.sutter@viprinet.com

common/env_nand.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-)

Applied to u-boot-nand-flash.

• if (readenv(CONFIG_ENV_OFFSET_REDUND, (u_char *) tmp_env2))

• puts("No Valid Redundant Environment Area found\n");
  

• if (read1_fail && read2_fail)

• puts("*** Error - No Valid Environment Area found\n");
  

• else if (read1_fail || read2_fail)

• puts("*** Warning - some problems detected "
  

•      "reading environment; recovered successfully\n");
  
  

  crc1_ok = crc32(0, tmp_env1->data, ENV_SIZE) == tmp_env1->crc; crc2_ok = crc32(0, tmp_env2->data, ENV_SIZE) == tmp_env2->crc;

We should also give a message if one of the CRCs is bad, though that's an existing problem.

-Scott