[PATCH v2 000/169] Kconfig: More cleanup of CONFIG options

Tom Rini trini at konsulko.com
Fri Feb 10 16:17:51 CET 2023


On Sun, Feb 05, 2023 at 03:35:46PM -0700, Simon Glass wrote:

> Quite a few CONFIG options without an SPL equivalent are used as if they
> have one. For example, we see CONFIG_IS_ENABLED(IOMMU) when there is no
> SPL_IOMMU Kconfig option.
> 
> This series adds new features to moveconfig to detect and remove these
> problems. It includes additions to the Kconfig for options which are
> needed as well as dropping unwanted us of CONFIG_IS_ENABLED, etc.
> 
> This follows on from v3 the previous series[1]
> 
> It is available at u-boot-dm/splb-working
> 
> [1] https://patchwork.ozlabs.org/project/uboot/list/?series=339724
> 
> Changes in v2:
> - Rebase to previous series
> - Correct the tag
> - Rebase to previous series
> - Drop patch to add a Kconfig for SPL_EFI_LOADER et al
> - Drop patch to add a Kconfig for SPL_GENERATE_ACPI_TABLE
> 
> Simon Glass (169):
>   moveconfig: Update to detect / correct missing SPL Kconfigs
>   x86: Correct Chrromebook typo
>   boot: Add a Kconfig for SPL_FIT_CIPHER
>   boot: Add a Kconfig for SPL_BOOTSTD_FULL
>   boot: Add Kconfigs for BOOTMETH_VBE_REQUEST
>   boot: Add a Kconfig for SPL_BOOTDEV_SPI_FLASH
>   boot: Add a Kconfig for SPL_BOOTDEV_ETH
>   dm: Add a Kconfig for SPL_DEVRES
>   dm: Add a Kconfig for SPL_DM_HWSPINLOCK
>   boot: Add a Kconfig for SPL_OF_LIVE
>   cmd: Add a Kconfig for SPL_CRC32_VERIFY
>   console: Add a Kconfig for SPL_CONSOLE_MUX et al
>   button: Add a Kconfig for SPL_BUTTON
>   lib: Add a Kconfig for SPL_ERRNO_STR
>   iommu: Add a Kconfig for SPL_IOMMU
>   pinctrl: Add a Kconfig for SPL_PINCTRL_ARMADA_38X
>   virtio: Add a Kconfig for SPL_VIRTIO
>   lib: Add a Kconfig for SPL_BZIP2
>   acpi: Add a Kconfig for SPL_ACPIGEN
>   rockchip: Add a Kconfig for SPL_RESET_ROCKCHIP
>   mediatek: Add a Kconfig for SPL_RESET_MEDIATEK
>   Correct SPL use of A003399_NOR_WORKAROUND
>   Correct SPL use of AHCI
>   Correct SPL uses of ALLEYCAT_5
>   Correct SPL uses of ARCH_MVEBU
>   Correct SPL use of ARCH_VERSAL
>   Correct SPL use of ARCH_ZYNQ
>   Correct SPL use of ARM64
>   Correct SPL uses of ARMADA_3700
>   Correct SPL uses of ARMADA_8K
>   Correct SPL use of ATMEL_PIO4
>   Correct SPL use of BNXT_ETH
>   Correct SPL use of BOOTSTAGE_FDT
>   Correct SPL uses of BOOTSTAGE_REPORT
>   Correct SPL use of CMD_BCB
>   Correct SPL uses of CMD_BOOTEFI_BOOTMGR
>   Correct SPL use of CMD_BOOTI
>   Correct SPL uses of CMD_BOOTM_PRE_LOAD
>   Correct SPL use of CMD_BOOTZ
>   Correct SPL uses of CMD_CLK
>   Correct SPL uses of CMD_DHCP
>   Correct SPL uses of CMD_EFICONFIG
>   Correct SPL use of CMD_ERASEENV
>   Correct SPL uses of CMD_FDT
>   Correct SPL use of CMD_FRU
>   Correct SPL use of CMD_HASH
>   Correct SPL use of CMD_MBR
>   Correct SPL uses of CMD_MMC
>   Correct SPL use of CMD_NET
>   Correct SPL use of CMD_NVEDIT_EFI
>   Correct SPL uses of CMD_NVME
>   Correct SPL use of CMD_PSTORE
>   Correct SPL uses of CMD_PXE
>   Correct SPL uses of CMD_SCSI
>   Correct SPL uses of CMD_SF
>   Correct SPL use of CMD_SHA1SUM
>   Correct SPL use of CMD_STBOARD
>   Correct SPL use of CMD_STM32PROG
>   Correct SPL uses of CMD_TFTPPUT
>   Correct SPL uses of CMD_USB
>   Correct SPL use of CMD_VIRTIO
>   Correct SPL uses of CPU_MICROBLAZE
>   Correct SPL use of DEBUG_UART_ZYNQ
>   Correct SPL uses of DFU_VIRT
>   Correct SPL uses of DISPLAY_AER_BRIEF
>   Correct SPL uses of DISPLAY_AER_FULL
>   Correct SPL use of DISPLAY_CPUINFO
>   Correct SPL use of DM_RNG
>   Correct SPL uses of DTB_RESELECT
>   Correct SPL uses of DWC_ETH_QOS
>   Correct SPL use of EFI_APP
>   Correct SPL use of EFI_DEVICE_PATH_UTIL
>   Correct SPL uses of EFI_HAVE_CAPSULE_SUPPORT
>   Correct SPL use of EFI_LOADER_HII
>   Correct SPL use of EFI_MM_COMM_TEE
>   Correct SPL use of EFI_SECURE_BOOT
>   Correct SPL use of EFI_TCG2_PROTOCOL
>   Correct SPL use of EFI_UNICODE_COLLATION_PROTOCOL2
>   Correct SPL uses of ENV_IS_IN_UBI
>   Correct SPL uses of ENV_VARS_UBOOT_RUNTIME_CONFIG
>   Correct SPL use of ENV_WRITEABLE_LIST
>   Correct SPL use of EXYNOS7420
>   Correct SPL uses of FASTBOOT
>   Correct SPL uses of FASTBOOT_FLASH
>   Correct SPL uses of FASTBOOT_FLASH_MMC
>   Correct SPL uses of FASTBOOT_FLASH_NAND
>   Correct SPL use of FASTBOOT_MMC_USER_SUPPORT
>   Correct SPL use of FDT_SIMPLEFB
>   Correct SPL uses of FEC_MXC
>   Correct SPL use of FSL_CAAM
>   Correct SPL uses of FSL_ISBC_KEY_EXT
>   Correct SPL use of FSL_MC_ENET
>   Correct SPL use of FSP_VERSION2
>   Correct SPL use of FS_EROFS
>   Correct SPL use of GENERATE_SMBIOS_TABLE
>   Correct SPL uses of GMAC_ROCKCHIP
>   Correct SPL use of HUSH_PARSER
>   Correct SPL uses of HW_WATCHDOG
>   Correct SPL uses of I8259_PIC
>   Correct SPL uses of IMX8M
>   Correct SPL use of IMX8MN_BEACON_2GB_LPDDR
>   Correct SPL uses of IMX_MODULE_FUSE
>   Correct SPL uses of IMX_RDC
>   Correct SPL uses of LMB
>   Correct SPL uses of MICROBLAZE
>   Correct SPL use of MIPS_CM
>   Correct SPL use of MMC_QUIRKS
>   Correct SPL uses of MTD
>   Correct SPL uses of MULTIPLEXER
>   Correct SPL uses of NEEDS_MANUAL_RELOC
>   Correct SPL use of NETDEVICES
>   Correct SPL uses of NXP_FSPI
>   Correct SPL use of OCTEON_SERIAL_BOOTCMD
>   Correct SPL use of OCTEON_SERIAL_PCIE_CONSOLE
>   Correct SPL uses of OF_BOARD
>   Correct SPL use of OF_EMBED
>   Correct SPL use of PANEL
>   Correct SPL use of PARTITION_TYPE_GUID
>   Correct SPL use of PCI_PNP
>   Correct SPL uses of PG_WCOM_UBOOT_BOOTPACKAGE
>   Correct SPL uses of PG_WCOM_UBOOT_UPDATE
>   Correct SPL use of PG_WCOM_UBOOT_UPDATE_SUPPORTED
>   Correct SPL use of PHY_CADENCE_SIERRA
>   Correct SPL use of PHY_CADENCE_TORRENT
>   Correct SPL uses of PHY_FIXED
>   Correct SPL uses of PMIC_STPMIC1
>   Correct SPL use of QFW_PIO
>   Correct SPL use of REGEX
>   Correct SPL use of RENESAS_SDHI
>   Correct SPL use of RESV_RAM
>   Correct SPL uses of ROCKCHIP_EFUSE
>   Correct SPL uses of ROCKCHIP_OTP
>   Correct SPL use of SANDBOX
>   Correct SPL uses of SANDBOX_CLK_CCF
>   Correct SPL uses of SAVE_PREV_BL_FDT_ADDR
>   Correct SPL uses of SAVE_PREV_BL_INITRAMFS_START_ADDR
>   Correct SPL uses of SEC_FIRMWARE_ARMV8_PSCI
>   Correct SPL uses of SIFIVE_OTP
>   Correct SPL use of SL28CPLD
>   Correct SPL use of SL28_SPL_LOADS_OPTEE_BL32
>   Correct SPL use of SPI_FLASH_BAR
>   Correct SPL uses of SPI_FLASH_MACRONIX
>   Correct SPL use of SPLASH_SOURCE
>   Correct SPL use of STM32MP15X_STM32IMAGE
>   Correct SPL use of SUPPORT_EMMC_RPMB
>   Correct SPL use of SYS_FSL_ERRATUM_A010539
>   Correct SPL uses of SYS_LONGHELP
>   Correct SPL use of SYS_MEM_RSVD_FOR_MMU
>   Correct SPL uses of TARGET_LX2160ARDB
>   Correct SPL use of TARGET_MX6UL_9X9_EVK
>   Correct SPL use of TARGET_PG_WCOM_EXPU1
>   Correct SPL use of TARGET_PG_WCOM_SELI8
>   Correct SPL use of TEE
>   Correct SPL uses of TEN64_CONTROLLER
>   Correct SPL use of TPM_RNG
>   Correct SPL use of UDP_FUNCTION_FASTBOOT
>   Correct SPL use of USB_DWC3_MESON_G12A
>   Correct SPL uses of USB_DWC3_MESON_GXL
>   Correct SPL use of USB_FUNCTION_FASTBOOT
>   Correct SPL uses of USB_GADGET_DWC2_OTG
>   Correct SPL uses of USB_KEYBOARD
>   Correct SPL uses of USB_MUSB_HOST
>   Correct SPL uses of VIDEO
>   Correct SPL use of VIDEO_ANSI
>   Correct SPL use of VIDEO_BPP16
>   Correct SPL use of VIDEO_BPP32
>   Correct SPL use of WATCHDOG_AUTOSTART
>   Correct SPL uses of XEN_SERIAL
>   Correct SPL uses of ZYNQMP_FIRMWARE

I have largely applied this series. In terms of next steps:
- As we've talked about, the def_bool n parts move to the split config
  series, to evaluate as part of the whole there.
- The CMD_* ones can liley be handled by moving the platforms to bootstd
  instead, so they don't need to do tricks to reduce the default / only
  env, in SPL.
- The rest of the size increase patches either need to be done
  individually and get ack'd by the relevant custodians, or for the
  MULTIPLEXER one, just re-worked to move the Makefile line to the block
  of never-xPL lines.

I'm not going to reply to each with an applied message, but I'll run the
script shortly to update patchwork status.

-- 
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/20230210/8beb9a1d/attachment.sig>


More information about the U-Boot mailing list