[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 2:

(1 comment)

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

https://review.coreboot.org/#/c/31160/2/src/soc/amd/common/block/s3/s3_resum... 
PS2, Line 50: size
> And do we need to check the full size?  Would 16 or 32 bytes be sufficient?

Good call.  It's not like we're trying to fully check the integrity, although that might be a nice todo.