[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: Add required FSP UPD changes for CML

[Maulik V Vaghela (Code Review)]

Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31284 )

Change subject: soc/intel/cannonlake: Add required FSP UPD changes for CML
......................................................................

Patch Set 14:

(4 comments)

https://review.coreboot.org/#/c/31284/13/src/soc/intel/cannonlake/fsp_params... 
File src/soc/intel/cannonlake/fsp_params.c:

https://review.coreboot.org/#/c/31284/13/src/soc/intel/cannonlake/fsp_params... 
PS13, Line 28: SerialIoDev
> just use serial_io_dev

Done

https://review.coreboot.org/#/c/31284/13/src/soc/intel/cannonlake/fsp_params... 
PS13, Line 59: for (i = 0; i < CONFIG_SOC_INTEL_COMMON_BLOCK_GSPI_MAX; i++, dev_offset++) {
> line greater than 80?

Done

https://review.coreboot.org/#/c/31284/13/src/soc/intel/cannonlake/fsp_params... 
PS13, Line 65: params->SerialIoSpiMode[i] = config->SerialIoDevMode[dev_offset];
> line greater than 80?

Done

https://review.coreboot.org/#/c/31284/13/src/soc/intel/cannonlake/fsp_params... 
PS13, Line 74: params->SerialIoUartMode[i] = config->SerialIoDevMode[dev_offset];
> line greater than 80?

Done