Attention is currently required from: Nicholas Chin.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67330 )
Change subject: Documentation: Add some more acronyms to the list. ......................................................................
Patch Set 1:
(7 comments)
Patchset:
PS1: Thank
File Documentation/acronyms.md:
https://review.coreboot.org/c/coreboot/+/67330/comment/dbdd7cee_aabeac68 PS1, Line 50: ]
Should be a * to match opening double asterisks
Thanks! definitely appreciate the thorough review.
Done.
https://review.coreboot.org/c/coreboot/+/67330/comment/fcf88643_b2fc3a54 PS1, Line 61: in processor chip
Seems like this is missing a word, perhaps "in a processor chip" or "in the processor chip"
Done
https://review.coreboot.org/c/coreboot/+/67330/comment/88c537a0_5e53e977 PS1, Line 305: EDP
eDP
Done
https://review.coreboot.org/c/coreboot/+/67330/comment/62e1cbb7_6168b459 PS1, Line 603: This has been : generally been : https://en.wikipedia.org/wiki/MultiProcessor_Specification by the ACPI : tables.
Noticed this when skimming the rendered markdown. […]
I'm going with `superseded`.
https://review.coreboot.org/c/coreboot/+/67330/comment/ef4d50c3_70d65bc9 PS1, Line 647: [Original Design Manufacturer]
Nit: Add double asterisks (bold) for consistency with other external links in this list
Done
https://review.coreboot.org/c/coreboot/+/67330/comment/5632638f_2356b9b2 PS1, Line 948: TME - Intel: Total Memory Encryption
Should be moved a line down as it breaks the TLB hyperlink
Done