Dear Mike Frysinger,
In message <201012010634.19596.vap...@gentoo.org> you wrote:
>
> hrm, after running this through our test bench, it seems the old code and new
> code are not functionality equivalent. it boils down to default values when
> the env var is not set. some places interpret this to mean "yes" while
> others
> expect "no". getenv_yesno() takes the "no" default which doesnt always work.
In addition to the default values, there is the difference that the
documentation states that "autostart" has to be set to "yes", i. e. a
plain "y" is not supposed to mean 'yes'.
> i can update the API to take a 2nd arg (the default value), or we can punt
> this patch. it's in "next", so there's less pressure to get it fixed
> immediately ...
I think the easiest way to fix this is to revert the commit.
Do you agree?
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: w...@denx.de
Unix is supported by IBM, like a hanging man is supported by rope.
- Don Libes & Sandy Ressler: _Life With Unix_
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
