build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68842 )
Change subject: lib/ramdetect: Limit probe size to function argument ......................................................................
Patch Set 1:
(2 comments)
File src/lib/ramdetect.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161263): https://review.coreboot.org/c/coreboot/+/68842/comment/52babb0e_af7936b2 PS1, Line 61: if ((discovered | (1ULL << i)) > probe_size) suspect code indent for conditional statements (16, 20)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161263): https://review.coreboot.org/c/coreboot/+/68842/comment/13c883c9_fd89a72f PS1, Line 62: continue; Statements should start on a tabstop