From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Schwierzeck Date: Wed, 6 May 2020 17:56:10 +0200 Subject: [PATCH 07/36] bdinfo: microblaze: Use the generic bd command In-Reply-To: <20200504171638.7.Ic4c05106579f39e7418db0a149582bc992327326@changeid> References: <20200504231732.98778-1-sjg@chromium.org> <20200504171638.7.Ic4c05106579f39e7418db0a149582bc992327326@changeid> Message-ID: <1e2a4214-0a62-0682-9473-e3dd52220528@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am 05.05.20 um 01:17 schrieb Simon Glass: > Microblaze prints out ethernet and FDT information. This is useful to > most archs, so move it into the generic code and move microblaze over to > use it. > > Signed-off-by: Simon Glass > --- > > cmd/bdinfo.c | 30 +++++++----------------------- > 1 file changed, 7 insertions(+), 23 deletions(-) > > diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c > index 405a915861..dc5a09f8ce 100644 > --- a/cmd/bdinfo.c > +++ b/cmd/bdinfo.c > @@ -224,29 +224,7 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) > > #elif defined(CONFIG_MICROBLAZE) > > -int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) > -{ > - bd_t *bd = gd->bd; > - > - print_bi_dram(bd); > - print_bi_flash(bd); > -#if defined(CONFIG_SYS_SRAM_BASE) > - print_num("sram start ", (ulong)bd->bi_sramstart); > - print_num("sram size ", (ulong)bd->bi_sramsize); > -#endif > -#if defined(CONFIG_CMD_NET) && !defined(CONFIG_DM_ETH) > - print_eths(); > -#endif > - print_baudrate(); > - print_num("relocaddr", gd->relocaddr); > - print_num("reloc off", gd->reloc_off); > - print_num("fdt_blob", (ulong)gd->fdt_blob); > - print_num("new_fdt", (ulong)gd->new_fdt); > - print_num("fdt_size", (ulong)gd->fdt_size); > - print_cpu_word_size(); > - > - return 0; > -} > +#define USE_GENERIC > > #elif defined(CONFIG_M68K) > > @@ -463,6 +441,12 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[]) > print_num("relocaddr", gd->relocaddr); > print_num("reloc off", gd->reloc_off); > print_cpu_word_size(); > +#if defined(CONFIG_CMD_NET) && !defined(CONFIG_DM_ETH) > + print_eths(); > +#endif > + print_num("fdt_blob", (ulong)gd->fdt_blob); > + print_num("new_fdt", (ulong)gd->new_fdt); > + print_num("fdt_size", (ulong)gd->fdt_size); shouldn't that be guarded with CONFIG_IS_ENABLED(OF_LIBFDT) ? > > return 0; > } > -- - Daniel