[coreboot-gerrit] Change in coreboot[master]: WIP: xeonsp: Add common IIO-related code

[Patrick Rudolph (Code Review)]

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

Change subject: WIP: xeonsp: Add common IIO-related code
......................................................................

Patch Set 8:

(2 comments)

Intel document Reference Number: 329595-002 seems to cover a few registers, but no implementation details.

https://review.coreboot.org/c/coreboot/+/38986/8/src/cpu/intel/xeonsp/iio.c 
File src/cpu/intel/xeonsp/iio.c:

https://review.coreboot.org/c/coreboot/+/38986/8/src/cpu/intel/xeonsp/iio.c@... 
PS8, Line 394: static void get_iiostack_info(struct iiostack_resource *info)
why is this data read from FSP instead of devicetree or PCI space?

https://review.coreboot.org/c/coreboot/+/38986/8/src/cpu/intel/xeonsp/iio.c@... 
PS8, Line 534: n
isn't a "stack" simply an MMCONF address area or a subregion of that?