All of lore.kernel.org
 help / color / mirror / Atom feed
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
> 

  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.