Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35547 )
Change subject: mb/supermicro/x11-lga1151-series: completely rework documentation ......................................................................
Patch Set 7:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35547/7/Documentation/mainboard/sup... File Documentation/mainboard/supermicro/x11-lga1151-series/x11-lga1151-series.md:
https://review.coreboot.org/c/coreboot/+/35547/7/Documentation/mainboard/sup... PS7, Line 1: # X11 LGA1151 series must be referenced from existing *.md
https://review.coreboot.org/c/coreboot/+/35547/7/Documentation/mainboard/sup... File Documentation/mainboard/supermicro/x11-lga1151-series/x11ssh-tf/x11ssh-tf.md:
https://review.coreboot.org/c/coreboot/+/35547/7/Documentation/mainboard/sup... PS7, Line 71: [flashing tutorial]: ../../../../flash_tutorial/ext_power.md why was it dropped?
https://review.coreboot.org/c/coreboot/+/35547/7/Documentation/mainboard/sup... PS7, Line 52: | Ethernet | 2x Intel® X550 10GBase-T Ethernet | what's the benefit to developers? The device topology can be determined by looking at the devicetree.cb.