[coreboot-gerrit] Change in coreboot[master]: Documentation/binaries: Add AMD FSP documentation

[Marshall Dawson (Code Review)]

Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34662 )

Change subject: Documentation/binaries: Add AMD FSP documentation
......................................................................

Patch Set 15:

(3 comments)

https://review.coreboot.org/c/coreboot/+/34662/15/Documentation/binaries/amd... 
File Documentation/binaries/amd/AMD_FSP_family_17h.md:

https://review.coreboot.org/c/coreboot/+/34662/15/Documentation/binaries/amd... 
PS15, Line 17: AGESA
> Originally I was told to not enter in details... blame AGESA instead of HW. I would prefer your way. […]

I disagree with a position of placing "blame" on AGESA.  What I'm asking for is already public:  https://doc.coreboot.org/soc/amd/family17h.html?highlight=family#problem-sta...

https://review.coreboot.org/c/coreboot/+/34662/15/Documentation/binaries/amd... 
PS15, Line 18: CAR
Add a period to match other bullet items.

https://review.coreboot.org/c/coreboot/+/34662/15/Documentation/binaries/amd... 
PS15, Line 47: TBD
Replace with "will be documented separately for each project using the FSP framework".  IIRC Intel calls these Integration Guides but they're separate from the EAS.