Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31949 )
Change subject: Doc/mb/asrock/h110m: update info about PEG ......................................................................
Patch Set 5:
(2 comments)
https://review.coreboot.org/#/c/31949/3/Documentation/mainboard/asrock/h110m... File Documentation/mainboard/asrock/h110m-dvs.md:
https://review.coreboot.org/#/c/31949/3/Documentation/mainboard/asrock/h110m... PS3, Line 26: You must prepare the FSP for integration into the coreboot image:
as patching is no longer necessary, can all the preparation steps be removed?
I removed it
https://review.coreboot.org/#/c/31949/3/Documentation/mainboard/asrock/h110m... PS3, Line 29: # split FSP.fd: Fsp_M.fd and Fsp_S.fd should be used in building the image
isn't it automatically split by the build process?
Yes, you are right. If I use CONFIG_FSP_USE_REPO, then the build system split fsp and adds it to the image. I removed it