[U-Boot] [PATCH] Add support for setting environment variable from RAM.
Wolfgang Denk
wd at denx.de
Thu Feb 5 20:30:38 CET 2009
Dear "Eric Nelson (Boundary Devices)",
In message <1233631739-17568-1-git-send-email-eric.nelson at boundarydevices.com> you wrote:
> This is useful for allowing scripts to read environment variables from
> file, among other things.
>
> This is a slightly modified version of what Alessandro submitted to the
> mailing list last July:
> http://www.mail-archive.com/u-boot-users@lists.sourceforge.net/msg07932.html
>
> I changed the name from 'setenvram' to 'ramenv' to prevent breakage of scripts
> that use the abbreviation 'set' (which my handss have the habit of doing).
>
> Signed-off-by: Eric Nelson <eric.nelson at boundarydevices.com>
We discussed thsi before, and I already NAKed it.
I hereby NAK it again.
This command makes no sense (especially since it will process only a
single variable).
Please use a script image with "askenv" instead.
Note: if the code would process a list of variables, say, in internal
U-Boot environment format (separated by single NUL, terminated by
double NUL characters), that could be a useful building block to
implement the "reset to default environment" / "reset factory
defaults" command we discussed earlier (see mailing list archive) -
and as such, it had much better chances to be considered.
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
Question: How does one get fresh air into a Russian church?
Answer: One clicks on an icon, and a window opens!
More information about the U-Boot
mailing list