Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39119 )
Change subject: mb/asrock/h110m: Explain why some SATA ports are empty ......................................................................
Patch Set 10:
(2 comments)
Thanks for the review
https://review.coreboot.org/c/coreboot/+/39119/9//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39119/9//COMMIT_MSG@7 PS9, Line 7: mb/asrock/h110m: remove the wrong comment about SATA
With the suggested comment change, this could be: […]
Done
https://review.coreboot.org/c/coreboot/+/39119/9/src/mainboard/asrock/h110m/... File src/mainboard/asrock/h110m/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/39119/9/src/mainboard/asrock/h110m/... PS9, Line 183: Max # of SATA 6.0 Gb/s Ports for H110 - 4
I would prefer if the original comment was extended. […]
Sounds very good. Thanks for the help