diff options
| author | Simon Glass <sjg@chromium.org> | 2020-05-10 14:16:47 -0600 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2020-06-25 13:24:12 -0400 |
| commit | 537cb0dfd2fbf40491595f55b2e03ce898b3896c (patch) | |
| tree | 8f42438b0ce97f0a19e355df1c0797de780c46c6 /cmd/bdinfo.c | |
| parent | 3e1cca2a2f6245a98b86e23222908dee234542e2 (diff) | |
bdinfo: sh: arc: Drop arch-specific print_bi_mem()
It isn't worth having arch-specific code for such minor output
differences. In fact it is better if all archs are consistent.
Drop the arch-specific code in print_bi_mem() and inline it to avoid a
two-line function.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Alexey Brodkin <abrodkin@synopsys.com>
Diffstat (limited to 'cmd/bdinfo.c')
| -rw-r--r-- | cmd/bdinfo.c | 17 |
1 files changed, 2 insertions, 15 deletions
diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c index 60bfb4e1da..9746433c23 100644 --- a/cmd/bdinfo.c +++ b/cmd/bdinfo.c @@ -80,20 +80,6 @@ static void print_bi_boot_params(const bd_t *bd) print_num("boot_params", (ulong)bd->bi_boot_params); } -static void print_bi_mem(const bd_t *bd) -{ -#if defined(CONFIG_SH) - print_num("mem start ", (ulong)bd->bi_memstart); - print_lnum("mem size ", (u64)bd->bi_memsize); -#elif defined(CONFIG_ARC) - print_num("mem start", (ulong)bd->bi_memstart); - print_lnum("mem size", (u64)bd->bi_memsize); -#else - print_num("memstart", (ulong)bd->bi_memstart); - print_lnum("memsize", (u64)bd->bi_memsize); -#endif -} - static void print_bi_dram(const bd_t *bd) { #ifdef CONFIG_NR_DRAM_BANKS @@ -155,7 +141,8 @@ int do_bdinfo(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) print_num("arch_number", bd->bi_arch_number); print_bi_boot_params(bd); print_bi_dram(bd); - print_bi_mem(bd); + print_num("memstart", (ulong)bd->bi_memstart); + print_lnum("memsize", (u64)bd->bi_memsize); print_bi_flash(bd); print_eth_ip_addr(); printf("baudrate = %u bps\n", gd->baudrate); |