Move strtomhz() to vsprintf.h
authorSimon Glass <sjg@chromium.org>
Thu, 14 Nov 2019 19:57:20 +0000 (12:57 -0700)
committerTom Rini <trini@konsulko.com>
Mon, 2 Dec 2019 23:23:09 +0000 (18:23 -0500)
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 <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
50 files changed:
arch/arc/lib/cpu.c
arch/arm/cpu/arm926ejs/mx25/generic.c
arch/arm/cpu/arm926ejs/mx27/generic.c
arch/arm/cpu/armv7/ls102xa/cpu.c
arch/arm/cpu/armv8/fsl-layerscape/cpu.c
arch/arm/mach-at91/arm926ejs/cpu.c
arch/arm/mach-at91/armv7/cpu.c
arch/m68k/cpu/mcf5227x/cpu.c
arch/m68k/cpu/mcf523x/cpu.c
arch/m68k/cpu/mcf52x2/cpu.c
arch/m68k/cpu/mcf530x/cpu.c
arch/m68k/cpu/mcf532x/cpu.c
arch/m68k/cpu/mcf5445x/cpu.c
arch/m68k/cpu/mcf547x_8x/cpu.c
arch/powerpc/cpu/mpc83xx/cpu.c
arch/powerpc/cpu/mpc83xx/spd_sdram.c
arch/powerpc/cpu/mpc83xx/speed.c
arch/powerpc/cpu/mpc85xx/cpu.c
arch/powerpc/cpu/mpc86xx/cpu.c
arch/powerpc/cpu/mpc8xx/cpu.c
arch/xtensa/cpu/cpu.c
board/Arcturus/ucp1020/ddr.c
board/atmel/at91sam9261ek/at91sam9261ek.c
board/atmel/at91sam9263ek/at91sam9263ek.c
board/atmel/at91sam9m10g45ek/at91sam9m10g45ek.c
board/atmel/at91sam9n12ek/at91sam9n12ek.c
board/atmel/at91sam9rlek/at91sam9rlek.c
board/atmel/common/video_display.c
board/esd/meesc/meesc.c
board/freescale/bsc9131rdb/ddr.c
board/freescale/bsc9132qds/ddr.c
board/freescale/corenet_ds/ddr.c
board/freescale/ls1043ardb/ddr.c
board/freescale/mpc8541cds/mpc8541cds.c
board/freescale/mpc8548cds/mpc8548cds.c
board/freescale/mpc8555cds/mpc8555cds.c
board/freescale/p1010rdb/ddr.c
board/freescale/p1_p2_rdb_pc/ddr.c
board/freescale/p1_twr/ddr.c
board/mini-box/picosam9g45/picosam9g45.c
board/ronetix/pm9261/pm9261.c
cmd/bdinfo.c
drivers/clk/clk_stm32mp1.c
drivers/clk/mpc83xx_clk.c
drivers/cpu/mpc83xx_cpu.c
include/common.h
include/vsprintf.h
lib/Makefile
lib/strmhz.c [deleted file]
lib/vsprintf.c

index 01cca95d5b1854cd07029fd978b3cdc8f67beffa..83246550ec1250ecac153d131939b8f5491ad8b5 100644 (file)
@@ -5,6 +5,7 @@
 
 #include <common.h>
 #include <malloc.h>
+#include <vsprintf.h>
 #include <asm/arcregs.h>
 #include <asm/cache.h>
 
index 5fcf06ae1a82d439701d8465471bd5caa58483c9..eeb61d0d102028ee57887f32957a19ed61a8a119 100644 (file)
@@ -11,6 +11,7 @@
 #include <common.h>
 #include <div64.h>
 #include <netdev.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch-imx/cpu.h>
 #include <asm/arch/imx-regs.h>
index 08b1b4de71779fd5f5cefc98f52a8d5911e4fef1..9bed0e91bea670b085a007e2807ff7b9aac0a900 100644 (file)
@@ -7,6 +7,7 @@
 #include <common.h>
 #include <div64.h>
 #include <netdev.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/imx-regs.h>
 #include <asm/arch/clock.h>
index 9ccfe1042ce532f4c720c63dea7f64ed535c880c..ec9984db795695c1931e5cb5d6f5c1e456de4586 100644 (file)
@@ -4,6 +4,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/arch/clock.h>
 #include <asm/io.h>
 #include <asm/arch/immap_ls102xa.h>
