Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31603 )
Change subject: mainboard: Add ASRock H110M-DVS ......................................................................
Patch Set 12:
(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) ../../Documentation/
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). ../../Documentation/
https://review.coreboot.org/#/c/31603/12/Documentation/mainboard/asrock/h110... PS12, Line 30: ```eval_rst that's not RST code. Use ``` for block quote
https://review.coreboot.org/#/c/31603/12/Documentation/mainboard/asrock/h110... PS12, Line 31: Peg0Enable = Enable no need for whitespace at beginning of line