[U-Boot] [PATCH PPC_4XX V2] Print simple row of dots instead of spinning wheel

Wolfgang Denk wd at denx.de
Thu Oct 23 12:53:53 CEST 2008


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 at denx.de>
---
This patch version fixes a few typos and also gets rid of a few now
unused variables.

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

diff --git a/board/esd/common/cmd_loadpci.c b/board/esd/common/cmd_loadpci.c
index d88b387..31d45b1 100644
--- a/board/esd/common/cmd_loadpci.c
+++ b/board/esd/common/cmd_loadpci.c
@@ -40,7 +40,6 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 	int count = 0;
 	int count2 = 0;
 	char addr[16];
-	char str[] = "\\|/-";
 	char *local_args[2];
 
 	while(1) {
@@ -60,8 +59,7 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 			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..670bc19 100644
--- a/board/esd/pci405/cmd_pci405.c
+++ b/board/esd/pci405/cmd_pci405.c
@@ -51,7 +51,6 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 	int status;
 	int i;
 	char addr[16];
-	char str[] = "\\|/-";
 	char *local_args[2];
 
 	/*
@@ -68,8 +67,7 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 		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..7108263 100644
--- a/board/esd/pmc440/cmd_pmc440.c
+++ b/board/esd/pmc440/cmd_pmc440.c
@@ -121,7 +121,6 @@ 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;
@@ -175,8 +174,7 @@ int do_fifo(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 					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");
-- 
1.5.5.1



More information about the U-Boot mailing list