index 83a33193211e8671010a88ce611d316460145ebc..59d03b167c46e25e10f68a53869ab2b79cdcac9c 100644 (file)
@@ -7,6 +7,7 @@
 #include <common.h>
 #include <env.h>
 #include <fsl_ddr_sdram.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <linux/errno.h>
 #include <asm/system.h>
index 6f5aa4274b595e110ca831eafb5ad76575910f4a..e9b4e06595e87cf24868d26eeb2462763b80636c 100644 (file)
@@ -7,6 +7,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/hardware.h>
 #include <asm/arch/at91_pit.h>
index 5da067cda10f9e348782d4dc9f6bdc91cd5b7aa7..638645c7e40dafdc86167640070386c99dd646ca 100644 (file)
@@ -9,6 +9,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/hardware.h>
 #include <asm/arch/at91_pit.h>
index 7ad023dac5a24e912a9738e332303fd39491a0fb..34534d876bc7b7880884bdaca62e10c31003057d 100644 (file)
@@ -9,6 +9,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 
index 79be04f7a0cb79aaaac088013a16919d672b01b2..429781945bfc038eb3bfa965f28bd9eea6827337 100644 (file)
@@ -9,6 +9,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <netdev.h>
index 29a17c57fcea0f382415eb3cc7f2023cc6becec5..b48a753f9b79ef3468eaca3e33c89b553ad17636 100644 (file)
@@ -14,6 +14,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <asm/immap.h>
index c7ae65afce9d03c3a17ae5f10ab6030450460929..a76deebc68ac366920ad84b59d418db07605777e 100644 (file)
@@ -5,6 +5,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/immap.h>
 #include <asm/io.h>
 
index a01b5e65a747c91b973d5f30773e131f5c67f98e..c8a1f205305dbe151703e0d3f9ea347cea15dfb3 100644 (file)
@@ -9,6 +9,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <netdev.h>
index 56e5585e09e3aedac73d39c562d1d3020e97d9a7..2f79380c8be4760ec7bd8b922cc56e4d7e7ab4e4 100644 (file)
@@ -9,6 +9,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <netdev.h>
index 819b25f280b7a9a266d27d1adc5cc9b7c4c0c5e8..dc5ed1aa79fda2badc33fbc3c8f96bb65e875c58 100644 (file)
@@ -9,6 +9,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <netdev.h>
index 3048ecf34ad326b6440347c84e07504288082ac3..363c0ffd2859e4ac36997f52cc40e875fcdfada5 100644 (file)
@@ -10,6 +10,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <mpc83xx.h>
index 8b5ecdb9ad1386de1365c391618a0269ad80da75..08918a9d75e1ac36bcbc60ebc29d26ae787f3775 100644 (file)
@@ -13,6 +13,7 @@
 #ifndef CONFIG_MPC83XX_SDRAM
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/processor.h>
 #include <asm/io.h>
 #include <i2c.h>
index e118a10fa8b439565e21f3c0a24bde062d659ce1..93af7f495fd3614e808626e539025630b1541aa3 100644 (file)
@@ -11,6 +11,7 @@
 #include <common.h>
 #include <mpc83xx.h>
 #include <command.h>
+#include <vsprintf.h>
 #include <asm/processor.h>
 
 DECLARE_GLOBAL_DATA_PTR;
index 5344b175a45d3975dcc7a5a48dd070f4c1d27427..64024612cd516fcd6e8269ccd1c6eb7a06ffc61c 100644 (file)
@@ -11,6 +11,7 @@
 #include <config.h>
 #include <common.h>
 #include <cpu_func.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <fsl_esdhc.h>
index c023d0684f8ad065e6ca3114bbe76e4ce0417c81..1c3c78217cb9c11674be4d0bd62f314f4092ece6 100644 (file)
@@ -6,6 +6,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <asm/cache.h>
index 798eabdc21c1db9dcf0cbbc0d5c7db80458f0023..6ad86e9a1d72aab3a416bc541099e99a281aa06c 100644 (file)
@@ -17,6 +17,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <watchdog.h>
 #include <command.h>
 #include <mpc8xx.h>
