build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68213 )
Change subject: nb/intel/i945/raminit: Use 'size_t' for banksize[] ......................................................................
Patch Set 2:
(2 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159980): https://review.coreboot.org/c/coreboot/+/68213/comment/1a553c59_d52ae8ae PS2, Line 10: Change-Id: I4fb845bb4145d47aea39d7e5493d854d00e289aa Duplicate signature
File src/northbridge/intel/i945/raminit.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159980): https://review.coreboot.org/c/coreboot/+/68213/comment/b3efe265_5a16f7b3 PS2, Line 428: printk(BIOS_DEBUG, "DIMM %d side 0 = %ld MB\n", i, sysinfo->banksize[i * 2] * 32); line length of 98 exceeds 96 columns