[coreboot-gerrit] [L] Change in coreboot[master]: soc/intel/xeon_sp/spr: Add Sapphire Rapids ramstage code

[Felix Held (Code Review)]

Attention is currently required from: Jonathan Zhang.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/72443 )

Change subject: soc/intel/xeon_sp/spr: Add Sapphire Rapids ramstage code
......................................................................

Patch Set 13:

(1 comment)

File src/soc/intel/xeon_sp/spr/cpu.c:

https://review.coreboot.org/c/coreboot/+/72443/comment/db5d4281_ec7dde33 
PS13, Line 177: static const struct cpu_device_id cpu_table[] = {
              : 	{X86_VENDOR_INTEL, CPUID_SAPPHIRERAPIDS_SP_D},
              : 	{X86_VENDOR_INTEL, CPUID_SAPPHIRERAPIDS_SP_E0},
              : 	{X86_VENDOR_INTEL, CPUID_SAPPHIRERAPIDS_SP_E2},
              : 	{X86_VENDOR_INTEL, CPUID_SAPPHIRERAPIDS_SP_E3},
              : 	{X86_VENDOR_INTEL, CPUID_SAPPHIRERAPIDS_SP_E4},
              : 	{X86_VENDOR_INTEL, CPUID_SAPPHIRERAPIDS_SP_Ex},
              : 	{0, 0},
              : };
> this probably won't behave as expected due to the last element of the struct that i introduced in ht […]

pushed a fix: CB:73835