8 Oct
2010
8 Oct
'10
6:21 p.m.
On Fri, Oct 8, 2010 at 9:05 AM, Stefano Babic sbabic@denx.de wrote:
Steve Sakoman wrote:
I'll revise my patch to add the missing gd->env_valid = 1; and re-test.
Hmm . . . looking at env_nand.c I see that it doesn't do gd->env_valid = 1 either, which is probably why I removed it last night.
Any idea whether it is actually required? Seems to work fine without it!
I have understood, it is not required if we call env_import and we require to verify the checksum.
OK, thanks!
Would you like me to submit the patch or would you like to revise & resubmit?
Steve