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

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak 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 6:

(1 comment)

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

https://review.coreboot.org/#/c/32513/6/src/soc/intel/cannonlake/cnl_memcfg_... 
PS6, Line 82: printk(BIOS_EMERG, "nonexistent memory slot: %u", mem_slot);
            : 		die("");
Same thing here, just pass that message to die(); I don't think the slot number is necessary.