From: Eric Nelson <eric.nelson@boundarydevices.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Add support for setting environment variable from RAM.
Date: Thu, 05 Feb 2009 14:15:42 -0700 [thread overview]
Message-ID: <498B56FE.1080507@boundarydevices.com> (raw)
In-Reply-To: <20090205193038.E99D18322908@gemini.denx.de>
Wolfgang Denk wrote:
> Dear "Eric Nelson (Boundary Devices)",
>
> In message <1233631739-17568-1-git-send-email-eric.nelson@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 at 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@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
>
next prev parent reply other threads:[~2009-02-05 21:15 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-03 3:28 [U-Boot] [PATCH] Add support for setting environment variable from RAM Eric Nelson
2009-02-05 19:30 ` Wolfgang Denk
2009-02-05 21:15 ` Eric Nelson [this message]
2009-02-05 21:26 ` Wolfgang Denk
2009-02-05 21:34 ` Eric Nelson
-- strict thread matches above, loose matches on Subject: below --
2009-02-03 15:28 Eric Nelson
2009-02-03 16:00 ` Peter Tyser
2009-02-03 18:56 ` Eric Nelson
2009-02-03 20:16 ` Wolfgang Denk
2009-02-03 21:48 ` Eric Nelson
2009-02-03 22:08 ` Wolfgang Denk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=498B56FE.1080507@boundarydevices.com \
--to=eric.nelson@boundarydevices.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.