On Thu, 4 Jul 2024 at 10:36, Sughosh Ganu <sughosh.g...@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.g...@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.apalodi...@linaro.org>
