Patchwork [U-Boot] MPC8xxx: drop redundant boot messages

login
register
mail settings
Submitter Wolfgang Denk
Date July 25, 2011, 8:13 a.m.
Message ID <1311581633-6205-1-git-send-email-wd@denx.de>
Download mbox | patch
Permalink /patch/106618/
State Accepted
Commit 21cd5815a7e956e689ac78be2ade05e61fd5dce7
Delegated to: Kumar Gala
Headers show

Comments

Wolfgang Denk - July 25, 2011, 8:13 a.m.
Current code would print RAM size information like this:

	DRAM:  DDR: 256 MiB (DDR1, 64-bit, CL=2, ECC off)

Turn a number of printf()s into debug() to get rid of the redundant
"DDR: " string like this:

	DRAM:  256 MiB (DDR1, 64-bit, CL=2, ECC off)

Signed-off-by: Wolfgang Denk <wd@denx.de>
Cc: Kumar Gala <galak@kernel.crashing.org>
---
 arch/powerpc/cpu/mpc85xx/cpu.c            |    2 +-
 board/freescale/corenet_ds/ddr.c          |    2 +-
 board/freescale/mpc8610hpcd/mpc8610hpcd.c |    2 +-
 board/freescale/mpc8641hpcn/mpc8641hpcn.c |    2 +-
 board/freescale/p2041rdb/ddr.c            |    2 +-
 board/sbc8641d/sbc8641d.c                 |    2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)
York Sun - July 28, 2011, 5:47 p.m.
On Mon, 2011-07-25 at 10:13 +0200, Wolfgang Denk wrote:
> Current code would print RAM size information like this:
> 
> 	DRAM:  DDR: 256 MiB (DDR1, 64-bit, CL=2, ECC off)
> 
> Turn a number of printf()s into debug() to get rid of the redundant
> "DDR: " string like this:
> 
> 	DRAM:  256 MiB (DDR1, 64-bit, CL=2, ECC off)
> 
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> Cc: Kumar Gala <galak@kernel.crashing.org>
> ---
>  arch/powerpc/cpu/mpc85xx/cpu.c            |    2 +-
>  board/freescale/corenet_ds/ddr.c          |    2 +-
>  board/freescale/mpc8610hpcd/mpc8610hpcd.c |    2 +-
>  board/freescale/mpc8641hpcn/mpc8641hpcn.c |    2 +-
>  board/freescale/p2041rdb/ddr.c            |    2 +-
>  board/sbc8641d/sbc8641d.c                 |    2 +-
>  6 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/powerpc/cpu/mpc85xx/cpu.c b/arch/powerpc/cpu/mpc85xx/cpu.c
> index 53f0887..8ee3b4f 100644
> --- a/arch/powerpc/cpu/mpc85xx/cpu.c
> +++ b/arch/powerpc/cpu/mpc85xx/cpu.c
> @@ -358,7 +358,7 @@ phys_size_t initdram(int board_type)
>  	lbc_sdram_init();
>  #endif
>  
> -	puts("DDR: ");
> +	debug("DDR: ");
>  	return dram_size;
>  }
>  #endif /* CONFIG_SYS_RAMBOOT */
> diff --git a/board/freescale/corenet_ds/ddr.c b/board/freescale/corenet_ds/ddr.c
> index a184592..b937015 100644
> --- a/board/freescale/corenet_ds/ddr.c
> +++ b/board/freescale/corenet_ds/ddr.c
> @@ -256,6 +256,6 @@ phys_size_t initdram(int board_type)
>  	dram_size = setup_ddr_tlbs(dram_size / 0x100000);
>  	dram_size *= 0x100000;
>  
> -	puts("    DDR: ");
> +	debug("    DDR: ");
>  	return dram_size;
>  }
> diff --git a/board/freescale/mpc8610hpcd/mpc8610hpcd.c b/board/freescale/mpc8610hpcd/mpc8610hpcd.c
> index 4e4b7c0..8aceddb 100644
> --- a/board/freescale/mpc8610hpcd/mpc8610hpcd.c
> +++ b/board/freescale/mpc8610hpcd/mpc8610hpcd.c
> @@ -145,7 +145,7 @@ initdram(int board_type)
>  
>  	setup_ddr_bat(dram_size);
>  
> -	puts(" DDR: ");
> +	debug(" DDR: ");
>  	return dram_size;
>  }
>  
> diff --git a/board/freescale/mpc8641hpcn/mpc8641hpcn.c b/board/freescale/mpc8641hpcn/mpc8641hpcn.c
> index e3916fc..455569e 100644
> --- a/board/freescale/mpc8641hpcn/mpc8641hpcn.c
> +++ b/board/freescale/mpc8641hpcn/mpc8641hpcn.c
> @@ -69,7 +69,7 @@ initdram(int board_type)
>  
>  	setup_ddr_bat(dram_size);
>  
> -	puts("    DDR: ");
> +	debug("    DDR: ");
>  	return dram_size;
>  }
>  
> diff --git a/board/freescale/p2041rdb/ddr.c b/board/freescale/p2041rdb/ddr.c
> index 46de910..e9c699c 100644
> --- a/board/freescale/p2041rdb/ddr.c
> +++ b/board/freescale/p2041rdb/ddr.c
> @@ -110,6 +110,6 @@ phys_size_t initdram(int board_type)
>  	dram_size = setup_ddr_tlbs(dram_size / 0x100000);
>  	dram_size *= 0x100000;
>  
> -	puts("    DDR: ");
> +	debug("    DDR: ");
>  	return dram_size;
>  }
> diff --git a/board/sbc8641d/sbc8641d.c b/board/sbc8641d/sbc8641d.c
> index dd58541..bed8f53 100644
> --- a/board/sbc8641d/sbc8641d.c
> +++ b/board/sbc8641d/sbc8641d.c
> @@ -63,7 +63,7 @@ phys_size_t initdram (int board_type)
>  	dram_size = fixed_sdram ();
>  #endif
>  
> -	puts ("    DDR: ");
> +	debug ("    DDR: ");
>  	return dram_size;
>  }
>  

