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

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34662 )

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

Patch Set 12:

(1 comment)
> Patch Set 11:
>
> (1 comment)

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

https://review.coreboot.org/c/coreboot/+/34662/11/Documentation/binaries/amd... 
PS11, Line 23:     already enabled.
> Ack

Just make a clear distinction, I think start of romstage would be a natural boundary where FSP documentation begins. Any details you write here about bootblock or PSP directory becomes redundant (and possibly out-of-sync) once the situation about PSP documentation improves.

I'd be happy if you just stated "For these platforms, romstage executes from DRAM. Phases prior to romstage are described in XXXX.md" and remove the bootblock details here.