Attention is currently required from: Ravi Kumar Bokka, Paul Menzel. Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63407 )
Change subject: commonlib/bsd: Add mem_chip_info_size() function ......................................................................
Patch Set 4: Code-Review+2
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63407/comment/3fcff0b8_7ddfab06 PS2, Line 8:
updated commit message with detail information
Please remember to mark comments as resolved when you resolve them.
Commit Message:
https://review.coreboot.org/c/coreboot/+/63407/comment/157b187d_2ecc1454 PS4, Line 9: strcuture typo
https://review.coreboot.org/c/coreboot/+/63407/comment/42d981cf_e38ebd1e PS4, Line 10: using "used"
https://review.coreboot.org/c/coreboot/+/63407/comment/9f441b52_ae73cee9 PS4, Line 13: and commit sha: 4573ca42e675fb8b61bcf05591d8d4d05f93944d No, it doesn't. The next patch does that.
-
Julius Werner (Code Review)