Johnny Lin has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/64041 )
Change subject: include/memory_info.h: Make DIMM_INFO_TOTAL a Kconfig variable ......................................................................
include/memory_info.h: Make DIMM_INFO_TOTAL a Kconfig variable
For multiple sockets platform 16 may not be enough, so make it a Kconfig variable.
Change-Id: If72a8622ac1e7e67646aa4dd24b99637fb8b1297 Signed-off-by: Johnny Lin johnny_lin@wiwynn.com --- M src/Kconfig M src/include/memory_info.h 2 files changed, 9 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/64041/1
diff --git a/src/Kconfig b/src/Kconfig index dc81979..cae678e 100644 --- a/src/Kconfig +++ b/src/Kconfig @@ -811,6 +811,14 @@ successive numbers from a monotonically increasing counter, with one counter for each device class.
+config DIMM_INFO_TOTAL + depends on GENERATE_SMBIOS_TABLES + int + default 16 + help + It configures the size of array memory_info->dimm_info that holds all + memory DIMM information generated from SOC for SMBIOS population. + config SMBIOS_PROVIDED_BY_MOBO bool default n diff --git a/src/include/memory_info.h b/src/include/memory_info.h index 9d02aef..d27f31e 100644 --- a/src/include/memory_info.h +++ b/src/include/memory_info.h @@ -8,7 +8,6 @@
#define DIMM_INFO_SERIAL_SIZE 4 #define DIMM_INFO_PART_NUMBER_SIZE 33 -#define DIMM_INFO_TOTAL 16
/** * If this table is filled and put in CBMEM, @@ -107,7 +106,7 @@
/* active DIMM configuration */ uint8_t dimm_cnt; - struct dimm_info dimm[DIMM_INFO_TOTAL]; + struct dimm_info dimm[CONFIG_DIMM_INFO_TOTAL]; } __packed;
/*