[coreboot-gerrit] [XL] Change in coreboot[master]: soc/intel/xeon_sp/spr: add ACPI code

[Patrick Rudolph (Code Review)]

Attention is currently required from: Jonathan Zhang, Simon Chou, TimLiu-SMCI, Johnny Lin, Christian Walter, Shuming Chu (Shuming), Arthur Heymans.

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

Change subject: soc/intel/xeon_sp/spr: add ACPI code
......................................................................

Patch Set 13:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/71967/comment/9d68be0f_00963f2d 
PS13, Line 387: 	for (uint8_t socket = 0; socket < hob->PlatformData.numofIIO; ++socket) {
> Got you. I think we should use hob->PlatformData. […]

it's fine to generate the DSDT from CONFIG_MAX_SOCKET as long as the SSDT is
1) generating all the resources consumed by DSDT
2) setting _STA and _CRS to 0 for all unpopulated devices/sockets