All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH PPC_4xx V3] Print simple row of dots instead of spinning wheel
Date: Fri, 31 Oct 2008 08:54:27 +0100	[thread overview]
Message-ID: <200810310854.28159.sr@denx.de> (raw)
In-Reply-To: <20081030231942.40B098522296@gemini.denx.de>

Hi Wolfgang,

On Friday 31 October 2008, Wolfgang Denk wrote:
> Dear Stefan & Matthias,
>
> In message <1224759565-1334-1-git-send-email-wd@denx.de> you wrote:
> > Replace the "spinning wheel" eye candy by printing a simple  row  of
> > dots. This avoids problems with control charactersin log files etc.
> >
> > Also, it saves a few bytes.
> >
> > Signed-off-by: Wolfgang Denk <wd@denx.de>
>
> Did you come to a decision about this patch?

Matthias already did send a NAK for the esd boards (on Monday).

I have to admit that I like the spinning wheel better as well. Especially the 
resulting output after the calibration is finished looks better on 
the "normal" terminal:

DRAM:  256 MB (original version)

instead of:

DRAM:  Auto calibration: ..........256 MB (your new version)

I understand that the spinning wheel is not optimal for logfiles, but for the 
normal usage it just looks nicer for my taste. So I would prefer to stay with 
the spinning wheel, but if you decide that the 4xx DDR2 calibration mustn't 
use this, then let's apply your patch for the cpu/ppc4xx files.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  reply	other threads:[~2008-10-31  7:54 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-23 10:59 [U-Boot] [PATCH PPC_4xx V3] Print simple row of dots instead of spinning wheel Wolfgang Denk
2008-10-23 11:08 ` Stefan Roese
2008-10-23 11:17   ` Wolfgang Denk
2008-10-28 23:05   ` Wolfgang Denk
2008-10-27 14:18 ` Matthias Fuchs
2008-10-30 23:19 ` Wolfgang Denk
2008-10-31  7:54   ` Stefan Roese [this message]
2008-10-31  8:45     ` Wolfgang Denk
2008-10-31  9:09       ` Stefan Roese
2008-10-31 11:20         ` Wolfgang Denk
2008-10-31 12:06           ` Stefan Roese
2008-10-31 12:41             ` Jean-Christophe PLAGNIOL-VILLARD

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=200810310854.28159.sr@denx.de \
    --to=sr@denx.de \
    --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.