[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Don't add memory resource twice

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47173 )

Change subject: soc/intel/xeon_sp: Don't add memory resource twice
......................................................................

soc/intel/xeon_sp: Don't add memory resource twice

The resource function is called for each device VID/DID. Only add
the memory resource map from the boot CPU (bus 0) and not for each
socket/CPU. This is a NUMA architecture and has a shared memory map.
All the resources must match across the sockets/CPUs, so they should
only be added to the map once.

Change-Id: Ia336f604441ae8d30b8418300da7c34ab9907cae
Signed-off-by: Marc Jones 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47173
Tested-by: build bot (Jenkins) 
Reviewed-by: Jay Talbott 
Reviewed-by: Arthur Heymans 
---
M src/soc/intel/xeon_sp/uncore.c
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved
  Jay Talbott: Looks good to me, but someone else must approve

diff --git a/src/soc/intel/xeon_sp/uncore.c b/src/soc/intel/xeon_sp/uncore.c
index 332b9a4..15a9f0f 100644
--- a/src/soc/intel/xeon_sp/uncore.c
+++ b/src/soc/intel/xeon_sp/uncore.c
@@ -155,6 +155,10 @@
    struct resource *resource;
    int index = *res_count;

+	/* Only add dram resources once. */
+	if (dev->bus->secondary != 0)
+		return;
+
    fsp_find_reserved_memory(&fsp_mem);

/* Read in the MAP registers and report their values. */