Attention is currently required from: Angel Pons, David Hendricks, Jakub Czapiga, Martin L Roth, Matt DeVillier, Nicholas Chin, Paul Menzel.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79441?usp=email )
Change subject: Documentation/getting_started: Add a FAQ document ......................................................................
Patch Set 5:
(11 comments)
File Documentation/getting_started/faq.md:
https://review.coreboot.org/c/coreboot/+/79441/comment/b14d90fe_e8cef063 : PS4, Line 30: This should lead to fewer : security issues.
nit: I feel the "should" weakens the statement unnecessarily. […]
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/88fd9f4d_f2f09d24 : PS4, Line 43: system then
In between there is a comma or "and" missing.
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/9a89b640_08b33b6c : PS4, Line 48:
nit: I would add something like: […]
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/f35ff92c_460b1165 : PS4, Line 64: coreboot uses EDK2 : as a payload.
"Uses" sounds too specific and limited to that. […]
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/a3c4e0a6_a42dfc71 : PS4, Line 75: (to date) the most
Well, remove "very"
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/9974866c_936a785f : PS4, Line 116: Thinkpads
nit, typo: `ThinkPads`
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/678335f5_eba06f50 : PS4, Line 142: Thinkpads
nit, typo: `ThinkPads`
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/e7a25bad_d63e95b3 : PS4, Line 148: coreboot
I wouldn't limit that to coreboot. I would just write `firmware`.
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/83210351_135f26f8 : PS4, Line 155: chromebooks
nit, typo: uppercase. See two words before.
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/2567f4a7_f62846b2 : PS4, Line 212: You can run : `intelmetool -b` to see if Boot Guard is enabled (although : intelmetool can fail because it wants to probe the ME beforehand).
s/relies/depends
Done
https://review.coreboot.org/c/coreboot/+/79441/comment/0ec4699e_f41c944a : PS4, Line 237: vendors
nit: Missing period.
Done