[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/xeon_sp: Add device to proximity domain map utils

[Shuo Liu (Code Review)]

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

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

Change subject: soc/intel/xeon_sp: Add device to proximity domain map utils
......................................................................

Patch Set 6:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81440/comment/cfc5c72c_c4379cfa :
PS6, Line 18: SNC
> What does SNC mean? I would at least expand the acronym

Sub-NUMA cluster, commit message updated.

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

https://review.coreboot.org/c/coreboot/+/81440/comment/84a46e5d_e62bd17f :
PS6, Line 122: uint32_t
> Use `enum proximity_domain_type`. […]

Nice catch! Updated.

https://review.coreboot.org/c/coreboot/+/81440/comment/86f43410_b541a24b :
PS6, Line 130: 	return PD_TYPE_MAX;
> Would it be a good idea to log a warning/error when the pd couldn't be found?

Sure, updated.

https://review.coreboot.org/c/coreboot/+/81440/comment/a2e2376f_76c0d550 :
PS6, Line 147: 	return PD_TYPE_MAX;
> Same here

Done