Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31603 )
Change subject: mainboard: Add ASRock H110M-DVS ......................................................................
Patch Set 13:
(4 comments)
https://review.coreboot.org/#/c/31603/12/Documentation/mainboard/asrock/h110... File Documentation/mainboard/asrock/h110m-dvs.md:
https://review.coreboot.org/#/c/31603/12/Documentation/mainboard/asrock/h110... PS12, Line 8: Intel company provides [Firmware Support Package (2.0)](Documentation/soc/intel/fsp/index.md)
../.. […]
Done
https://review.coreboot.org/#/c/31603/12/Documentation/mainboard/asrock/h110... PS12, Line 10: [document](Documentation/soc/intel/code_development_model/code_development_model.md).
../.. […]
Done
https://review.coreboot.org/#/c/31603/12/Documentation/mainboard/asrock/h110... PS12, Line 30: ```eval_rst
that's not RST code. […]
Done
https://review.coreboot.org/#/c/31603/12/Documentation/mainboard/asrock/h110... PS12, Line 31: Peg0Enable = Enable
no need for whitespace at beginning of line
Done