[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Move common northbridge ACPI to nb_acpi.c

[Jonathan Zhang (Code Review)]

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

Change subject: soc/intel/xeon_sp: Move common northbridge ACPI to nb_acpi.c
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45850/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/45850/5//COMMIT_MSG@12 
PS5, Line 12: and CSTACKS that are managed with #if (CONFIG(SOC_INTEL_*_SP)).
> Maybe try to convince Intel to have SKX FSP use the same API as CPX FSP?

Intel would not do it until there is strong revenue opportunity. Another route is an IBV gets SKX-SP FSP maintenance right from Intel and gets funding to do so.