[PATCH v4 07/27] lmb: remove config symbols used for lmb region count
Ilias Apalodimas
ilias.apalodimas at linaro.org
Tue Aug 27 08:24:12 CEST 2024
On Mon, 26 Aug 2024 at 15:00, 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>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
> Changes since V3: None
>
> 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 8852e83a52..edbb624baf 100644
> --- a/configs/qcom_defconfig
> +++ b/configs/qcom_defconfig
> @@ -117,4 +117,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 2059219a12..f8ac8daad3 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