[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Add chip config to override CPU flex ratio

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36864 )

Change subject: soc/intel/cannonlake: Add chip config to override CPU flex ratio
......................................................................

Patch Set 8:

(1 comment)
> Patch Set 8:
>
> (1 comment)
>
> I would still like to have a detailed documentation of the UPD. It's
> still unclear to me how the system is affected after boot, as there
> is also `BootFrequency` which seems independent from this.

I guess you can refer to BWG chapter 11 title power management

https://review.coreboot.org/c/coreboot/+/36864/2/src/soc/intel/cannonlake/ro... 
File src/soc/intel/cannonlake/romstage/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/36864/2/src/soc/intel/cannonlake/ro... 
PS2, Line 80: if (CONFIG(OVERRIDE_CPU_FLEX_RATIO))
> Heh, interesting. I would have suspected that such things can impact […]

yes I remember Kyosti always had question why 1001 is taking that long time. so we have answer for that hopefully now