aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2019-12-28 10:44:40 -0700
committerTom Rini <trini@konsulko.com>2020-01-17 13:26:48 -0500
commit0ee48252b456f310225fa8dd0f3a9921cfcf3399 (patch)
treef580fa4d2d2076d00d034d79f011885e5ca0169f
parent6b8d3ceaf59a6c4a2d3930aa8fa0c33e15093498 (diff)
downloadu-boot-0ee48252b456f310225fa8dd0f3a9921cfcf3399.zip
u-boot-0ee48252b456f310225fa8dd0f3a9921cfcf3399.tar.gz
u-boot-0ee48252b456f310225fa8dd0f3a9921cfcf3399.tar.bz2
common: Move flash_perror() to flash.h
This function belongs more in flash.h than common.h so move it. Also remove the space before the bracket in some calls. Signed-off-by: Simon Glass <sjg@chromium.org>
-rw-r--r--cmd/load.c3
-rw-r--r--cmd/mem.c3
-rw-r--r--common/flash.c2
-rw-r--r--env/flash.c1
-rw-r--r--include/common.h3
-rw-r--r--include/flash.h7
-rw-r--r--net/nfs.c1
7 files changed, 14 insertions, 6 deletions
diff --git a/cmd/load.c b/cmd/load.c
index 3bfc1b4..68ff301 100644
--- a/cmd/load.c
+++ b/cmd/load.c
@@ -12,6 +12,7 @@
#include <console.h>
#include <cpu_func.h>
#include <env.h>
+#include <flash.h>
#include <s_record.h>
#include <net.h>
#include <exports.h>
@@ -980,7 +981,7 @@ static ulong load_serial_ymodem(ulong offset, int mode)
rc = flash_write((char *) ymodemBuf,
store_addr, res);
if (rc != 0) {
- flash_perror (rc);
+ flash_perror(rc);
return (~0);
}
} else
diff --git a/cmd/mem.c b/cmd/mem.c
index 4ec450b..4de5ab0 100644
--- a/cmd/mem.c
+++ b/cmd/mem.c
@@ -16,6 +16,7 @@
#include <cli.h>
#include <command.h>
#include <console.h>
+#include <flash.h>
#include <hash.h>
#include <mapmem.h>
#include <watchdog.h>
@@ -339,7 +340,7 @@ static int do_mem_cp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
rc = flash_write((char *)src, (ulong)dst, count * size);
if (rc != 0) {
- flash_perror (rc);
+ flash_perror(rc);
unmap_sysmem(src);
unmap_sysmem(dst);
return (1);
diff --git a/common/flash.c b/common/flash.c
index efe8f53..5f155ae 100644
--- a/common/flash.c
+++ b/common/flash.c
@@ -179,7 +179,7 @@ flash_write (char *src, ulong addr, ulong cnt)
/*-----------------------------------------------------------------------
*/
-void flash_perror (int err)
+void flash_perror(int err)
{
switch (err) {
case ERR_OK:
diff --git a/env/flash.c b/env/flash.c
index b487e67..e05f7ef 100644
--- a/env/flash.c
+++ b/env/flash.c
@@ -13,6 +13,7 @@
#include <command.h>
#include <env.h>
#include <env_internal.h>
+#include <flash.h>
#include <linux/stddef.h>
#include <malloc.h>
#include <search.h>
diff --git a/include/common.h b/include/common.h
index c35af99..c3dcf6d 100644
--- a/include/common.h
+++ b/include/common.h
@@ -78,9 +78,6 @@ extern u8 __dtb_dt_spl_begin[]; /* embedded device tree blob for SPL/TPL */
*/
int arch_fixup_fdt(void *blob);
-/* common/flash.c */
-void flash_perror (int);
-
/* common/cmd_source.c */
int source (ulong addr, const char *fit_uname);
diff --git a/include/flash.h b/include/flash.h
index 807800b..2655c72 100644
--- a/include/flash.h
+++ b/include/flash.h
@@ -117,6 +117,13 @@ extern int jedec_flash_match(flash_info_t *info, ulong base);
#define CFI_CMDSET_AMD_LEGACY 0xFFF0
#endif
+/**
+ * flash_perror() - Print a flash error
+ *
+ * @err: Error number of message to print (ERR_... as below)
+ */
+void flash_perror(int err);
+
/*-----------------------------------------------------------------------
* return codes from flash_write():
*/
diff --git a/net/nfs.c b/net/nfs.c
index 97e62f1..f4101eb 100644
--- a/net/nfs.c
+++ b/net/nfs.c
@@ -28,6 +28,7 @@
#include <common.h>
#include <command.h>
+#include <flash.h>
#include <net.h>
#include <malloc.h>
#include <mapmem.h>