Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38591 )
Change subject: Documentation/vendorcode/eltan: Update security document ......................................................................
Patch Set 9:
(9 comments)
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... File Documentation/vendorcode/eltan/security.md:
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 15: the drop "the"?
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 20: next following
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 21: added to the list? or to CBFS?
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 24: v V
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 26: next following
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 34: The "A"?
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 35: specified item I understand the rest of the text that this means "list itself", or is it really for each item in a list?
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 59: point to location of the public key maybe "point to the public key file"?
https://review.coreboot.org/c/coreboot/+/38591/9/Documentation/vendorcode/el... PS9, Line 96: next following