From 6be82a4cd8231a496ebe2e7cf6605150cb22e1f4 Mon Sep 17 00:00:00 2001 From: David Milosevic Date: Tue, 18 Oct 2022 19:17:19 +0200 Subject: soc/intel: Add node_num to dimm_info struct + adjust dimm_info_fill The dimm_info structure (defined in src/include/memory_info.h) currently does not hold information about the DIMM's node/controller ID. This patch extends the dimm_info structure by adding a new field for the node ID, called node_num. Also, adapt the dimm_info_fill() function accordingly to populate the newly-added field. Background: These changes are necessary for the Atlas mainboard, where we are currently experiencing issues with the DIMMs device/bank locator. Our 2 DIMMs share the same CHANNEL and DIMM ID but have a distinct NODE ID. By looking at the smbios table we see Channel-0-DIMM-0 for both DIMMs. Thus, we need their NODE IDs in order to distinguish them. This patch was tested by building and booting for the Alderlake-P RVP board, which has the same DIMM slot configuration as the Prodrive Atlas mainboard. Signed-off-by: David Milosevic Change-Id: I6ffa5bdff0ba0e3c4a4a51f2419291fd1278cd68 Reviewed-on: https://review.coreboot.org/c/coreboot/+/68525 Reviewed-by: Jonathan Zhang Reviewed-by: Lean Sheng Tan Tested-by: build bot (Jenkins) --- src/soc/intel/meteorlake/romstage/romstage.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/soc/intel/meteorlake') diff --git a/src/soc/intel/meteorlake/romstage/romstage.c b/src/soc/intel/meteorlake/romstage/romstage.c index 2d3b14415e..bac3cafc93 100644 --- a/src/soc/intel/meteorlake/romstage/romstage.c +++ b/src/soc/intel/meteorlake/romstage/romstage.c @@ -106,7 +106,8 @@ static void save_dimm_info(void) meminfo_hob->VddVoltage[memProfNum], meminfo_hob->EccSupport, src_dimm->MfgId, - src_dimm->SpdModuleType); + src_dimm->SpdModuleType, + node); index++; } } -- cgit v1.2.3