[coreboot-gerrit] Change in ...coreboot[master]: src/soc/intel/cannonlake: Add PsysPmax setting

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31505 )

Change subject: src/soc/intel/cannonlake: Add PsysPmax setting
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31505/2/src/soc/intel/cannonlake/fsp_params.... 
File src/soc/intel/cannonlake/fsp_params.c:

https://review.coreboot.org/#/c/31505/2/src/soc/intel/cannonlake/fsp_params.... 
PS2, Line 93: printk(BIOS_DEBUG, "psys_pmax = %dW\n", tconfig->PsysPmax);
How is this helpful? Now this would just end up printing the default value in tconfig->PsysPmax?