Attention is currently required from: Martin L Roth, Werner Zeh, Nicholas Chin. Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64846 )
Change subject: Documentation: Update coreboot 4.17 release notes ......................................................................
Patch Set 3: Code-Review+2
(5 comments)
File Documentation/releases/coreboot-4.17-relnotes.md:
https://review.coreboot.org/c/coreboot/+/64846/comment/19d52ea9_f3306b47 PS3, Line 11: successful Add a dot.
https://review.coreboot.org/c/coreboot/+/64846/comment/c959f076_17306e12 PS3, Line 122: nit: Add second empty line
https://review.coreboot.org/c/coreboot/+/64846/comment/c252712d_8b6749ea PS3, Line 141: (because it is part of another firmware section (IFWI or a different : CBFS), the CRTM measurement fails.
A closing bracket is missing here.
I would replace the outer brackets with commas.
https://review.coreboot.org/c/coreboot/+/64846/comment/068ddc75_cc65b59b PS3, Line 246: nit: Add second empty line
https://review.coreboot.org/c/coreboot/+/64846/comment/2b5af0af_36c8b38c PS3, Line 285: linux nit: Uppercase L