build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/22215 )
Change subject: [WIP]nb/intel/sandybridge/raminit: Add ECC support ......................................................................
Patch Set 10:
(1 comment)
https://review.coreboot.org/c/coreboot/+/22215/10/src/northbridge/intel/sand... File src/northbridge/intel/sandybridge/raminit_common.c:
https://review.coreboot.org/c/coreboot/+/22215/10/src/northbridge/intel/sand... PS10, Line 3135: for (row = 0; row < rowsize; row += 16) { suspect code indent for conditional statements (24, 16)