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

[Jamie Chen (Code Review)]

Jamie Chen 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 2:

(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 ?

CML v1 U6/U4/U2 have the same CPUID.
So I use Did to identify them.

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");
> ok, Jamie you might want to reduce the log a little bit. the width of this line is 100 now.

Reduce the length to 90.