[coreboot-gerrit] Change in coreboot[master]: Revert "broadwell: update processor power limits configuration"

[Angel Pons (Code Review)]

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

Change subject: Revert "broadwell: update processor power limits configuration"
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45011/1/src/soc/intel/broadwell/sys... 
File src/soc/intel/broadwell/systemagent.c:

https://review.coreboot.org/c/coreboot/+/45011/1/src/soc/intel/broadwell/sys... 
PS1, Line 428: 	config = config_of_soc();
> This is where things clash. It seems it was forgotten to actually […]

Indeed. CB:46700 changes the layout of the chip config and broke booting because the PL values were around 1.5 kW. Apparently, the MSR doesn't think such a high value makes sense, a GPF# happens.