From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Nelson Date: Thu, 05 Feb 2009 14:15:42 -0700 Subject: [U-Boot] [PATCH] Add support for setting environment variable from RAM. In-Reply-To: <20090205193038.E99D18322908@gemini.denx.de> References: <1233631739-17568-1-git-send-email-eric.nelson@boundarydevices.com> <20090205193038.E99D18322908@gemini.denx.de> Message-ID: <498B56FE.1080507@boundarydevices.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.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 > > 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 >