[coreboot-gerrit] Change in coreboot[master]: [RFC] Documentation/fsp: Discuss FSP-S issues

[Marshall Dawson (Code Review)]

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

Change subject: [RFC] Documentation/fsp: Discuss FSP-S issues
......................................................................

Patch Set 5:

Since this primarily seems Intel-centric, I'll offer an Ack instead of a code-review score.  Nice writeup though.

I don't want to risk sounding like I'm taking shots at a competitor.  Having been an FSP consumer for years, I'm very sensitive to the challenges the coreboot community and customers face.  Ironically, now back at AMD and trying to package reference code again, I have some newfound sympathy for Intel engineers that needs to support their feature sets.