Re: [U-Boot] [PATCH v4 12/42] Move strtomhz() to vsprintf.h

2019-11-19 Thread Tom Rini
On Thu, Nov 14, 2019 at 12:57:20PM -0700, Simon Glass wrote:

> At present this function sits in its own file but it does not really
> justify it. There are similar string functions in vsprintf.h, so move it
> there. Also add the missing function comment.
> 
> Use the vsprintf.h include file explicitly where needed.
> 
> Signed-off-by: Simon Glass 

Reviewed-by: Tom Rini 

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH v4 12/42] Move strtomhz() to vsprintf.h

2019-11-14 Thread Simon Glass
At present this function sits in its own file but it does not really
justify it. There are similar string functions in vsprintf.h, so move it
there. Also add the missing function comment.

Use the vsprintf.h include file explicitly where needed.

Signed-off-by: Simon Glass 
---

Changes in v4: None
Changes in v3:
- Put copyright info into vsprintf.c

Changes in v2: None

 arch/arc/lib/cpu.c|  1 +
 arch/arm/cpu/arm926ejs/mx25/generic.c |  1 +
 arch/arm/cpu/arm926ejs/mx27/generic.c |  1 +
 arch/arm/cpu/armv7/ls102xa/cpu.c  |  1 +
 arch/arm/cpu/armv8/fsl-layerscape/cpu.c   |  1 +
 arch/arm/mach-at91/arm926ejs/cpu.c|  1 +
 arch/arm/mach-at91/armv7/cpu.c|  1 +
 arch/m68k/cpu/mcf5227x/cpu.c  |  1 +
 arch/m68k/cpu/mcf523x/cpu.c   |  1 +
 arch/m68k/cpu/mcf52x2/cpu.c   |  1 +
 arch/m68k/cpu/mcf530x/cpu.c   |  1 +
 arch/m68k/cpu/mcf532x/cpu.c   |  1 +
 arch/m68k/cpu/mcf5445x/cpu.c  |  1 +
 arch/m68k/cpu/mcf547x_8x/cpu.c|  1 +
 arch/powerpc/cpu/mpc83xx/cpu.c|  1 +
 arch/powerpc/cpu/mpc83xx/spd_sdram.c  |  1 +
 arch/powerpc/cpu/mpc83xx/speed.c  |  1 +
 arch/powerpc/cpu/mpc85xx/cpu.c|  1 +
 arch/powerpc/cpu/mpc86xx/cpu.c|  1 +
 arch/powerpc/cpu/mpc8xx/cpu.c |  1 +
 arch/xtensa/cpu/cpu.c |  1 +
 board/Arcturus/ucp1020/ddr.c  |  1 +
 board/atmel/at91sam9261ek/at91sam9261ek.c |  1 +
 board/atmel/at91sam9263ek/at91sam9263ek.c |  1 +
 .../atmel/at91sam9m10g45ek/at91sam9m10g45ek.c |  1 +
 board/atmel/at91sam9n12ek/at91sam9n12ek.c |  1 +
 board/atmel/at91sam9rlek/at91sam9rlek.c   |  1 +
 board/atmel/common/video_display.c|  1 +
 board/esd/meesc/meesc.c   |  1 +
 board/freescale/bsc9131rdb/ddr.c  |  1 +
 board/freescale/bsc9132qds/ddr.c  |  1 +
 board/freescale/corenet_ds/ddr.c  |  1 +
 board/freescale/ls1043ardb/ddr.c  |  1 +
 board/freescale/mpc8541cds/mpc8541cds.c   |  1 +
 board/freescale/mpc8548cds/mpc8548cds.c   |  1 +
 board/freescale/mpc8555cds/mpc8555cds.c   |  1 +
 board/freescale/p1010rdb/ddr.c|  1 +
 board/freescale/p1_p2_rdb_pc/ddr.c|  1 +
 board/freescale/p1_twr/ddr.c  |  1 +
 board/mini-box/picosam9g45/picosam9g45.c  |  1 +
 board/ronetix/pm9261/pm9261.c |  1 +
 cmd/bdinfo.c  |  1 +
 drivers/clk/clk_stm32mp1.c|  1 +
 drivers/clk/mpc83xx_clk.c |  1 +
 drivers/cpu/mpc83xx_cpu.c |  1 +
 include/common.h  |  3 ---
 include/vsprintf.h| 10 +
 lib/Makefile  |  4 ++--
 lib/strmhz.c  | 21 ---
 lib/vsprintf.c| 19 +
 50 files changed, 76 insertions(+), 26 deletions(-)
 delete mode 100644 lib/strmhz.c

diff --git a/arch/arc/lib/cpu.c b/arch/arc/lib/cpu.c
index 01cca95d5b..83246550ec 100644
--- a/arch/arc/lib/cpu.c
+++ b/arch/arc/lib/cpu.c
@@ -5,6 +5,7 @@
 
 #include 
 #include 
+#include 
 #include 
 #include 
 
diff --git a/arch/arm/cpu/arm926ejs/mx25/generic.c 
b/arch/arm/cpu/arm926ejs/mx25/generic.c
index 5fcf06ae1a..eeb61d0d10 100644
--- a/arch/arm/cpu/arm926ejs/mx25/generic.c
+++ b/arch/arm/cpu/arm926ejs/mx25/generic.c
@@ -11,6 +11,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
diff --git a/arch/arm/cpu/arm926ejs/mx27/generic.c 
b/arch/arm/cpu/arm926ejs/mx27/generic.c
index 08b1b4de71..9bed0e91be 100644
--- a/arch/arm/cpu/arm926ejs/mx27/generic.c
+++ b/arch/arm/cpu/arm926ejs/mx27/generic.c
@@ -7,6 +7,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
diff --git a/arch/arm/cpu/armv7/ls102xa/cpu.c b/arch/arm/cpu/armv7/ls102xa/cpu.c
index 9ccfe1042c..ec9984db79 100644
--- a/arch/arm/cpu/armv7/ls102xa/cpu.c
+++ b/arch/arm/cpu/armv7/ls102xa/cpu.c
@@ -4,6 +4,7 @@
  */
 
 #include 
+#include 
 #include 
 #include 
 #include 
diff --git a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c 
b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
index 83a3319321..59d03b167c 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
+++ b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
@@ -7,6 +7,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
diff --git a/arch/arm/mach-at91/arm926ejs/cpu.c 
b/arch/arm/mach-at91/arm926ejs/cpu.c
index 6f5aa4274b..e9b4e06595 100644
--- a/arch/arm/mach-at91/arm926ejs/cpu.c
+++ b/arch/arm/mach-at91/arm926ejs/cpu.c
@@ -7,6 +7,7 @@
  */
 
 #include 
+#include 
 #include 
 #include 
 #include 
diff --git a/arch/arm/mach-at91/armv7/cpu.c b/arch/arm/mach-at91/armv7/cpu.c
index