[coreboot-gerrit] [XL] Change in coreboot[master]: soc/intel/xeon_sp/spr: Add ACPI support for Sapphire Rapids

[Patrick Rudolph (Code Review)]

Attention is currently required from: Marc Jones, Simon Chou, Jonathan Zhang, TimLiu-SMCI, Johnny Lin, Paul Menzel, Christian Walter, Angel Pons, Arthur Heymans, Shuming Chu (Shuming), Shelly Chang, Elyes Haouas.

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71967 )

Change subject: soc/intel/xeon_sp/spr: Add ACPI support for Sapphire Rapids
......................................................................

Patch Set 25:

(1 comment)

File src/soc/intel/xeon_sp/spr/soc_acpi.c:

https://review.coreboot.org/c/coreboot/+/71967/comment/8e0d96b2_9a0054fd 
PS13, Line 387: 	for (uint8_t socket = 0; socket < hob->PlatformData.numofIIO; ++socket) {
> PlatformData. […]

There's not. CONFIG_MAX_SOCKET is a preprocessor directive, hob->PlatformData.numofIIO is a runtime variable.