[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/xeon_sp: Further share domain creation logics in Xeon-SP

[Shuo Liu (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Lean Sheng Tan, Tim Chu.

Shuo Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81098?usp=email )

Change subject: soc/intel/xeon_sp: Further share domain creation logics in Xeon-SP
......................................................................

Patch Set 4:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/81098/comment/b8e9d241_525eeee3 :
PS4, Line 130: attach_iio_stacks
> Does it need to run after fsp_silicon_init() or can it be run before that?

Put it after fsp_silicon_init() is a safe implementation to make sure all stacks, include IOAT, are usable after that, although in some implementation, it can be pull in to before fsp_silicon_init() (and after fsp_memory_init()).