Acked-by: York Sun <yorksun@freescale.com>
Kumar Gala - July 28, 2011, 7:52 p.m.
On Jul 25, 2011, at 3:13 AM, Wolfgang Denk wrote:

> Current code would print RAM size information like this:
> 
> 	DRAM:  DDR: 256 MiB (DDR1, 64-bit, CL=2, ECC off)
> 
> Turn a number of printf()s into debug() to get rid of the redundant
> "DDR: " string like this:
> 
> 	DRAM:  256 MiB (DDR1, 64-bit, CL=2, ECC off)
> 
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> Cc: Kumar Gala <galak@kernel.crashing.org>
> ---
> arch/powerpc/cpu/mpc85xx/cpu.c            |    2 +-
> board/freescale/corenet_ds/ddr.c          |    2 +-
> board/freescale/mpc8610hpcd/mpc8610hpcd.c |    2 +-
> board/freescale/mpc8641hpcn/mpc8641hpcn.c |    2 +-
> board/freescale/p2041rdb/ddr.c            |    2 +-
> board/sbc8641d/sbc8641d.c                 |    2 +-
> 6 files changed, 6 insertions(+), 6 deletions(-)

applied to 85xx

- k

Patch

diff --git a/arch/powerpc/cpu/mpc85xx/cpu.c b/arch/powerpc/cpu/mpc85xx/cpu.c
index 53f0887..8ee3b4f 100644
--- a/arch/powerpc/cpu/mpc85xx/cpu.c
+++ b/arch/powerpc/cpu/mpc85xx/cpu.c
@@ -358,7 +358,7 @@  phys_size_t initdram(int board_type)
 	lbc_sdram_init();
 #endif
 
-	puts("DDR: ");
+	debug("DDR: ");
 	return dram_size;
 }
 #endif /* CONFIG_SYS_RAMBOOT */
diff --git a/board/freescale/corenet_ds/ddr.c b/board/freescale/corenet_ds/ddr.c
index a184592..b937015 100644
--- a/board/freescale/corenet_ds/ddr.c
+++ b/board/freescale/corenet_ds/ddr.c
@@ -256,6 +256,6 @@  phys_size_t initdram(int board_type)
 	dram_size = setup_ddr_tlbs(dram_size / 0x100000);
 	dram_size *= 0x100000;
 
-	puts("    DDR: ");
+	debug("    DDR: ");
 	return dram_size;
 }
diff --git a/board/freescale/mpc8610hpcd/mpc8610hpcd.c b/board/freescale/mpc8610hpcd/mpc8610hpcd.c
index 4e4b7c0..8aceddb 100644
--- a/board/freescale/mpc8610hpcd/mpc8610hpcd.c
+++ b/board/freescale/mpc8610hpcd/mpc8610hpcd.c
@@ -145,7 +145,7 @@  initdram(int board_type)
 
 	setup_ddr_bat(dram_size);
 
-	puts(" DDR: ");
+	debug(" DDR: ");
 	return dram_size;
 }
 
diff --git a/board/freescale/mpc8641hpcn/mpc8641hpcn.c b/board/freescale/mpc8641hpcn/mpc8641hpcn.c
index e3916fc..455569e 100644
--- a/board/freescale/mpc8641hpcn/mpc8641hpcn.c
+++ b/board/freescale/mpc8641hpcn/mpc8641hpcn.c
@@ -69,7 +69,7 @@  initdram(int board_type)
 
 	setup_ddr_bat(dram_size);
 
-	puts("    DDR: ");
+	debug("    DDR: ");
 	return dram_size;
 }
 
diff --git a/board/freescale/p2041rdb/ddr.c b/board/freescale/p2041rdb/ddr.c
index 46de910..e9c699c 100644
--- a/board/freescale/p2041rdb/ddr.c
+++ b/board/freescale/p2041rdb/ddr.c
@@ -110,6 +110,6 @@  phys_size_t initdram(int board_type)
 	dram_size = setup_ddr_tlbs(dram_size / 0x100000);
 	dram_size *= 0x100000;
 
-	puts("    DDR: ");
+	debug("    DDR: ");
 	return dram_size;
 }
diff --git a/board/sbc8641d/sbc8641d.c b/board/sbc8641d/sbc8641d.c
index dd58541..bed8f53 100644
--- a/board/sbc8641d/sbc8641d.c
+++ b/board/sbc8641d/sbc8641d.c
@@ -63,7 +63,7 @@  phys_size_t initdram (int board_type)
 	dram_size = fixed_sdram ();
 #endif
 
-	puts ("    DDR: ");
+	debug ("    DDR: ");
 	return dram_size;
 }