[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: Support different SPD read type for each slot

[Furquan Shaikh (Code Review)]

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

Change subject: soc/intel/cannonlake: Support different SPD read type for each slot
......................................................................

Patch Set 15: Code-Review+2

(1 comment)

https://review.coreboot.org/#/c/32513/13/src/soc/intel/cannonlake/cnl_memcfg... 
File src/soc/intel/cannonlake/cnl_memcfg_init.c:

https://review.coreboot.org/#/c/32513/13/src/soc/intel/cannonlake/cnl_memcfg... 
PS13, Line 57: 	static size_t last_set_spd_data_len;
> OK, SGTM.

Static variables should end up in .bss which is initialized to 0. So, by default this will be set to 0.