[U-Boot] [PATCH] cmd_bdinfo: replace print_str() with print_mhz()

[U-Boot] [PATCH] cmd_bdinfo: replace print_str() with print_mhz()

[Timur Tabi]

The print_str() helper function for cmd_bdinfo can print any string, but it
is only used to print MHz values.  Replace it with print_mhz() that takes
a number and converts it to a string internally.

Signed-off-by: Timur Tabi <timur@freescale.com>
---

This patch applies on top of "powerpc/85xx: don't display address map size
(32-bit vs. 36-bit) during boot".

I've only tested this on PowerPC.

 common/cmd_bdinfo.c |   72 ++++++++++++++++++++++++--------------------------
 1 files changed, 35 insertions(+), 37 deletions(-)

diff --git a/common/cmd_bdinfo.c b/common/cmd_bdinfo.c
index 12863f2..3f7f6e6 100644
--- a/common/cmd_bdinfo.c
+++ b/common/cmd_bdinfo.c
@@ -40,12 +40,11 @@ static void print_lnum(const char *, u64);
 #endif
 
 #if defined(CONFIG_PPC)
-static void print_str(const char *, const char *);
+static void print_mhz(const char *, unsigned long);
 
 int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
 	bd_t *bd = gd->bd;
-	char buf[32];
 
 #ifdef DEBUG
 	print_num("bd address",		(ulong)bd);
@@ -68,32 +67,32 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 	defined(CONFIG_440GR) || defined(CONFIG_440GRX) || \
 	defined(CONFIG_440SP) || defined(CONFIG_440SPE) || \
 	defined(CONFIG_XILINX_405)
-	print_str("procfreq",		strmhz(buf, bd->bi_procfreq));
-	print_str("plb_busfreq",	strmhz(buf, bd->bi_plb_busfreq));
+	print_mhz("procfreq",		bd->bi_procfreq);
+	print_mhz("plb_busfreq",	bd->bi_plb_busfreq);
 #if	defined(CONFIG_405EP) || defined(CONFIG_405GP) || \
 	defined(CONFIG_440EP) || defined(CONFIG_440EPX) || \
 	defined(CONFIG_440GR) || defined(CONFIG_440GRX) || \
 	defined(CONFIG_440SPE) || defined(CONFIG_XILINX_405)
-	print_str("pci_busfreq",	strmhz(buf, bd->bi_pci_busfreq));
+	print_mhz("pci_busfreq",	bd->bi_pci_busfreq);
 #endif
 #else	/* ! CONFIG_405GP, CONFIG_405CR, CONFIG_405EP, CONFIG_XILINX_405, CONFIG_440EP CONFIG_440GR */
 #if defined(CONFIG_CPM2)
-	print_str("vco",		strmhz(buf, bd->bi_vco));
-	print_str("sccfreq",		strmhz(buf, bd->bi_sccfreq));
-	print_str("brgfreq",		strmhz(buf, bd->bi_brgfreq));
+	print_mhz("vco",		bd->bi_vco);
+	print_mhz("sccfreq",		bd->bi_sccfreq);
+	print_mhz("brgfreq",		bd->bi_brgfreq);
 #endif
-	print_str("intfreq",		strmhz(buf, bd->bi_intfreq));
+	print_mhz("intfreq",		bd->bi_intfreq);
 #if defined(CONFIG_CPM2)
-	print_str("cpmfreq",		strmhz(buf, bd->bi_cpmfreq));
+	print_mhz("cpmfreq",		bd->bi_cpmfreq);
 #endif
-	print_str("busfreq",		strmhz(buf, bd->bi_busfreq));
+	print_mhz("busfreq",		bd->bi_busfreq);
 #endif /* CONFIG_405GP, CONFIG_405CR, CONFIG_405EP, CONFIG_XILINX_405, CONFIG_440EP CONFIG_440GR */
 #if defined(CONFIG_MPC8220)
-	print_str("inpfreq",		strmhz(buf, bd->bi_inpfreq));
-	print_str("flbfreq",		strmhz(buf, bd->bi_flbfreq));
-	print_str("pcifreq",		strmhz(buf, bd->bi_pcifreq));
-	print_str("vcofreq",		strmhz(buf, bd->bi_vcofreq));
-	print_str("pevfreq",		strmhz(buf, bd->bi_pevfreq));
+	print_mhz("inpfreq",		bd->bi_inpfreq);
+	print_mhz("flbfreq",		bd->bi_flbfreq);
+	print_mhz("pcifreq",		bd->bi_pcifreq);
+	print_mhz("vcofreq",		bd->bi_vcofreq);
+	print_mhz("pevfreq",		bd->bi_pevfreq);
 #endif
 
 #ifdef CONFIG_ENABLE_36BIT_PHYS
@@ -122,7 +121,7 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 #endif
 
 #ifdef CONFIG_HERMES
-	print_str("ethspeed",		strmhz(buf, bd->bi_ethspeed));
+	print_mhz("ethspeed",		bd->bi_ethspeed);
 #endif
 	printf("IP addr     = %pI4\n", &bd->bi_ip_addr);
 	printf("baudrate    = %6ld bps\n", bd->bi_baudrate);
@@ -214,12 +213,11 @@ int do_bdinfo(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[])
 
 #elif defined(CONFIG_M68K)
 
-static void print_str(const char *, const char *);
+static void print_mhz(const char *, unsigned long);
 
 int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
 	bd_t *bd = gd->bd;
-	char buf[32];
 
 	print_num("memstart",		(ulong)bd->bi_memstart);
 	print_lnum("memsize",		(u64)bd->bi_memsize);
@@ -233,15 +231,15 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 #if defined(CONFIG_SYS_MBAR)
 	print_num("mbar",		bd->bi_mbar_base);
 #endif
