[RFC PATCH v2 14/48] lmb: remove config symbols used for lmb region count

Ilias Apalodimas ilias.apalodimas at linaro.org
Tue Jul 23 10:48:43 CEST 2024


On Thu, 4 Jul 2024 at 10:36, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> The LMB memory maps are now being maintained through a couple of
> alloced lists, one for the available(added) memory, and one for the
> used memory. These lists are not static arrays but can be extended at
> runtime. Remove the config symbols which were being used to define the
> size of these lists with the earlier implementation of static arrays.
>
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> ---
> Changes since V1: New patch
>
>  configs/a3y17lte_defconfig           |  1 -
>  configs/a5y17lte_defconfig           |  1 -
>  configs/a7y17lte_defconfig           |  1 -
>  configs/apple_m1_defconfig           |  1 -
>  configs/mt7981_emmc_rfb_defconfig    |  1 -
>  configs/mt7981_rfb_defconfig         |  1 -
>  configs/mt7981_sd_rfb_defconfig      |  1 -
>  configs/mt7986_rfb_defconfig         |  1 -
>  configs/mt7986a_bpir3_emmc_defconfig |  1 -
>  configs/mt7986a_bpir3_sd_defconfig   |  1 -
>  configs/mt7988_rfb_defconfig         |  1 -
>  configs/mt7988_sd_rfb_defconfig      |  1 -
>  configs/qcom_defconfig               |  1 -
>  configs/stm32mp13_defconfig          |  3 ---
>  configs/stm32mp15_basic_defconfig    |  3 ---
>  configs/stm32mp15_defconfig          |  3 ---
>  configs/stm32mp15_trusted_defconfig  |  3 ---
>  configs/stm32mp25_defconfig          |  3 ---
>  configs/th1520_lpi4a_defconfig       |  1 -
>  lib/Kconfig                          | 34 ----------------------------
>  20 files changed, 63 deletions(-)
>
> diff --git a/configs/a3y17lte_defconfig b/configs/a3y17lte_defconfig
> index 5c15d51fdc..b012b985a3 100644
> --- a/configs/a3y17lte_defconfig
> +++ b/configs/a3y17lte_defconfig
> @@ -23,4 +23,3 @@ CONFIG_HUSH_PARSER=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_I2C=y
>  CONFIG_DM_I2C_GPIO=y
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/a5y17lte_defconfig b/configs/a5y17lte_defconfig
> index 7c9b6b2511..25a7d5bc98 100644
> --- a/configs/a5y17lte_defconfig
> +++ b/configs/a5y17lte_defconfig
> @@ -23,4 +23,3 @@ CONFIG_HUSH_PARSER=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_I2C=y
>  CONFIG_DM_I2C_GPIO=y
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/a7y17lte_defconfig b/configs/a7y17lte_defconfig
> index c7297f7d75..c87379ab39 100644
> --- a/configs/a7y17lte_defconfig
> +++ b/configs/a7y17lte_defconfig
> @@ -23,4 +23,3 @@ CONFIG_HUSH_PARSER=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_I2C=y
>  CONFIG_DM_I2C_GPIO=y
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/apple_m1_defconfig b/configs/apple_m1_defconfig
> index 20d2cff93f..dca6e0ca8b 100644
> --- a/configs/apple_m1_defconfig
> +++ b/configs/apple_m1_defconfig
> @@ -26,4 +26,3 @@ CONFIG_SYS_WHITE_ON_BLACK=y
>  CONFIG_NO_FB_CLEAR=y
>  CONFIG_VIDEO_SIMPLE=y
>  # CONFIG_SMBIOS is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7981_emmc_rfb_defconfig b/configs/mt7981_emmc_rfb_defconfig
> index 76ee2aa2d6..d3e833905f 100644
> --- a/configs/mt7981_emmc_rfb_defconfig
> +++ b/configs/mt7981_emmc_rfb_defconfig
> @@ -62,4 +62,3 @@ CONFIG_MTK_SERIAL=y
>  CONFIG_FAT_WRITE=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7981_rfb_defconfig b/configs/mt7981_rfb_defconfig
> index 3989c79d2b..4bc2173f13 100644
> --- a/configs/mt7981_rfb_defconfig
> +++ b/configs/mt7981_rfb_defconfig
> @@ -65,4 +65,3 @@ CONFIG_DM_SPI=y
>  CONFIG_MTK_SPIM=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7981_sd_rfb_defconfig b/configs/mt7981_sd_rfb_defconfig
> index 9b33245527..8721b4074a 100644
> --- a/configs/mt7981_sd_rfb_defconfig
> +++ b/configs/mt7981_sd_rfb_defconfig
> @@ -62,4 +62,3 @@ CONFIG_MTK_SERIAL=y
>  CONFIG_FAT_WRITE=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7986_rfb_defconfig b/configs/mt7986_rfb_defconfig
> index 4d0cc85d0e..15c31de236 100644
> --- a/configs/mt7986_rfb_defconfig
> +++ b/configs/mt7986_rfb_defconfig
> @@ -65,4 +65,3 @@ CONFIG_DM_SPI=y
>  CONFIG_MTK_SPIM=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7986a_bpir3_emmc_defconfig b/configs/mt7986a_bpir3_emmc_defconfig
> index 3c296ab803..56921f3605 100644
> --- a/configs/mt7986a_bpir3_emmc_defconfig
> +++ b/configs/mt7986a_bpir3_emmc_defconfig
> @@ -62,4 +62,3 @@ CONFIG_MTK_SERIAL=y
>  CONFIG_FAT_WRITE=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7986a_bpir3_sd_defconfig b/configs/mt7986a_bpir3_sd_defconfig
> index f644070f4e..4ed06b72d5 100644
> --- a/configs/mt7986a_bpir3_sd_defconfig
> +++ b/configs/mt7986a_bpir3_sd_defconfig
> @@ -62,4 +62,3 @@ CONFIG_MTK_SERIAL=y
>  CONFIG_FAT_WRITE=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7988_rfb_defconfig b/configs/mt7988_rfb_defconfig
> index d0ed2cc1c9..f7ceaceb30 100644
> --- a/configs/mt7988_rfb_defconfig
> +++ b/configs/mt7988_rfb_defconfig
> @@ -81,4 +81,3 @@ CONFIG_MTK_SPIM=y
>  CONFIG_LZO=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/mt7988_sd_rfb_defconfig b/configs/mt7988_sd_rfb_defconfig
> index 5631eaa338..808c8b9011 100644
> --- a/configs/mt7988_sd_rfb_defconfig
> +++ b/configs/mt7988_sd_rfb_defconfig
> @@ -69,4 +69,3 @@ CONFIG_MTK_SPIM=y
>  CONFIG_LZO=y
>  CONFIG_HEXDUMP=y
>  # CONFIG_EFI_LOADER is not set
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/qcom_defconfig b/configs/qcom_defconfig
> index 37966bfb20..1ce46a0625 100644
> --- a/configs/qcom_defconfig
> +++ b/configs/qcom_defconfig
> @@ -106,4 +106,3 @@ CONFIG_SYS_WHITE_ON_BLACK=y
>  CONFIG_NO_FB_CLEAR=y
>  CONFIG_VIDEO_SIMPLE=y
>  CONFIG_HEXDUMP=y
> -CONFIG_LMB_MAX_REGIONS=64
> diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
> index caaabf39ef..9aa3560c7e 100644
> --- a/configs/stm32mp13_defconfig
> +++ b/configs/stm32mp13_defconfig
> @@ -103,6 +103,3 @@ CONFIG_USB_GADGET_VENDOR_NUM=0x0483
>  CONFIG_USB_GADGET_PRODUCT_NUM=0x5720
>  CONFIG_USB_GADGET_DWC2_OTG=y
>  CONFIG_ERRNO_STR=y
> -# CONFIG_LMB_USE_MAX_REGIONS is not set
> -CONFIG_LMB_MEMORY_REGIONS=2
> -CONFIG_LMB_RESERVED_REGIONS=16
> diff --git a/configs/stm32mp15_basic_defconfig b/configs/stm32mp15_basic_defconfig
> index 2e22bf8600..806935f389 100644
> --- a/configs/stm32mp15_basic_defconfig
> +++ b/configs/stm32mp15_basic_defconfig
> @@ -190,6 +190,3 @@ CONFIG_WDT=y
>  CONFIG_WDT_STM32MP=y
>  # CONFIG_BINMAN_FDT is not set
>  CONFIG_ERRNO_STR=y
> -# CONFIG_LMB_USE_MAX_REGIONS is not set
> -CONFIG_LMB_MEMORY_REGIONS=2
> -CONFIG_LMB_RESERVED_REGIONS=16
> diff --git a/configs/stm32mp15_defconfig b/configs/stm32mp15_defconfig
> index ffe7512650..5f050ee0d0 100644
> --- a/configs/stm32mp15_defconfig
> +++ b/configs/stm32mp15_defconfig
> @@ -166,6 +166,3 @@ CONFIG_WDT=y
>  CONFIG_WDT_STM32MP=y
>  # CONFIG_BINMAN_FDT is not set
>  CONFIG_ERRNO_STR=y
> -# CONFIG_LMB_USE_MAX_REGIONS is not set
> -CONFIG_LMB_MEMORY_REGIONS=2
> -CONFIG_LMB_RESERVED_REGIONS=16
> diff --git a/configs/stm32mp15_trusted_defconfig b/configs/stm32mp15_trusted_defconfig
> index 74deaaba2e..3c591d74af 100644
> --- a/configs/stm32mp15_trusted_defconfig
> +++ b/configs/stm32mp15_trusted_defconfig
> @@ -166,6 +166,3 @@ CONFIG_WDT=y
>  CONFIG_WDT_STM32MP=y
>  # CONFIG_BINMAN_FDT is not set
>  CONFIG_ERRNO_STR=y
> -# CONFIG_LMB_USE_MAX_REGIONS is not set
> -CONFIG_LMB_MEMORY_REGIONS=2
> -CONFIG_LMB_RESERVED_REGIONS=16
> diff --git a/configs/stm32mp25_defconfig b/configs/stm32mp25_defconfig
> index 87038cc773..f5623a19bb 100644
> --- a/configs/stm32mp25_defconfig
> +++ b/configs/stm32mp25_defconfig
> @@ -49,6 +49,3 @@ CONFIG_WDT_STM32MP=y
>  CONFIG_WDT_ARM_SMC=y
>  CONFIG_ERRNO_STR=y
>  # CONFIG_EFI_LOADER is not set
> -# CONFIG_LMB_USE_MAX_REGIONS is not set
> -CONFIG_LMB_MEMORY_REGIONS=2
> -CONFIG_LMB_RESERVED_REGIONS=32
> diff --git a/configs/th1520_lpi4a_defconfig b/configs/th1520_lpi4a_defconfig
> index 49ff92f6de..db80e33870 100644
> --- a/configs/th1520_lpi4a_defconfig
> +++ b/configs/th1520_lpi4a_defconfig
> @@ -79,4 +79,3 @@ CONFIG_BZIP2=y
>  CONFIG_ZSTD=y
>  CONFIG_LIB_RATIONAL=y
>  # CONFIG_EFI_LOADER is not set
> -# CONFIG_LMB_USE_MAX_REGIONS is not set
> diff --git a/lib/Kconfig b/lib/Kconfig
> index b3baa4b85b..072ed0ecfa 100644
> --- a/lib/Kconfig
> +++ b/lib/Kconfig
> @@ -1105,40 +1105,6 @@ config LMB
>         help
>           Support the library logical memory blocks.
>
> -config LMB_USE_MAX_REGIONS
> -       bool "Use a common number of memory and reserved regions in lmb lib"
> -       default y
> -       help
> -         Define the number of supported memory regions in the library logical
> -         memory blocks.
> -         This feature allow to reduce the lmb library size by using compiler
> -         optimization when LMB_MEMORY_REGIONS == LMB_RESERVED_REGIONS.
> -
> -config LMB_MAX_REGIONS
> -       int "Number of memory and reserved regions in lmb lib"
> -       depends on LMB_USE_MAX_REGIONS
> -       default 16
> -       help
> -         Define the number of supported regions, memory and reserved, in the
> -         library logical memory blocks.
> -
> -config LMB_MEMORY_REGIONS
> -       int "Number of memory regions in lmb lib"
> -       depends on !LMB_USE_MAX_REGIONS
> -       default 8
> -       help
> -         Define the number of supported memory regions in the library logical
> -         memory blocks.
> -         The minimal value is CONFIG_NR_DRAM_BANKS.
> -
> -config LMB_RESERVED_REGIONS
> -       int "Number of reserved regions in lmb lib"
> -       depends on !LMB_USE_MAX_REGIONS
> -       default 8
> -       help
> -         Define the number of supported reserved regions in the library logical
> -         memory blocks.
> -
>  config PHANDLE_CHECK_SEQ
>         bool "Enable phandle check while getting sequence number"
>         help
> --
> 2.34.1
>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>


More information about the U-Boot mailing list