[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp/acpi: Fix uncore dsdt for multiple cpus

[Marc Jones (Code Review)]

Marc Jones has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47060 )

Change subject: soc/intel/xeon_sp/acpi: Fix uncore dsdt for multiple cpus
......................................................................

soc/intel/xeon_sp/acpi: Fix uncore dsdt for multiple cpus

Fix the asl to use CONFIG_MAX_CPUS  to create entries for
multiple cpu uncores. Don't add the RTxx resource entries multiple
times. The function is called for each CPUs.

Change-Id: Ia4eb9716ae4bd72fb4eb98649105be629623cbef
Signed-off-by: Marc Jones 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47060
Reviewed-by: Jay Talbott 
Reviewed-by: Stefan Reinauer 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/xeon_sp/acpi/iiostack.asl
M src/soc/intel/xeon_sp/cpx/soc_acpi.c
M src/soc/intel/xeon_sp/skx/soc_acpi.c
3 files changed, 9 insertions(+), 1 deletion(-)

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

diff --git a/src/soc/intel/xeon_sp/acpi/iiostack.asl b/src/soc/intel/xeon_sp/acpi/iiostack.asl
index dca5569..4b2b65b 100644
--- a/src/soc/intel/xeon_sp/acpi/iiostack.asl
+++ b/src/soc/intel/xeon_sp/acpi/iiostack.asl
@@ -69,7 +69,7 @@
 MAKE_IIO_DEV(02, 20)
 MAKE_IIO_DEV(03, 28)

-#if MAX_SOCKET > 1
+#if (CONFIG_MAX_SOCKET > 1)
 MAKE_IIO_DEV(06, 40)
 MAKE_IIO_DEV(07, 50)
 MAKE_IIO_DEV(08, 60)
diff --git a/src/soc/intel/xeon_sp/cpx/soc_acpi.c b/src/soc/intel/xeon_sp/cpx/soc_acpi.c
index d60684b..c35c248 100644
--- a/src/soc/intel/xeon_sp/cpx/soc_acpi.c
+++ b/src/soc/intel/xeon_sp/cpx/soc_acpi.c
@@ -60,6 +60,10 @@
 {
    struct iiostack_resource stack_info = {0};

+	/* Only add RTxx entries once. */
+	if (device->bus->secondary != 0)
+		return;
+
    get_iiostack_info(&stack_info);

acpigen_write_scope("\_SB");
diff --git a/src/soc/intel/xeon_sp/skx/soc_acpi.c b/src/soc/intel/xeon_sp/skx/soc_acpi.c
index df2550c..b392619 100644
--- a/src/soc/intel/xeon_sp/skx/soc_acpi.c
+++ b/src/soc/intel/xeon_sp/skx/soc_acpi.c
@@ -104,6 +104,10 @@
    const IIO_UDS *hob = fsp_find_extension_hob_by_guid(uds_guid, &hob_size);
    assert(hob != NULL && hob_size != 0);

+	/* Only add RTxx entries once. */
+	if (device->bus->secondary != 0)
+		return;
+
    acpigen_write_scope("\_SB");
    for (int socket = 0; socket < hob->PlatformData.numofIIO; ++socket) {
    	IIO_RESOURCE_INSTANCE iio_resource =