Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46662 )
Change subject: soc/intel/xeon_sp/skx: Add missing includes ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46662/1/src/soc/intel/xeon_sp/skx/c... File src/soc/intel/xeon_sp/skx/cpu.c:
https://review.coreboot.org/c/coreboot/+/46662/1/src/soc/intel/xeon_sp/skx/c... PS1, Line 11: #include <soc/soc_util.h> : #include <soc/util.h>
these includes really don't have good names. No wonder stuff like this is happening.
Done