index 64bb0b6ee0bd45bdc1ad2b171bd6837b44de099c..6f4b88f88e1243daea31431a0ba59686fa244726 100644 (file)
@@ -10,6 +10,7 @@
 
 #include <common.h>
 #include <command.h>
+#include <vsprintf.h>
 #include <linux/stringify.h>
 #include <asm/global_data.h>
 #include <asm/cache.h>
index 4b84a1a4c0154de658a57c2ed0f97e9cd83dd792..a3285ebe5cdffed2254b216ce8bb436a37c2f8ed 100644 (file)
@@ -8,6 +8,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
 #include <asm/processor.h>
index 6f15bc677671a3b9d80321c6f0a2206573d87219..28765315f74f6caf435b9f79d968a6a83d8887d3 100644 (file)
@@ -7,6 +7,7 @@
 
 #include <common.h>
 #include <debug_uart.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/at91sam9261.h>
 #include <asm/arch/at91sam9261_matrix.h>
index 2d882174de0d5c9f68be3efd3fa6029153432415..31bb72c0cdc3758bb2261d0cf71638e1fb6323be 100644 (file)
@@ -7,6 +7,7 @@
 
 #include <common.h>
 #include <debug_uart.h>
+#include <vsprintf.h>
 #include <linux/sizes.h>
 #include <asm/arch/at91sam9263.h>
 #include <asm/arch/at91sam9_smc.h>
index 3b277fc575ee885463293b46044dfd4cc123be96..85004310b1fcd2b1abd9412e5a78a1b21a5dc0eb 100644 (file)
@@ -7,6 +7,7 @@
 
 #include <common.h>
 #include <debug_uart.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/clk.h>
 #include <asm/arch/at91sam9g45_matrix.h>
index b2bb7ec9928568acb4fa39c53d0f92cc4cd13c5c..51b24e1d5922de27b9d43035342a8854ce1fd60e 100644 (file)
@@ -5,6 +5,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/at91sam9x5_matrix.h>
 #include <asm/arch/at91sam9_smc.h>
index 0393307315766c2e96a489216196d40ee57b4f7a..4e674d49b31876bed96f1a82d7e5ef5145bcb856 100644 (file)
@@ -7,6 +7,7 @@
 
 #include <common.h>
 #include <debug_uart.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/mach-types.h>
 #include <asm/arch/at91sam9rl.h>
index cc051d2e0cc2bc0e7ab9f975814ae3a0f42cd1d8..5cc52139a7796beff032d05875b16b4a9bbb4c8a 100644 (file)
@@ -11,6 +11,7 @@
 #include <version.h>
 #include <video.h>
 #include <video_console.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/clk.h>
 
index b0d2f7b6f87b1b76b97a416b769006ce951d26d5..221048a91f1865b6c111e4a88cf2b06b7ce06e08 100644 (file)
@@ -11,6 +11,7 @@
 
 #include <common.h>
 #include <env.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/gpio.h>
 #include <asm/mach-types.h>
index f9f8d8021355b2fc7666c9137f270284081fd3b5..4b6d91df219acc2c472243816894adf25a26eeca 100644 (file)
@@ -4,6 +4,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
 #include <asm/processor.h>
index 191ef01d23e48923030bd6f543d68a37595e50e9..f4effe5a2d637fca90d97b065d9044026e6f4acf 100644 (file)
@@ -4,6 +4,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
 #include <asm/processor.h>
index 38f13ce4b2ec39ee30e71015bc044662f6b7e8aa..9c8731ce78064b401275f9e3a75c9a624b201b58 100644 (file)
@@ -6,6 +6,7 @@
 #include <common.h>
 #include <i2c.h>
 #include <hwconfig.h>
+#include <vsprintf.h>
 #include <asm/mmu.h>
 #include <fsl_ddr_sdram.h>
 #include <fsl_ddr_dimm_params.h>
index 784e482f32fb3b8213b034df73eea518ffc8e617..2677b7928f5584d39c18cc24feadba36b97373a5 100644 (file)
@@ -7,6 +7,7 @@
 #include <fsl_ddr_sdram.h>
 #include <fsl_ddr_dimm_params.h>
 #include "ddr.h"
+#include <vsprintf.h>
 #ifdef CONFIG_FSL_DEEP_SLEEP
 #include <fsl_sleep.h>
 #endif
