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

(1 comment)

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

https://review.coreboot.org/#/c/31160/3/src/soc/amd/common/block/s3/s3_resum... 
PS3, Line 50: for (i = 0 ; i < 4 ; i++)
> Address Paul's comment? […]

I took it more as a suggestion than a request.  I don't think we have hard and fast rules on separation within a for() statement, and the above is always my preference.  But meh, I don't want to argue over it.