Attention is currently required from: Felix Singer, Nicholas Chin. Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64846 )
Change subject: Documentation: Update coreboot 4.17 release notes ......................................................................
Patch Set 2:
(6 comments)
File Documentation/releases/coreboot-4.17-relnotes.md:
https://review.coreboot.org/c/coreboot/+/64846/comment/e48e6a65_66f60261 PS2, Line 45: particularly : commits interesting
particularly interesting commits
Done
https://review.coreboot.org/c/coreboot/+/64846/comment/e8c29a1d_91ec2b9c PS2, Line 54: cbmeme
cbmem?
Done
https://review.coreboot.org/c/coreboot/+/64846/comment/4eebe17a_1152a03c PS2, Line 54: would : initializing
would be initializing
Done
https://review.coreboot.org/c/coreboot/+/64846/comment/d530b237_de512bc1 PS2, Line 57: This reason
The reason
Done
https://review.coreboot.org/c/coreboot/+/64846/comment/8adea724_e3b59084 PS2, Line 65: * Add new coreDOOM payload, a port of Doom to libpayload
This line seems a bit redundant now that the full commit message is included below. […]
I moved this to the last line and changed it to say "* Add new coreDOOM payload. See commit message below."
https://review.coreboot.org/c/coreboot/+/64846/comment/a7e5848c_6f39aa07 PS2, Line 67: the
Remove "the". I probably missed this when I originally wrote that line in a previous patch.
Done
-
Martin L Roth (Code Review)