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

[Rocky Phagura (Code Review)]

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

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

Patch Set 6:

(4 comments)

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

https://review.coreboot.org/c/coreboot/+/42289/6/src/soc/intel/xeon_sp/pmc.c... 
PS6, Line 11: console
> move up to be in alphabetical order

Done

https://review.coreboot.org/c/coreboot/+/42289/6/src/soc/intel/xeon_sp/pmc.c... 
PS6, Line 21: #if defined(__SIMPLE_DEVICE__)
> This file will be built in SMM at some point, that's why there's so many guards. […]

Done

https://review.coreboot.org/c/coreboot/+/42289/6/src/soc/intel/xeon_sp/pmc.c... 
PS6, Line 27: dev
> PCH_DEV_PMC

Done

https://review.coreboot.org/c/coreboot/+/42289/6/src/soc/intel/xeon_sp/pmc.c... 
PS6, Line 32: dev
> PCH_DEV_PMC

Done