[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: memory spd data debug

[Furquan Shaikh (Code Review)]

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

Change subject: soc/intel/cannonlake: memory spd data debug
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35235/3/src/soc/intel/cannonlake/cn... 
File src/soc/intel/cannonlake/cnl_memcfg_init.c:

https://review.coreboot.org/c/coreboot/+/35235/3/src/soc/intel/cannonlake/cn... 
PS3, Line 115: 		last_spd_index = spd_index;
Wouldn't it be better to print here? If the same SPD is used for multiple slots, you would end up printing multiple times above.