build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47300 )
Change subject: soc/intel/xeon_sp/acpi.c Look over HOB stack for MADT generation ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/47300/1/src/soc/intel/xeon_sp/acpi.... File src/soc/intel/xeon_sp/acpi.c:
https://review.coreboot.org/c/coreboot/+/47300/1/src/soc/intel/xeon_sp/acpi.... PS1, Line 75: static unsigned long add_madt_ioapic(unsigned long current, int socket, int stack, int ioapic_id, line over 96 characters
https://review.coreboot.org/c/coreboot/+/47300/1/src/soc/intel/xeon_sp/acpi.... PS1, Line 107: const STACK_RES *ri = &hob->PlatformData.IIO_resource[socket].StackRes[stack]; line over 96 characters
https://review.coreboot.org/c/coreboot/+/47300/1/src/soc/intel/xeon_sp/acpi.... PS1, Line 114: current += add_madt_ioapic(current, socket, stack, ioapic_id, ri->IoApicBase, gsi_base); line over 96 characters