[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/xeon_sp: Add PD_TYPE_CLUSTER

[Patrick Rudolph (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Chen, Gang C, Christian Walter, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Paul Menzel, Shuo Liu, Tim Chu, Ziang Wang.

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

Change subject: soc/intel/xeon_sp: Add PD_TYPE_CLUSTER
......................................................................

Patch Set 7: Code-Review+2

(2 comments)

File src/soc/intel/xeon_sp/include/soc/numa.h:

https://review.coreboot.org/c/coreboot/+/81443/comment/6c9946cb_da4190dc?usp... :
PS7, Line 22: PD_TYPE_CLUSTER
need a comment that explains what a cluster is about. It doesn't exists in ACPI spec.

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

https://review.coreboot.org/c/coreboot/+/81443/comment/100822b1_f2b46113?usp... :
PS5, Line 40: When
> Good catch! Updated. […]

Done