<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
  <title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Wolfgang Denk wrote:
<blockquote cite="mid:20080613064116.46768248CE@gemini.denx.de"
 type="cite">
  <pre wrap="">In message <a class="moz-txt-link-rfc2396E" href="mailto:48515F51.5030504@harris.com">&lt;48515F51.5030504@harris.com&gt;</a> you wrote:
  </pre>
  <blockquote type="cite">
    <pre wrap="">The silent console as currently implemented is not silent if a board has been
newly manufactured.  I.e., some messages are printed prior to being able to set
the "silent" environment variable the first time.

The following patch adds a new configuration option,
CONFIG_VERY_SILENT_CONSOLE, which modifies the behavior slightly.  If this
option is selected, then the absence of the "silent" variable will result in
a default behavior of "silent".  Also, if "silent" is set to "1", then the
behavior will be "silent".  Only if a different value, say "0", is selected
will the behavior be "verbose".

This patch doesn't change the behavior for any existing BSP's because they will
not have selected the CONFIG_VERY_SILENT_CONSOLE option.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
I consider this seriously confusing.

Also I don't see what your problem is, assuming you provide a valid
environment with your newly manufactured systems.

I thend to reject this patch.

Best regards,

Wolfgang Denk
  </pre>
</blockquote>
<tt><br>
When we manufacture a new board, we will load the flash via jtag. All I
had planned to put in at that stage was the u-boot image itself.&nbsp; I did
not plan to put in an initial environment, because the environment can
easily be constructed by u-boot.&nbsp; The problem is that by the time the
initial environment is constructed, some messages that I want to
suppress will already have been printed, because "silent" is not
defined yet.&nbsp; The purpose of the patch is to suppress messages in this
case.<br>
<br>
I currently construct the environment in my BSP.c file, like so:<br>
<br>
&nbsp;&nbsp;&nbsp; if(setenv("flash_self",&nbsp;&nbsp;&nbsp;&nbsp; "run flash_dhcp"))&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return 1;<br>
&nbsp;&nbsp;&nbsp; if(setHex("kernel0_addr",&nbsp;&nbsp; CFG_KERNEL0_ADDR))&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return 1;<br>
<br>
I do this rather than using CONFIG_EXTRA_ENV_SETTINGS because some of
the values have to be computed.&nbsp; So, I guess I could write a host tool
to construct the environment by linking with my BSP.c file, and then
merge the constructed environment with the u-boot executable, so that
jtag could put everything in at once.&nbsp; But that is much more
complicated.<br>
<br>
How do other people construct the initial environment, so that it can
be loaded via jtag along with the u-boot executable?<br>
<br>
&nbsp;&nbsp;&nbsp; Steve<br>
<br>
</tt><br>
<br>
</body>
</html>