Change in coreboot[master]: soc/intel/xeon_sp/cpx: Finalize PCU configuration

[Angel Pons (Code Review)]

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

Change subject: soc/intel/xeon_sp/cpx: Finalize PCU configuration
......................................................................

Patch Set 27:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40386/27/src/soc/intel/xeon_sp/cpx/... 
File src/soc/intel/xeon_sp/cpx/soc_util.c:

https://review.coreboot.org/c/coreboot/+/40386/27/src/soc/intel/xeon_sp/cpx/... 
PS27, Line 304: 	uint32_t sbsp_socket_id = 0;
> Yes, it may not be socket 0. I will add a comment about it. […]

I would add a FIXME comment:

/* FIXME: This may need to be changed for multi-socket platforms */