-	print_str("cpufreq",		strmhz(buf, bd->bi_intfreq));
-	print_str("busfreq",		strmhz(buf, bd->bi_busfreq));
+	print_mhz("cpufreq",		bd->bi_intfreq);
+	print_mhz("busfreq",		bd->bi_busfreq);
 #ifdef CONFIG_PCI
-	print_str("pcifreq",		strmhz(buf, bd->bi_pcifreq));
+	print_mhz("pcifreq",		bd->bi_pcifreq);
 #endif
 #ifdef CONFIG_EXTRA_CLOCK
-	print_str("flbfreq",		strmhz(buf, bd->bi_flbfreq));
-	print_str("inpfreq",		strmhz(buf, bd->bi_inpfreq));
-	print_str("vcofreq",		strmhz(buf, bd->bi_vcofreq));
+	print_mhz("flbfreq",		bd->bi_flbfreq);
+	print_mhz("inpfreq",		bd->bi_inpfreq);
+	print_mhz("vcofreq",		bd->bi_vcofreq);
 #endif
 #if defined(CONFIG_CMD_NET)
 	print_eth(0);
@@ -264,19 +262,18 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
 #elif defined(CONFIG_BLACKFIN)
 
-static void print_str(const char *, const char *);
+static void print_mhz(const char *, unsigned long);
 
 int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
 	bd_t *bd = gd->bd;
-	char buf[32];
 
 	printf("U-Boot      = %s\n", bd->bi_r_version);
 	printf("CPU         = %s\n", bd->bi_cpu);
 	printf("Board       = %s\n", bd->bi_board_name);
-	print_str("VCO",	strmhz(buf, bd->bi_vco));
-	print_str("CCLK",	strmhz(buf, bd->bi_cclk));
-	print_str("SCLK",	strmhz(buf, bd->bi_sclk));
+	print_mhz("VCO",	bd->bi_vco);
+	print_mhz("CCLK",	bd->bi_cclk);
+	print_mhz("SCLK",	bd->bi_sclk);
 
 	print_num("boot_params",	(ulong)bd->bi_boot_params);
 	print_num("memstart",		(ulong)bd->bi_memstart);
@@ -385,13 +382,12 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
 #elif defined(CONFIG_X86)
 
-static void print_str(const char *, const char *);
+static void print_mhz(const char *, unsigned long);
 
 int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
 	int i;
 	bd_t *bd = gd->bd;
-	char buf[32];
 
 	print_num("boot_params",	(ulong)bd->bi_boot_params);
 	print_num("bi_memstart",	bd->bi_memstart);
@@ -402,8 +398,8 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 	print_num("bi_sramstart",	bd->bi_sramstart);
 	print_num("bi_sramsize",	bd->bi_sramsize);
 	print_num("bi_bootflags",	bd->bi_bootflags);
-	print_str("cpufreq",		strmhz(buf, bd->bi_intfreq));
-	print_str("busfreq",		strmhz(buf, bd->bi_busfreq));
+	print_mhz("cpufreq",		bd->bi_intfreq);
+	print_mhz("busfreq",		bd->bi_busfreq);
 
 	for (i = 0; i < CONFIG_NR_DRAM_BANKS; ++i) {
 		print_num("DRAM bank",	i);
@@ -414,7 +410,7 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 #if defined(CONFIG_CMD_NET)
 	print_eth(0);
 	printf("ip_addr     = %pI4\n", &bd->bi_ip_addr);
-	print_str("ethspeed",	    strmhz(buf, bd->bi_ethspeed));
+	print_mhz("ethspeed",	    bd->bi_ethspeed);
 #endif
 	printf("baudrate    = %d bps\n", bd->bi_baudrate);
 
@@ -456,9 +452,11 @@ static void print_lnum(const char *name, u64 value)
 	defined(CONFIG_M68K) || \
 	defined(CONFIG_BLACKFIN) || \
 	defined(CONFIG_X86)
-static void print_str(const char *name, const char *str)
+static void print_mhz(const char *name, unsigned long hz)
 {
-	printf("%-12s= %6s MHz\n", name, str);
+	char buf[32];
+
+	printf("%-12s= %6s MHz\n", name, strmhz(buf, hz));
 }
 #endif	/* CONFIG_PPC */
 
-- 
1.7.3.4

[U-Boot] [PATCH] cmd_bdinfo: replace print_str() with print_mhz()

[Mike Frysinger]

Tested-by: Mike Frysinger <vapier@gentoo.org>
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111009/f111ef74/attachment.pgp 

[U-Boot] [PATCH] cmd_bdinfo: replace print_str() with print_mhz()

[Wolfgang Denk]

Dear Timur Tabi,

In message <1317852487-12499-1-git-send-email-timur@freescale.com> you wrote:
> The print_str() helper function for cmd_bdinfo can print any string, but it
> is only used to print MHz values.  Replace it with print_mhz() that takes
> a number and converts it to a string internally.
> 
> Signed-off-by: Timur Tabi <timur@freescale.com>
> ---
> 
> This patch applies on top of "powerpc/85xx: don't display address map size
> (32-bit vs. 36-bit) during boot".
> 
> I've only tested this on PowerPC.
> 
>  common/cmd_bdinfo.c |   72 ++++++++++++++++++++++++--------------------------
>  1 files changed, 35 insertions(+), 37 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
8) Use common sense in routing  cable.  Avoid  wrapping  coax  around
   sources  of  strong  electric  or magnetic fields. Do not wrap the
   cable  around  flourescent  light  ballasts  or  cyclotrons,   for
   example.
- Ethernet Headstart Product, Information and Installation Guide,
   Bell Technologies, pg. 11