[coreboot-gerrit] Change in coreboot[master]: soc/skylake/vr_config: Add VR config for Skylake S

[Angel Pons (Code Review)]

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

Change subject: soc/skylake/vr_config: Add VR config for Skylake S
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35167/3/src/soc/intel/skylake/vr_co... 
File src/soc/intel/skylake/vr_config.c:

https://review.coreboot.org/c/coreboot/+/35167/3/src/soc/intel/skylake/vr_co... 
PS3, Line 148: uint16_t icc_max[NUM_VR_DOMAINS] = {
             : 			VR_CFG_AMP(11.1),
             : 			VR_CFG_AMP(40),
             : 			VR_CFG_AMP(48),
             : 			VR_CFG_AMP(48),
             : 		};
Specifically, NUM_VR_DOMAINS = 5 for FSP 1.1, and the extra element (VR_RING) is in the middle, so values would get mixed up...