Attention is currently required from: Angel Pons.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34833 )
Change subject: nb/intel/gm45: Split DDR2 I/O init out ......................................................................
Patch Set 9:
(2 comments)
File src/northbridge/intel/gm45/raminit.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-166430): https://review.coreboot.org/c/coreboot/+/34833/comment/57e6d34f_ff52d77d PS9, Line 1720: for (s = 1; s < 4; ++s) { braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-166430): https://review.coreboot.org/c/coreboot/+/34833/comment/0702ef69_126ac705 PS9, Line 1729: for (s = 1; s < 4; ++s) { braces {} are not necessary for single statement blocks
-
build bot (Jenkins) (Code Review)