[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Enable PMC support

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42289 )

Change subject: soc/intel/xeon_sp: Enable PMC support
......................................................................

Patch Set 7: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/42289/7/src/soc/intel/xeon_sp/pmc.c 
File src/soc/intel/xeon_sp/pmc.c:

https://review.coreboot.org/c/coreboot/+/42289/7/src/soc/intel/xeon_sp/pmc.c... 
PS7, Line 3: #include "chip.h"
very minor nit: Generally, the "foo.h" includes are placed at the bottom, separated from the <foo.h> includes. I'd rather reorder all the xeon_sp includes at once in a separate patch, though.