Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
>> The saveenv also looks funky.  I only mucked with the cmd_mem.c command 
>> to make it display better with the progress dots, obviously the saveenv 
>> command needs to have the same changes
>>    s/"Writing to Flash... "/"Writing to Flash\n"/
> 
> Please don't.
> 
> That shall not become a 200 lines output. Please keep this on a single
> line.
> 
> It makes no sense to have all  useful  information  srolling  out  of
> sight all too quickly just for such bells and whistles.
> 
> Best regards,
> Wolfgang Denk

I was too terse.  The problem is that I added a progress bar. 
Previously, the printout sequence was:
   Writing to Flash...
followed by an indeterminate wait, followed by the string "done." which 
formed the composite:
   Writing to Flash... done.
I added a progress bar, but the progress bar needs to be on a new line. 
  I fixed this in cmd_mem.c, but not in the env code, resulting in 
Stefan's complaint (b).

With the (next version of the) progress dots patch, the sequence will be:
   Writing to Flash
                                                    v
   ..................................................
   done.

One thing I am debating is to add a parameter to the flash write command 
that says whether to do the progress dots or not.  Then we could 
suppress the (silly) dots on short writes.

Best regards,
gvb

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to