[coreboot-gerrit] [M] Change in coreboot[master]: soc/intel/xeon_sp/chip_common.c: Add support for SPR

[Jonathan Zhang (Code Review)]

Attention is currently required from: Simon Chou, Johnny Lin, Christian Walter, Arthur Heymans, Tim Chu.

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71965 )

Change subject: soc/intel/xeon_sp/chip_common.c: Add support for SPR
......................................................................

Patch Set 3:

(1 comment)

File src/soc/intel/xeon_sp/chip_common.c:

https://review.coreboot.org/c/coreboot/+/71965/comment/3b073ea2_bca2c82e 
PS2, Line 519: max_subordinate
> Yes this depends on CB:59395, should we make it depends on it?

[CB:59395] needs to be merged in first; Arthur, please let us know if there is anything we could do to help.
In general, spr-upstream are dependents for str-enable, some are hard dependents (like in this case), others are softer.