Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39119 )
Change subject: mb/asrock/h110m: remove the wrong comment about SATA ......................................................................
Patch Set 9: Code-Review+1
(1 comment)
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. The thing is that SATA4 and SATA5 exist on the mainboard, but H110 only has four ports (so they would not work). This is because the same PCB design is reused to make models with better PCHs. For example:
SATA4 and SATA5 are located in the lower right corner of the board, but they are not populated. This is because the same PCB is used to make boards with better PCHs, which can have up to six SATA ports. However, the H110 PCH only has four SATA ports, which explains why two connectors are missing.
-
Angel Pons (Code Review)