Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38713 )
Change subject: Documentation/soc/amd/family17: Update to match current design ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/38713/1/Documentation/soc/amd/famil... File Documentation/soc/amd/family17h.md:
https://review.coreboot.org/c/coreboot/+/38713/1/Documentation/soc/amd/famil... PS1, Line 57: AP AP and not BSP? I mostly know that acronym from SMP system, where the first processor is the BootStrap Processor and the other ones the Application Processor. If the PSP is considered the bootstrap processor in the system, it should probably be mentioned that the acronyms have a slightly differently meaning here
https://review.coreboot.org/c/coreboot/+/38713/1/Documentation/soc/amd/famil... PS1, Line 190: enabled "already available" instead of "enabled"?
https://review.coreboot.org/c/coreboot/+/38713/1/Documentation/soc/amd/famil... PS1, Line 250: 5. [https://www.intel.com/content/www/us/en/intelligent-systems/intel-firmware-s...) please keep the newline at the end