build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/29406 )
Change subject: soc/amd/stoneridge: Add DRAM check for s3 ......................................................................
Patch Set 3:
(6 comments)
https://review.coreboot.org/#/c/29406/3/src/soc/amd/stoneyridge/s3test_util.... File src/soc/amd/stoneyridge/s3test_util.c:
https://review.coreboot.org/#/c/29406/3/src/soc/amd/stoneyridge/s3test_util.... PS3, Line 252: wbinvd(); code indent should use tabs where possible
https://review.coreboot.org/#/c/29406/3/src/soc/amd/stoneyridge/s3test_util.... PS3, Line 252: wbinvd(); please, no space before tabs
https://review.coreboot.org/#/c/29406/3/src/soc/amd/stoneyridge/s3test_util.... PS3, Line 252: wbinvd(); please, no spaces at the start of a line
https://review.coreboot.org/#/c/29406/3/src/soc/amd/stoneyridge/s3test_util.... PS3, Line 254: err = 0; code indent should use tabs where possible
https://review.coreboot.org/#/c/29406/3/src/soc/amd/stoneyridge/s3test_util.... PS3, Line 254: err = 0; please, no space before tabs
https://review.coreboot.org/#/c/29406/3/src/soc/amd/stoneyridge/s3test_util.... PS3, Line 254: err = 0; please, no spaces at the start of a line