[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