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

[Wolfgang Denk]

Dear Eric,

In message <4988936E.3070504 at boundarydevices.com> you wrote:
> 
> > In the email thread you mentioned above, Detlev mentions 2 alternatives
> > to the "ramenv" command - loading a uImage script and running it via
> > autoscr, or modifying autoscr to be able to raw files (non-uImages).
> > Both of these methods seem cleaner and more flexible at a glance.  Is
> > there a specific reason using autoscr wouldn't work for your setup?
> > 
> The customer requesting this feature operates in a regulated environment with
> pretty strict rules about separation of code and data. Autoscr is kind of
> a big stick for what we're trying to achieve: (configuring an LCD with
> settings from a file on SD card).

Why is it a big stick? It has a couple of significant advantages over
your implementation:

* It already exissts in mainline.
* It can set several environment variables at once (and also perform
  any other commands you might need on your system).
* It is based on text files which are easy to edit without need for
  new tools.
* It will verify that your file has been transmitted correctly.

> Our particular need is just for a single environment variable, so the
> update works pretty well. I started by updating our 'lcdpanel' U-Boot
> command to read from file, but this is much more useful.

I suggest you use autoscr instead.

I think I will reject your patch.

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
The most exciting phrase to hear in science, the one that heralds new
discoveries, is not "Eureka!" (I found it!) but "That's funny ..."
                                                      -- Isaac Asimov