John Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33152 )
Change subject: src/soc/intel: Fix Coverity scan report ......................................................................
Patch Set 3:
(1 comment)
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/33152/2/src/soc/intel/broadwell/romstage/ram... File src/soc/intel/broadwell/romstage/raminit.c:
https://review.coreboot.org/#/c/33152/2/src/soc/intel/broadwell/romstage/ram... PS2, Line 126: if (!mem_info)
Probably add a print indicating error adding cbmem entry?
done