Hi Wolfgang,

please do not remove the spinning cursor from the 'esd' files.

a) Changing this will trigger some documentation update processes which I do not like :-) b) Your patch results in 10 dots per second. Sometimes the touched boards stay a long time in the waiting states and I want to keep the previous messages in my console. Lots of dots will scroll them out. c) I like the spinning cursor :-) d) The control characters did not cause trouble for me or our customers so far.

So please accept my NAK.

Matthias

On Thursday 23 October 2008 12:59, Wolfgang Denk 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

Hey, seems I'm heading for the record of submitting the maximum number of patch versions in the shorted time :-(

This patch version fixes a few typos in the first version, and gets rid of a few variables no longer needed because of the changes.

board/esd/common/cmd_loadpci.c | 6 +----- board/esd/pci405/cmd_pci405.c | 6 +----- board/esd/pmc440/cmd_pmc440.c | 6 +----- cpu/ppc4xx/44x_spd_ddr2.c | 14 ++++---------- cpu/ppc4xx/4xx_ibm_ddr2_autocalib.c | 16 ++++------------ 5 files changed, 11 insertions(+), 37 deletions(-)

diff --git a/board/esd/common/cmd_loadpci.c b/board/esd/common/cmd_loadpci.c index d88b387..aaf8b0c 100644 --- a/board/esd/common/cmd_loadpci.c +++ b/board/esd/common/cmd_loadpci.c @@ -38,9 +38,7 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) { unsigned int *ptr = 0; int count = 0;

• int count2 = 0; char addr[16];

• char str[] = "\|/-"; char *local_args[2];

  while(1) {

@@ -59,9 +57,7 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) while (*ptr == 0xffffffff) { count++; if (!(count % 100)) {

• 		count2++;
  

• 		putc(0x08); /* backspace */
  

• 		putc(str[count2 % 4]);
  

• 		putc('.');
  }
  
  /* Abort if ctrl-c was pressed */
  

diff --git a/board/esd/pci405/cmd_pci405.c b/board/esd/pci405/cmd_pci405.c index 5c717e2..9ffe686 100644 --- a/board/esd/pci405/cmd_pci405.c +++ b/board/esd/pci405/cmd_pci405.c @@ -47,11 +47,9 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) { unsigned int *ptr = 0; int count = 0;

• int count2 = 0; int status; int i; char addr[16];

• char str[] = "\|/-"; char *local_args[2];

  /*

@@ -67,9 +65,7 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) while (*ptr == 0xffffffff) { count++; if (!(count % 100)) {

• 	count2++;
  

• 	putc(0x08); /* backspace */
  

• 	putc(str[count2 % 4]);
  

• 	putc('.');
  

  }

  /* Abort if ctrl-c was pressed */

diff --git a/board/esd/pmc440/cmd_pmc440.c b/board/esd/pmc440/cmd_pmc440.c index 38ee74e..42d2fe6 100644 --- a/board/esd/pmc440/cmd_pmc440.c +++ b/board/esd/pmc440/cmd_pmc440.c @@ -121,10 +121,8 @@ int do_fifo(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) int i; int n = 0; u32 ctrl, data, f;

• char str[] = "\|/-"; int abort = 0; int count = 0;

• int count2 = 0;

  switch (argc) { case 1:

@@ -174,9 +172,7 @@ int do_fifo(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) while (!got_fifoirq) { count++; if (!(count % 100)) {

• 				count2++;
  

• 				putc(0x08); /* backspace */
  

• 				putc(str[count2 % 4]);
  

• 				putc('.');
  		}
  
  		/* Abort if ctrl-c was pressed */
  

diff --git a/cpu/ppc4xx/44x_spd_ddr2.c b/cpu/ppc4xx/44x_spd_ddr2.c index b40e4b1..586fa74 100644 --- a/cpu/ppc4xx/44x_spd_ddr2.c +++ b/cpu/ppc4xx/44x_spd_ddr2.c @@ -2362,10 +2362,8 @@ static void program_ecc_addr(unsigned long start_address, unsigned long end_address; unsigned long address_increment; unsigned long mcopt1;

• char str[] = "ECC generation -";
• char slash[] = "\|/-\|/-";

• char str[] = "ECC generation: "; int loop = 0;

• int loopi = 0;

  current_address = start_address; mfsdram(SDRAM_MCOPT1, mcopt1);

@@ -2390,8 +2388,7 @@ static void program_ecc_addr(unsigned long start_address, current_address += address_increment;

		if ((loop++ % (2 << 20)) == 0) {

• 			putc('\b');
  

• 			putc(slash[loopi++ % 8]);
  

• 			putc('.');
  	}
  }
  
  

@@ -2571,9 +2568,7 @@ static void DQS_calibration_process(void) u32 rqfd; u32 rqfd_start; u32 rqfd_average;

• int loopi = 0;
• char str[] = "Auto calibration -";
• char slash[] = "\|/-\|/-";

• char str[] = "Auto calibration: ";

  /*------------------------------------------------------------------

  • Test to determine the best read clock delay tuning bits.

@@ -2760,8 +2755,7 @@ calibration_loop: *-----------------------------------------------------------------*/ if (window_found == FALSE) { if (rqfd_start < SDRAM_RQDC_RQFD_MAX) {

• 	putc('\b');
  

• 	putc(slash[loopi++ % 8]);
  

• 	putc('.');
  
  /* try again from with a different RQFD start value */
  rqfd_start++;
  

diff --git a/cpu/ppc4xx/4xx_ibm_ddr2_autocalib.c b/cpu/ppc4xx/4xx_ibm_ddr2_autocalib.c index 1e3e20d..109bcc7 100644 --- a/cpu/ppc4xx/4xx_ibm_ddr2_autocalib.c +++ b/cpu/ppc4xx/4xx_ibm_ddr2_autocalib.c @@ -428,9 +428,6 @@ static u32 DQS_calibration_methodA(struct ddrautocal *cal) #endif ulong rdcc;

• char slash[] = "\|/-\|/-";
• int loopi = 0;
• /* start */ in_window = 0;

@@ -483,15 +480,13 @@ static u32 DQS_calibration_methodA(struct ddrautocal *cal) debug("<%s>SDRAM_RFDC=0x%x\n", __func__, temp); #endif

• putc(' '); for (rqfd = 0; rqfd <= SDRAM_RQDC_RQFD_MAX; rqfd++) {

  mfsdram(SDRAM_RQDC, rqdc_reg); rqdc_reg &= ~(SDRAM_RQDC_RQFD_MASK); mtsdram(SDRAM_RQDC, rqdc_reg | SDRAM_RQDC_RQFD_ENCODE(rqfd));

• putc('\b');
  

• putc(slash[loopi++ % 8]);
  

• putc('.');
  
  

  curr_win_min.rffd = 0; curr_win_max.rffd = 0;

@@ -944,8 +939,6 @@ u32 DQS_autocalibration(void) u32 val; int verbose_lvl = 0; char *str;

• char slash[] = "\|/-\|/-";
• int loopi = 0; struct sdram_timing *scan_list;

#if defined(DEBUG_PPC4xx_DDR_AUTOCALIBRATION) @@ -962,9 +955,9 @@ u32 DQS_autocalibration(void)

mfsdram(SDRAM_MCOPT1, val); if ((val & SDRAM_MCOPT1_MCHK_CHK_REP) == SDRAM_MCOPT1_MCHK_CHK_REP)

• str = "ECC Auto calibration -";
  

• str = "ECC Auto calibration: ";
  

  else

• str = "Auto calibration -";
  

• str = "Auto calibration: ";
  
  

  puts(str);

@@ -1000,8 +993,7 @@ u32 DQS_autocalibration(void)

relock_memory_DLL();

• putc('\b');
  

• putc(slash[loopi++ % 8]);
  

• putc('.');
  
  

#ifdef DEBUG debug("\n");