[U-Boot] [PATCH] fw_env: add NAND support

[Wolfgang Denk]

Dear Guennadi Liakhovetski,

In message <Pine.LNX.4.64.0809030014190.12823 at axis700.grange> you wrote:
>
> If you were first setting the flag to "invalid, write in progress", then 
> wrote the environment, then reset the flag to "valid, write completed 
> successfully", then yes, writing per one write would be essentially 
> different.

But that's exactly what we are doing. The flag  value  for  "invalid,
write  in progress" is 0xFF and is set as soon as we start to destroy
the valid data by erasing the flash.

> In short, I think, CRC provides sufficient protection of data integrity.

CRC32 is not a perfect test. Yes, it's good enough for most  ordinary
cases,  but  when we can design in additional robustness we should do
this. And we should not give up what we  already  have  without  very
good reasons.

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
The Wright Bothers weren't the first to fly. They were just the first
not to crash.