Attention is currently required from: Paul Menzel.
Maximilian Brune has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71665 )
Change subject: Documentation/acronyms: Add several acronyms ......................................................................
Patch Set 2:
(6 comments)
This change is ready for review.
Commit Message:
https://review.coreboot.org/c/coreboot/+/71665/comment/e9d54502_2df6b04b PS1, Line 7: Documentation/acronyms: Add acronyms
Add several acronyms
Done
File Documentation/acronyms.md:
https://review.coreboot.org/c/coreboot/+/71665/comment/100a88e6_ae48d560 PS1, Line 250: DMI - Direct Media Interface is a link/bus between CPU and PCH
Missing bullet?
Done
https://review.coreboot.org/c/coreboot/+/71665/comment/763f7a5c_da21cc05 PS1, Line 623: and Range Register**](https://www.kernel.org/doc/html/v5.19/x86/pat.html)
Maybe make that a separate commit. It’d be great if you started sentences with a capital letter.
Done
https://review.coreboot.org/c/coreboot/+/71665/comment/7e3f91e5_ddf63333 PS1, Line 678: set's
sets
Done
https://review.coreboot.org/c/coreboot/+/71665/comment/d6ca7b75_ae79307c PS1, Line 678: physicall
physical
Done
https://review.coreboot.org/c/coreboot/+/71665/comment/10ff25c7_a4f58d2d PS1, Line 678: set's
sets
Done
-
Maximilian Brune (Code Review)