index 779c99c2686a6222856ea0f27971279ea441b745..be79a7ca7b81b53b4a3900cb48e6e4c383260d85 100644 (file)
@@ -7,6 +7,7 @@
 
 #include <common.h>
 #include <pci.h>
+#include <vsprintf.h>
 #include <asm/processor.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
index 2799b5b5a4b0cf46372bbee01d972db0faf54198..28f8a998ce7541538c603a32ebf09ce33ec94268 100644 (file)
@@ -7,6 +7,7 @@
 
 #include <common.h>
 #include <pci.h>
+#include <vsprintf.h>
 #include <asm/processor.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
index 6cfb8d5191a7ceeed4572e97465b942e09416191..bc1ebc51edbce4c02881e23cedcca93fabdf7689 100644 (file)
@@ -5,6 +5,7 @@
 
 #include <common.h>
 #include <pci.h>
+#include <vsprintf.h>
 #include <asm/processor.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
index 01e3eafd12a7158bab1e79fc009cbc8c72300a6a..71f6259b60e459a99e2937baecb0f7638a68cfd9 100644 (file)
@@ -4,6 +4,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
 #include <asm/processor.h>
index b9bd7b52765186842547648e900d699713b25ce8..2346f6a0c274194e37d2e7bfda7118d4fee8fe49 100644 (file)
@@ -4,6 +4,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
 #include <asm/processor.h>
index 7e8bd6b8c2110c2b2e5f02d79d402e1eba730dc3..85f1f6344adf5782b60f50fac543d5249f322562 100644 (file)
@@ -4,6 +4,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/mmu.h>
 #include <asm/immap_85xx.h>
 #include <asm/processor.h>
index 9554fefd4ecafe1d0eee8a6ecffe4699847a5954..9a724040b07209271bef4c531bc8246eff92528e 100644 (file)
@@ -11,6 +11,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <asm/io.h>
 #include <asm/arch/clk.h>
 #include <asm/arch/at91sam9g45_matrix.h>
index f1e7aab6296ae5abca33b697f722b7ca94f54dfc..bad673412a1f4d36e03192eb8015aa75fa4ef356 100644 (file)
@@ -8,6 +8,7 @@
  */
 
 #include <common.h>
+#include <vsprintf.h>
 #include <linux/sizes.h>
 #include <asm/io.h>
 #include <asm/gpio.h>
index ae6006f85ffa47bae8f3521619166e9e296a4425..abd9151432ed945e161a50ac54107cad1b0232f9 100644 (file)
@@ -10,6 +10,7 @@
 #include <common.h>
 #include <command.h>
 #include <env.h>
+#include <vsprintf.h>
 #include <linux/compiler.h>
 
 DECLARE_GLOBAL_DATA_PTR;
index e87307fa60ca59dfb5fa94b892c03b30e6856746..e223f84eac028df663537532aa9734ba7a361658 100644 (file)
@@ -10,6 +10,7 @@
 #include <regmap.h>
 #include <spl.h>
 #include <syscon.h>
+#include <vsprintf.h>
 #include <linux/io.h>
 #include <linux/iopoll.h>
 #include <dt-bindings/clock/stm32mp1-clks.h>
index 32d2db9edadbb290a7c3d037756ebd16f598a0db..cbccda55943a6ce61bf4d8ea47d0d5cafdd377c1 100644 (file)
@@ -7,6 +7,7 @@
 #include <common.h>
 #include <clk-uclass.h>
 #include <dm.h>
+#include <vsprintf.h>
 #include <dm/lists.h>
 #include <dt-bindings/clk/mpc83xx-clk.h>
 #include <asm/arch/soc.h>
index 7bc86bf9b208a07fd3301848ba84c253d0fed576..1e584577421181c90361242500952b4f2ec3d310 100644 (file)
@@ -9,6 +9,7 @@
 #include <clk.h>
 #include <cpu.h>
 #include <dm.h>
+#include <vsprintf.h>
 
 #include "mpc83xx_cpu.h"
 
index d17a2b2642c941251cdb89a2405778b35eb01fe8..f993e4bef333ce45f7137bd13e319f7677d67781 100644 (file)
@@ -310,9 +310,6 @@ int ulz4fn(const void *src, size_t srcn, void *dst, size_t *dstn);
 /* lib/vsprintf.c */
 #include <vsprintf.h>
 
