[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 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36864/3/src/soc/intel/cannonlake/ch... 
File src/soc/intel/cannonlake/chip.h:

https://review.coreboot.org/c/coreboot/+/36864/3/src/soc/intel/cannonlake/ch... 
PS3, Line 428: /*
             : 	 * Override CPU ratio value:
             : 	 * 0: Use FIT programmed default value,
             : 	 * 1: coreboot to override CPU flex ratio
             : 	 */
             : 	uint8_t cpu_ratio_override;
> make sense will check with ==1

Done

https://review.coreboot.org/c/coreboot/+/36864/3/src/soc/intel/cannonlake/ch... 
PS3, Line 437:  * CPU ratio value controls the maximum processor non-turbo ratio
             : 	 * Valid Range 0 to 63. CPU Ratio is 0 when disabled
> Was trying to echo what ever header says. […]

Done