[PATCH v2 00/39] Tidy up the 'bd' command.

[Tom Rini]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200625/1c4187b5/attachment.sig>