[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Use common cpu/intel romstage entry

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46658 )

Change subject: soc/intel/xeon_sp: Use common cpu/intel romstage entry
......................................................................

Patch Set 9:

(1 comment)

https://review.coreboot.org/c/coreboot/+/46658/9/src/soc/intel/xeon_sp/Kconf... 
File src/soc/intel/xeon_sp/Kconfig:

https://review.coreboot.org/c/coreboot/+/46658/9/src/soc/intel/xeon_sp/Kconf... 
PS9, Line 58: 	select NO_SMM
> Why do you add this? What does it to do with this commit subject?

The common cpu/intel/car/romstage.c code has different codepaths depending on CONFIG_TSEG. So in this CL CONFIG_NO_SMM needs to be explicitly set. It is removed in one of the next CL when SMM is properly set up. I'll update the commit message to reflect this.