soc/intel/jasperlake: Allow mainboard to override DRAM part number

In order to support mainboards that do not store DRAM part number in
the traditional way i.e. within the CBFS SPD for soldered memory, this
change provides a runtime callback to allow mainboards to provide DRAM
part number from a custom location e.g. external EEPROM on dedede.

For other boards it should be a NOP since the weak implementation of
mainboard_get_dram_part_num does nothing.

BUG=b:152019429

Change-Id: I7ba635f5504ba288308d7d7a4935f405f289aa8d
Signed-off-by: Marco Chen <marcochen@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40302
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
This commit is contained in:
Marco Chen 2020-04-10 14:33:19 +08:00 committed by Patrick Georgi
parent dd1a0acc4a
commit 4f771fe98b
2 changed files with 23 additions and 0 deletions

View File

@ -6,6 +6,8 @@
#include <fsp/api.h> #include <fsp/api.h>
/* Provide a callback to allow mainboard to override the DRAM part number. */
bool mainboard_get_dram_part_num(const char **part_num, size_t *len);
void mainboard_memory_init_params(FSPM_UPD *mupd); void mainboard_memory_init_params(FSPM_UPD *mupd);
void systemagent_early_init(void); void systemagent_early_init(void);
void pch_init(void); void pch_init(void);

View File

@ -22,6 +22,12 @@
0x8d, 0x09, 0x11, 0xcf, 0x8b, 0x9f, 0x03, 0x23 \ 0x8d, 0x09, 0x11, 0xcf, 0x8b, 0x9f, 0x03, 0x23 \
} }
bool __weak mainboard_get_dram_part_num(const char **part_num, size_t *len)
{
/* Default weak implementation, no need to override part number. */
return false;
}
/* Save the DIMM information for SMBIOS table 17 */ /* Save the DIMM information for SMBIOS table 17 */
static void save_dimm_info(void) static void save_dimm_info(void)
{ {
@ -36,6 +42,9 @@ static void save_dimm_info(void)
const uint8_t smbios_memory_info_guid[16] = const uint8_t smbios_memory_info_guid[16] =
FSP_SMBIOS_MEMORY_INFO_GUID; FSP_SMBIOS_MEMORY_INFO_GUID;
const uint8_t *serial_num; const uint8_t *serial_num;
const char *dram_part_num = NULL;
size_t dram_part_num_len;
bool is_dram_part_overridden = false;
/* Locate the memory info HOB, presence validated by raminit */ /* Locate the memory info HOB, presence validated by raminit */
meminfo_hob = fsp_find_extension_hob_by_guid( meminfo_hob = fsp_find_extension_hob_by_guid(
@ -57,6 +66,10 @@ static void save_dimm_info(void)
} }
memset(mem_info, 0, sizeof(*mem_info)); memset(mem_info, 0, sizeof(*mem_info));
/* Allow mainboard to override DRAM part number. */
is_dram_part_overridden = mainboard_get_dram_part_num(&dram_part_num,
&dram_part_num_len);
/* Save available DIMM information */ /* Save available DIMM information */
index = 0; index = 0;
dimm_max = ARRAY_SIZE(mem_info->dimm); dimm_max = ARRAY_SIZE(mem_info->dimm);
@ -75,6 +88,14 @@ static void save_dimm_info(void)
if (src_dimm->Status != DIMM_PRESENT) if (src_dimm->Status != DIMM_PRESENT)
continue; continue;
/* If there is no DRAM part number overridden by
* mainboard then use original one. */
if (!is_dram_part_overridden) {
dram_part_num_len = sizeof(src_dimm->ModulePartNum);
dram_part_num = (const char *)
&src_dimm->ModulePartNum[0];
}
u8 memProfNum = meminfo_hob->MemoryProfile; u8 memProfNum = meminfo_hob->MemoryProfile;
serial_num = src_dimm->SpdSave + serial_num = src_dimm->SpdSave +
SPD_SAVE_OFFSET_SERIAL; SPD_SAVE_OFFSET_SERIAL;