Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58886 )
Change subject: Documentation: Some notes about how to integrate FSP ......................................................................
Patch Set 1: Code-Review+1
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58886/comment/c6084fba_769133b5 PS1, Line 7: Documentation: Some notes about how to integrate FSP Nit: Please add verb: Add some notes …
File Documentation/soc/intel/fsp/index.md:
https://review.coreboot.org/c/coreboot/+/58886/comment/541fd127_b62a3abb PS1, Line 8: has an effect on affects
https://review.coreboot.org/c/coreboot/+/58886/comment/43ee9665_e64ad98e PS1, Line 11: * It should be possible to replace FSP based boot with a native : coreboot implementation for a given chipset without touching the : mainboard code. : * The devicetree configures coreboot and part of what coreboot does : with the information is setting some FSP UPDs. The devicetree isn't : supposed to directly configure FSP. Reflow for full textwidth?