[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp/skx: Fix MADT CPU indexes

[Marc Jones (Code Review)]

Marc Jones has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47277 )

Change subject: soc/intel/xeon_sp/skx: Fix MADT CPU indexes
......................................................................

soc/intel/xeon_sp/skx: Fix MADT CPU indexes

The CPU index wasn't getting updated. Confirm MADT sets IOAPIC and CPU
ID numbers.

Change-Id: I72430cc48f4609ac408e723172ba1ed263cca8e3
Signed-off-by: Marc Jones 
---
M src/soc/intel/xeon_sp/skx/soc_acpi.c
1 file changed, 2 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/47277/1

diff --git a/src/soc/intel/xeon_sp/skx/soc_acpi.c b/src/soc/intel/xeon_sp/skx/soc_acpi.c
index b392619..95563f5 100644
--- a/src/soc/intel/xeon_sp/skx/soc_acpi.c
+++ b/src/soc/intel/xeon_sp/skx/soc_acpi.c
@@ -182,7 +182,7 @@
 unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current)
 {
    struct device *cpu;
-	int num_cpus = 0;
+	uint8_t num_cpus = 0;

for (cpu = all_devices; cpu; cpu = cpu->next) {
    	if ((cpu->path.type != DEVICE_PATH_APIC) ||
@@ -193,6 +193,7 @@
    		continue;
    	current += acpi_create_madt_lapic((acpi_madt_lapic_t *)current,
    		num_cpus, cpu->path.apic.apic_id);
+		num_cpus++;
    }

return current;