[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/systemagent: Add SOC_INTEL_COMMON_BLOCK_SA_VERSION_2

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38456 )

Change subject: soc/intel/common/systemagent: Add SOC_INTEL_COMMON_BLOCK_SA_VERSION_2
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38456/2/src/soc/intel/common/block/... 
File src/soc/intel/common/block/acpi/acpi/northbridge.asl:

https://review.coreboot.org/c/coreboot/+/38456/2/src/soc/intel/common/block/... 
PS2, Line 49: #endif
Can’t this be detected at run-time, and the table be created at run-time?

https://review.coreboot.org/c/coreboot/+/38456/2/src/soc/intel/common/block/... 
File src/soc/intel/common/block/systemagent/systemagent_early.c:

https://review.coreboot.org/c/coreboot/+/38456/2/src/soc/intel/common/block/... 
PS2, Line 44: (unsigned long)
Why is this needed?