Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/40486 )
Change subject: mb/google/deltaur: Add memory topology SODIMM and MEMORYDOWN ......................................................................
mb/google/deltaur: Add memory topology SODIMM and MEMORYDOWN
Update memory topology for spd info. Deltan supports SODIMM and Deltaur supports MEMORYDOWN.
BUG=b:151702387
Signed-off-by: Eric Lai ericr_lai@compal.corp-partner.google.com Change-Id: If314894325d6f222807030a36f8c4cefecfe5bd3 Reviewed-on: https://review.coreboot.org/c/coreboot/+/40486 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Tim Wawrzynczak twawrzynczak@chromium.org Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net --- M src/mainboard/google/deltaur/variants/deltan/memory.c M src/mainboard/google/deltaur/variants/deltaur/memory.c 2 files changed, 2 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Tim Wawrzynczak: Looks good to me, approved
diff --git a/src/mainboard/google/deltaur/variants/deltan/memory.c b/src/mainboard/google/deltaur/variants/deltan/memory.c index 90fa642..d51ba70 100644 --- a/src/mainboard/google/deltaur/variants/deltan/memory.c +++ b/src/mainboard/google/deltaur/variants/deltan/memory.c @@ -47,6 +47,7 @@ void variant_memory_init(FSP_M_CONFIG *mem_cfg) { const struct spd_info spd_info = { + .topology = SODIMM, .smbus_info[0] = {.addr_dimm0 = 0xa0, .addr_dimm1 = 0 }, .smbus_info[1] = {.addr_dimm0 = 0xa4, diff --git a/src/mainboard/google/deltaur/variants/deltaur/memory.c b/src/mainboard/google/deltaur/variants/deltaur/memory.c index c2df467..c25df39 100644 --- a/src/mainboard/google/deltaur/variants/deltaur/memory.c +++ b/src/mainboard/google/deltaur/variants/deltaur/memory.c @@ -83,6 +83,7 @@ { const struct lpddr4x_cfg *board_cfg = variant_memory_params(); const struct spd_info spd_info = { + .topology = MEMORY_DOWN, .md_spd_loc = SPD_CBFS, .cbfs_index = variant_memory_sku(), };