[coreboot-gerrit] Change in ...coreboot[master]: soc/amd/stoneyridge: Reboot if missing MRC cache info

[Marshall Dawson (Code Review)]

Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31160 )

Change subject: soc/amd/stoneyridge: Reboot if missing MRC cache info
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/31160/1/src/soc/amd/common/block/s3/s3_resum... 
File src/soc/amd/common/block/s3/s3_resume.c:

https://review.coreboot.org/#/c/31160/1/src/soc/amd/common/block/s3/s3_resum... 
PS1, Line 34: return;
> Wouldn't we want to reboot here as well?

As a matter of style, I agree.  I haven't looked to see whether mrc_cache_get_current() is kind enough to write base/size to 0 if there's an error.  I'll add it.