Attention is currently required from: Nico Huber. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61682 )
Change subject: nb/amd/common: Move RAM calcualtion to common code ......................................................................
Patch Set 1:
(4 comments)
File src/northbridge/amd/common/ram_calc.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-140189): https://review.coreboot.org/c/coreboot/+/61682/comment/6fe2f82a_e28951dd PS1, Line 25: /* [N-1:24] at [N-1-8:16], if N < 40 addres_mask will clear unused high bits */ 'addres' may be misspelled - perhaps 'address'?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-140189): https://review.coreboot.org/c/coreboot/+/61682/comment/a9777811_58c8b1f8 PS1, Line 29: /* [N-1:40] at [N-1-40:0], addres_mask will clear unused high bits */ 'addres' may be misspelled - perhaps 'address'?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-140189): https://review.coreboot.org/c/coreboot/+/61682/comment/890e6413_565fdcb2 PS1, Line 40: /* [N-1:24] at [N-1-8:16], if N < 40 addres_mask will clear unused high bits */ 'addres' may be misspelled - perhaps 'address'?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-140189): https://review.coreboot.org/c/coreboot/+/61682/comment/8713f7db_81530f59 PS1, Line 45: /* [N-1:40] at [N-1-40:0] addres_mask will clear unused high bits */ 'addres' may be misspelled - perhaps 'address'?