Bill XIE has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39969 )
Change subject: nb/intel/sandybridge: Read SPDs only once if measured boot is enabled ......................................................................
Patch Set 3:
(7 comments)
https://review.coreboot.org/c/coreboot/+/39969/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39969/2//COMMIT_MSG@7 PS2, Line 7: spds
SPDs
Done
https://review.coreboot.org/c/coreboot/+/39969/2//COMMIT_MSG@9 PS2, Line 9: spd
SPD
Done
https://review.coreboot.org/c/coreboot/+/39969/2//COMMIT_MSG@10 PS2, Line 10: condition
conditions
Done
https://review.coreboot.org/c/coreboot/+/39969/2//COMMIT_MSG@10 PS2, Line 10: spd
SPD
Done
https://review.coreboot.org/c/coreboot/+/39969/2//COMMIT_MSG@13 PS2, Line 13: spd
SPD
Done
https://review.coreboot.org/c/coreboot/+/39969/2//COMMIT_MSG@13 PS2, Line 13: CB:39906
Do you mean the commit or review comments? Please add the URL for people using `git log`.
Done
https://review.coreboot.org/c/coreboot/+/39969/2/src/northbridge/intel/sandy... File src/northbridge/intel/sandybridge/raminit.c:
https://review.coreboot.org/c/coreboot/+/39969/2/src/northbridge/intel/sandy... PS2, Line 107: sum += spds[i][j];
Can’t you just return, if one element is greater 0, that means the first time you find that element?
Done