[M] Change in coreboot[master]: drivers/intel/fsp2_0/include/fsp: accomodate xeon_sp FSPX_CONFIG defi...

[Arthur Heymans (Code Review)]

Attention is currently required from: Jonathan Zhang, Johnny Lin, Christian Walter, Andrey Petrov, Tim Chu.

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

Change subject: drivers/intel/fsp2_0/include/fsp: accomodate xeon_sp FSPX_CONFIG definitions
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/xeon_sp/cpx/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/69090/comment/ade05bc7_b74149a8 
PS1, Line 19: CPPFLAGS_common += -i
You now need to include soc_binding.h in the right files 'above' other header files. Because needing an explicit order in header files is painful (and sometimes not possible if it's common code) a CPP flag is easier to deal with.