-/* lib/strmhz.c */
-char * strmhz(char *buf, unsigned long hz);
-
 /*
  * STDIO based functions (can always be used)
  */
index d1740a378e0a7b5e8bb0e8bacfa221ab10d9b03e..56844dd2de85b8054db96fc6a67e489382b27ea1 100644 (file)
@@ -212,4 +212,14 @@ void print_grouped_ull(unsigned long long int_val, int digits);
 
 bool str2off(const char *p, loff_t *num);
 bool str2long(const char *p, ulong *num);
+
+/**
+ * strmhz() - Convert a value to a Hz string
+ *
+ * This creates a string indicating the number of MHz of a value. For example,
+ * 2700000 produces "2.7".
+ * @buf: Buffer to hold output string, which must be large enough
+ * @hz: Value to convert
+ */
+char *strmhz(char *buf, unsigned long hz);
 #endif
index d248d8626cea1ee2951c9380bd749b0754fe8418..e15a189f65ba1f6d45e40c5582a23d011290fb02 100644 (file)
@@ -109,12 +109,12 @@ ifeq ($(CONFIG_$(SPL_TPL_)BUILD),y)
 ifdef CONFIG_$(SPL_TPL_)USE_TINY_PRINTF
 obj-$(CONFIG_$(SPL_TPL_)SPRINTF) += tiny-printf.o
 else
-obj-$(CONFIG_$(SPL_TPL_)SPRINTF) += vsprintf.o strmhz.o
+obj-$(CONFIG_$(SPL_TPL_)SPRINTF) += vsprintf.o
 endif
 obj-$(CONFIG_$(SPL_TPL_)STRTO) += strto.o
 else
 # Main U-Boot always uses the full printf support
-obj-y += vsprintf.o strto.o strmhz.o
+obj-y += vsprintf.o strto.o
 endif
 
 subdir-ccflags-$(CONFIG_CC_OPTIMIZE_LIBS_FOR_SPEED) += -O2
diff --git a/lib/strmhz.c b/lib/strmhz.c
deleted file mode 100644 (file)
index 66afe91..0000000
+++ /dev/null
@@ -1,21 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0+
-/*
- * (C) Copyright 2002-2006
- * Wolfgang Denk, DENX Software Engineering, wd@denx.de.
- */
-#include <common.h>
-
-char *strmhz (char *buf, unsigned long hz)
-{
-       long l, n;
-       long m;
-
-       n = DIV_ROUND_CLOSEST(hz, 1000) / 1000L;
-       l = sprintf (buf, "%ld", n);
-
-       hz -= n * 1000000L;
-       m = DIV_ROUND_CLOSEST(hz, 1000L);
-       if (m != 0)
-               sprintf (buf + l, ".%03ld", m);
-       return (buf);
-}
index 425f2f53f79770d53047a27893aad6b0b776216b..c6467ecd5ff9d08a8fa03e5df3064b6b273162bb 100644 (file)
@@ -2,6 +2,8 @@
  *  linux/lib/vsprintf.c
  *
  *  Copyright (C) 1991, 1992  Linus Torvalds
+ * (C) Copyright 2000-2009
+ * Wolfgang Denk, DENX Software Engineering, wd@denx.de.
  */
 
 /* vsprintf.c -- Lars Wirzenius & Linus Torvalds. */
@@ -17,6 +19,7 @@
 #include <div64.h>
 #include <hexdump.h>
 #include <stdarg.h>
+#include <vsprintf.h>
 #include <linux/ctype.h>
 #include <linux/err.h>
 #include <linux/types.h>
@@ -873,3 +876,19 @@ bool str2long(const char *p, ulong *num)
        *num = simple_strtoul(p, &endptr, 16);
        return *p != '\0' && *endptr == '\0';
 }
+
+char *strmhz(char *buf, unsigned long hz)
+{
+       long l, n;
+       long m;
+
+       n = DIV_ROUND_CLOSEST(hz, 1000) / 1000L;
+       l = sprintf(buf, "%ld", n);
+
+       hz -= n * 1000000L;
+       m = DIV_ROUND_CLOSEST(hz, 1000L);
+       if (m != 0)
+               sprintf(buf + l, ".%03ld", m);
+
+       return buf;
+}