lmb: Default to not-LMB_USE_MAX_REGIONS
authorTom Rini <trini@konsulko.com>
Wed, 25 Jan 2023 23:08:23 +0000 (18:08 -0500)
committerTom Rini <trini@konsulko.com>
Mon, 6 Feb 2023 17:07:18 +0000 (12:07 -0500)
The LMB code allows for picking a hard limit on the number of regions it
can know of, or to dynamically allocate these regions. The reason for
this choice is to allow for the compiler to perform a size optimization
in the common case. This optimization however, is very small, ranging
from 196 bytes to 15 bytes saved, or in some cases, being larger. Now
that we also have more regions covered by LMB (in order to protect
various parts of our self at run time), the default of 8 is also much
easier to hit and leads to non-obvious error messages (which imply that
an area is protected, not that we're out of areas to add to the list).

Switch to the dynamic use as the default.

Signed-off-by: Tom Rini <trini@konsulko.com>
13 files changed:
configs/a3y17lte_defconfig
configs/a5y17lte_defconfig
configs/a7y17lte_defconfig
configs/dragonboard845c_defconfig
configs/mt7981_emmc_rfb_defconfig
configs/mt7981_rfb_defconfig
configs/mt7981_sd_rfb_defconfig
configs/mt7986_rfb_defconfig
configs/mt7986a_bpir3_emmc_defconfig
configs/mt7986a_bpir3_sd_defconfig
configs/qcs404evb_defconfig
configs/starqltechn_defconfig
lib/Kconfig

index 124fb8d..0215c5f 100644 (file)
@@ -24,4 +24,3 @@ CONFIG_SYS_BOOTM_LEN=0x2000000
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_DM_I2C_GPIO=y
-CONFIG_LMB_MAX_REGIONS=64
index ffb0b3f..d1cd97b 100644 (file)
@@ -24,4 +24,3 @@ CONFIG_SYS_BOOTM_LEN=0x2000000
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_DM_I2C_GPIO=y
-CONFIG_LMB_MAX_REGIONS=64
index 8c1229d..bef2d09 100644 (file)
@@ -24,4 +24,3 @@ CONFIG_SYS_BOOTM_LEN=0x2000000
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_I2C=y
 CONFIG_DM_I2C_GPIO=y
-CONFIG_LMB_MAX_REGIONS=64
index a69d827..cef82f5 100644 (file)
@@ -26,4 +26,3 @@ CONFIG_DM_PMIC=y
 CONFIG_PMIC_QCOM=y
 CONFIG_MSM_GENI_SERIAL=y
 CONFIG_SPMI_MSM=y
-CONFIG_LMB_MAX_REGIONS=64
index b3b37b6..4832a22 100644 (file)
@@ -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
index b7ffb4d..c397527 100644 (file)
@@ -64,4 +64,3 @@ CONFIG_SPI=y
 CONFIG_DM_SPI=y
 CONFIG_MTK_SPIM=y
 CONFIG_HEXDUMP=y
-CONFIG_LMB_MAX_REGIONS=64
index 85be9bb..17592dc 100644 (file)
@@ -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
index ac91c93..1363f9d 100644 (file)
@@ -64,4 +64,3 @@ CONFIG_SPI=y
 CONFIG_DM_SPI=y
 CONFIG_MTK_SPIM=y
 CONFIG_HEXDUMP=y
-CONFIG_LMB_MAX_REGIONS=64
index 2d4876f..354159d 100644 (file)
@@ -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
index 08edfe7..db7ef98 100644 (file)
@@ -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
index dae1551..d3608ca 100644 (file)
@@ -51,4 +51,3 @@ CONFIG_USB_XHCI_DWC3=y
 CONFIG_USB_DWC3=y
 CONFIG_USB_DWC3_GENERIC=y
 CONFIG_USB_STORAGE=y
-CONFIG_LMB_MAX_REGIONS=64
index 7a64f2a..81fd7b2 100644 (file)
@@ -33,4 +33,3 @@ CONFIG_VIDEO=y
 CONFIG_SYS_WHITE_ON_BLACK=y
 CONFIG_VIDEO_SIMPLE=y
 CONFIG_VIDEO_DT_SIMPLEFB=y
-CONFIG_LMB_MAX_REGIONS=64
index 549bd35..59e4304 100644 (file)
@@ -1028,7 +1028,6 @@ config LMB
 config LMB_USE_MAX_REGIONS
        bool "Use a common number of memory and reserved regions in lmb lib"
        depends on LMB
-       default y
        help
          Define the number of supported memory regions in the library logical
          memory blocks.