Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36328 )
Change subject: [RFC, scratch] Documentation/fsp: Start discussion around FSP-S issues ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/36328/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36328/1//COMMIT_MSG@15 PS1, Line 15: Feel free to amend this commit in any way :) I'm very tempted to talk badly about FSP-T integration since it became optional since 2.0, but I suppose it's not a major issue since there is an open source alternative :)
https://review.coreboot.org/c/coreboot/+/36328/1/Documentation/fsp/fsp-s_dis... File Documentation/fsp/fsp-s_discussion.md:
https://review.coreboot.org/c/coreboot/+/36328/1/Documentation/fsp/fsp-s_dis... PS1, Line 102: POSTBOOT_SAI Please explain briefly what this does/tries to do.
https://review.coreboot.org/c/coreboot/+/36328/1/Documentation/fsp/fsp-s_dis... PS1, Line 128: function pointers Could you give some examples (mainly console + AP PPI, I guess) here?