Hello build bot (Jenkins), Nico Huber, Patrick Georgi, Jonathan Zhang, Maxim Polyakov, Johnny Lin, Paul Menzel, Andrey Petrov, Angel Pons, Andrey Petrov, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/41116
to look at the new patch set (#4).
Change subject: soc/intel/xeon_sp: make CPX ramstage.h common for CPX, SKX ......................................................................
soc/intel/xeon_sp: make CPX ramstage.h common for CPX, SKX
CB:41106 revealed that mb/intel/cedarisland already sets FSP-S UPD (see CB:40735) while the required includes are still missing in CPX. Buildbot did not fail because `ramstage.c` never was (implicitly) included.
Fix this problem by making SKX/CPX share a common ramstage header for now by moving the one from SKX.
Test: Build cedarisland_crb
Change-Id: I9cd25edd167ec71ee98c7ffa4fa6f95ca73a75e9 Signed-off-by: Michael Niewöhner foss@mniewoehner.de --- D src/soc/intel/xeon_sp/cpx/include/soc/ramstage.h R src/soc/intel/xeon_sp/include/soc/ramstage.h 2 files changed, 0 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/16/41116/4