[PATCH v2 00/19] Kconfig: Tidy up some options

Tom Rini trini at konsulko.com
Thu Aug 31 16:12:25 CEST 2023


On Wed, Aug 30, 2023 at 09:52:54PM -0600, Simon Glass wrote:

> The view from 'make menuconfig' is confusing in places. This series aims
> to improve the top-level menu and also the boot menu.
> 
> It also groups FDT-fixup options tegether, at least the ones I could fine.
> 
> Finally this series marks the distro scripts as deprecated, so people have
> a pointer to standard boot.
> 
> Changes in v2:
> - Fix FMU typo in the subject
> - Drop now-unnecessary depends on FWU_MULTI_BANK_UPDATE
> - Mention in the DISTRO_DEFAULTS option that it is script-based
> - Expand and rewrite the commit message
> - Use the word 'Mark' instead of 'Make' to improve the English
> 
> Simon Glass (19):
>   lib: rational: Move the Kconfigs into the correct place
>   Kconfig: Move API into general setup
>   video: Hide the BMP options
>   video: Move BMP options and code to video directory
>   net: Move SYS_RX_ETH_BUFFER into the network menu
>   FWU: Avoid showing an unselectable menu option
>   test: Move POST under a renamed Testing section
>   boot: Move fdt_support to boot/
>   Move fdt_simplefb to boot/
>   boot: Move some other fdt-fixup options to the same menu
>   boot: Rename Android-boot text
>   Kconfig: Create a menu for FIT
>   Kconfig: Move SPL_FIT under FIT
>   spl: Tidy up load address in spl_ram
>   boot: Make standard boot a menu
>   Kconfig: Move TEXT_BASE et al under general setup
>   Mark DISTRO_DEFAULTS as deprecated
>   boot: Join FDT_FIXUP_PARTITIONS with related options
>   boot: Join ARCH_FIXUP_FDT_MEMORY with related options
> 
>  Kconfig                         |  67 ++++++++++-
>  boot/Kconfig                    | 202 +++++++++++++-------------------
>  boot/Makefile                   |   4 +
>  {common => boot}/fdt_simplefb.c |   0
>  {common => boot}/fdt_support.c  |   0
>  common/Kconfig                  |  20 ----
>  common/Makefile                 |   4 -
>  common/spl/spl_ram.c            |  19 ++-
>  doc/develop/bootstd.rst         |  23 ++++
>  drivers/video/Kconfig           |  11 ++
>  drivers/video/Makefile          |   2 +
>  {common => drivers/video}/bmp.c |   0
>  include/net.h                   |   9 +-
>  lib/Kconfig                     |  17 +--
>  lib/fwu_updates/Kconfig         |   9 +-
>  net/Kconfig                     |   4 +-
>  test/Kconfig                    |  12 +-
>  17 files changed, 220 insertions(+), 183 deletions(-)
>  rename {common => boot}/fdt_simplefb.c (100%)
>  rename {common => boot}/fdt_support.c (100%)
>  rename {common => drivers/video}/bmp.c (100%)

Please run a world build before/after and check for both size changes
and Kconfig unmet dependency issues (which CI does not catch).  Based on
at least the BMP patch I suspect there are problems and unintended
behavior changes.

-- 
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/20230831/557b8d66/attachment.sig>


More information about the U-Boot mailing list