[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Add IccMax limitation for CML U22

[Subrata Banik (Code Review)]

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

Change subject: soc/intel/cannonlake: Add IccMax limitation for CML U22
......................................................................

Patch Set 1:

(2 comments)

https://review.coreboot.org/c/coreboot/+/37189/1/src/soc/intel/cannonlake/vr... 
File src/soc/intel/cannonlake/vr_config.c:

https://review.coreboot.org/c/coreboot/+/37189/1/src/soc/intel/cannonlake/vr... 
PS1, Line 109: did 
can't we check CPUID ? or its same ?

https://review.coreboot.org/c/coreboot/+/37189/1/src/soc/intel/cannonlake/vr... 
PS1, Line 111: 			printk(BIOS_DEBUG, "VR_CONFIG: IccMax[IA_CORE] has been limited to 35A\n");
> over 80 chars limit

right now we have 96 i guess.

question is do you need this debug msg ?