[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/xeon_sp: Make NUMA support by default

[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, Patrick Rudolph, Tim Chu, Ziang Wang.

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

Change subject: soc/intel/xeon_sp: Make NUMA support by default
......................................................................

Patch Set 5:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/81439/comment/541ecafd_1bb435e5 :
PS5, Line 75: #if CONFIG(SOC_INTEL_HAS_CXL)
> Is this preprocessor needed? Unless something causes a compilation error on the non-CXL case, I woul […]

Done

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

https://review.coreboot.org/c/coreboot/+/81439/comment/85d9a527_60860ec3 :
PS5, Line 578: 	if (CONFIG(SOC_INTEL_HAS_NUMA)) {
> Is there any Xeon-SP platform that is not NUMA?

Yes, updated.