On Sun, May 10, 2020 at 02:16:23PM -0600, Simon Glass wrote: > The code for the 'bd' command never got the 'generic board' treatment many > years ago when global_data and bd_info were converted. As a result it > still has a lot of arch-specific duplication of generic code. > > This series aims to make as much code in this file generic as possible, so > that it is easy to add new info on all architectures. > > For the three architectures that actually need additional code (ARM, PPC > and m68k) this is moved into arch-specific files. > > With this series, bdinfo.c drops from nearly 500 lines to just over 100. > > It also makes x86 report the frame buffer address properly (the original > goal of my effort). > > Changes in v2: > - Add a new patch to drop the 'bd' command for tbs2910 > - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches > - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches > - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches > - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches > - Drop __maybe_unused on print_lnum() too > - Move __maybe_unused on print_lnum() to the early patch > - Update commit message with an extra motivation > - Squash in the next patch to inline print_eths() > - Leave MULTI_DTB_FIT in the generic file as it is not really ARM-specific > - Move ARM's bi_arch_number over to ARM-specific code as well > - Update the commit message to explain the address more > > Simon Glass (39): > tbs2910: Drop the 'bd' command > bdinfo: nds32: Use generic bd_info > bdinfo: riscv: Use generic bd_info > bdinfo: m68k: Drop bd_info->bi_ipbfreq > bdinfo: xtensa: Create a generic do_bdinfo for xtensa > bdinfo: mips: Use the generic bd command > bdinfo: nios2: Use the generic bd command > bdinfo: microblaze: Use the generic bd command > bdinfo: sh: Use the generic bd command > bdinfo: x86: Use the generic bd command > bdinfo: sandbox: Use the generic bd command > bdinfo: nds32: Use the generic bd command > bdinfo: riscv: Use the generic bd command > bdinfo: powerpc: Use the generic bd command > bdinfo: m68k: Use the generic bd command > bdinfo: arm: Use the generic bd command > bdinfo: arc: Use the generic bd command > bdinfo: Drop the option to not use the generic 'bd' command > bdinfo: Drop unused __maybe_unused > bdinfo: microblaze: sh: nios2: Drop arch-specific flash info > bdinfo: Drop unnecessary inline on functions > bdinfo: Drop print_std_bdinfo() > bdinfo: ppc: Drop arch-specific print_baudrate() > bdinfo: sh: arc: Drop arch-specific print_bi_mem() > bdinfo: Drop print_bi_boot_params() > bdinfo: Drop print_bi_flash() > bdinfo: Drop print_cpu_word_size() > bdinfo: net: ppc: Drop bi_enet1addr and other similar info > bdinfo: net: ppc: Drop prints for CONFIG_HAS_ETHn > bdinfo: net: Drop legacy ethernet bdinfo > bdinfo: net: Inline print_eth_ip_addr() > bdinfo: Export some basic printing functions > bdinfo: arm: Move ARM-specific info into its own file > bdinfo: ppc: Move PPC-specific info into its own file > bdinfo: m68k: Move m68k-specific info into its own file > bdinfo: m68k: ppc: Move arch-specific code from bdinfo > bdinfo: Update the file comments > bdinfo: dm: Update fb_base when using driver model > bdinfo: x86: vesa: Update fb_base to the correct value
For the series, trivially updated for struct cmd_tbl and a few whitespace corrections since this was posted, applied to u-boot/next, thanks! -- Tom
signature.asc
Description: PGP signature
