Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/70662 )
Change subject: include/memory_info.h: Add soc_num to dimm_info struct ......................................................................
include/memory_info.h: Add soc_num to dimm_info struct
Sometimes, server platforms may have more than one socket on server board. However, there's no field to store information about which socket the DIMM comes from in dimm_info structure.
This patch adds soc_num field in dimm_info structure to store socket ID of the DIMM.
Signed-off-by: Tim Chu Tim.Chu@quantatw.com Change-Id: I1b9e2b87fda2d7c32ecb8ce9d989795c8b869cea Reviewed-on: https://review.coreboot.org/c/coreboot/+/70662 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Jonathan Zhang jonzhang@fb.com Reviewed-by: Marc Jones marc@marcjonesconsulting.com --- M src/include/memory_info.h 1 file changed, 25 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Marc Jones: Looks good to me, approved Jonathan Zhang: Looks good to me, but someone else must approve
diff --git a/src/include/memory_info.h b/src/include/memory_info.h index 7534a81..59b82fd 100644 --- a/src/include/memory_info.h +++ b/src/include/memory_info.h @@ -34,6 +34,10 @@ uint16_t ddr_frequency; uint8_t rank_per_dimm; /* + * Socket-ID + */ + uint8_t soc_num; + /* * Memory-Controller-ID */ uint8_t ctrlr_num;