[U-Boot] fw_setenv broken?

[Wolfgang Denk]

Dear Stefano Babic,

In message <4CE4092B.7090209 at denx.de> you wrote:
> On 11/17/2010 05:30 PM, Steve Sakoman wrote:
> > I'm seeing some strange behavior with the fw_setenv tools on OMAP.
> > 
> > Here's what I see when using the tools on OMAP (overo in this case):
> > 
> > 1. fw_printenv prints the environment with no issues [1]
> > 2. fw_setenv allows me to change a variable with no reported errors [2]
> > 3. fw_printenv will print the changed environment, however the variables
> > are not sorted [3]
> 
> I tested yesterday on a davinci board, I can confirm this behavior, I
> have not thought was an error. I do not see any code in fw_env.c to sort
> variables. I konow the variables are sorted in u-boot, but do we ever
> have this feature on the userland fw_printenv ?

Indeed this behaviour is normal. fw_printenv does not sort the output
(not yet - patches welcome).

> > I added debug printf's to readenv() in env_nand.c and the root cause is
> > an error return from ret=nand_read(&nand_info[0], offset, &len,
> > char_ptr)).  I get an error code of -74
> > 
> > Before I spend too much time on this I wanted to check to see if others
> > are seeing this issue, or whether it might be OMAP specific.
> 
> At least this should not be a general failure, because it works on my
> target. It could be also nand specific.

Thanks for confirming this.

Well, the next step should be a review of the code, where error -74
gets set and what that probably means...

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
No one is fit to be trusted with power. ... No one. ... Any  man  who
has lived at all knows the follies and wickedness he's capabel of ...
And  if  he  does  know it, he knows also that neither he nor any man
ought to be allowed to decide a single human fate.
- C. P. Snow,  The Light and the Dark