[coreboot-gerrit] Change in coreboot[master]: soc/intel: skl,cnl,icl: rely on TOLUM as cbmem_top returned by FSP

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36136 )

Change subject: soc/intel: skl,cnl,icl: rely on TOLUM as cbmem_top returned by FSP
......................................................................

Patch Set 10: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/36136/10/src/soc/intel/skylake/memm... 
File src/soc/intel/skylake/memmap.c:

https://review.coreboot.org/c/coreboot/+/36136/10/src/soc/intel/skylake/memm... 
PS10, Line 73: 		die("9.3: FSP_BOOTLOADER_TOLUM_HOB missing!\n");
Should we maybe move this into fsp_find_bootloader_tolum()? Is there
any call where we wouldn't want to die()?