[U-Boot] [PATCH] Add support for setting environment variable from RAM.

[Eric Nelson]

Wolfgang Denk wrote:
> 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.
> 
I'm not sure how this made it back onto the list. Sorry about that.

> This command makes no sense (especially since it will process only a
> single variable).
> 
> Please use a script image with "askenv" instead.
> 
Thanks for the tip. I hadn't noticed askenv previously.

> 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
>