Change in coreboot[master]: mb/asrock/h110m: Explain why some SATA ports are empty

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/39119 )

Change subject: mb/asrock/h110m: Explain why some SATA ports are empty
......................................................................

mb/asrock/h110m: Explain why some SATA ports are empty

Change-Id: Ib0a24fab22ee082367b82b3e8ee7383f1f02a4ad
Signed-off-by: Maxim Polyakov 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39119
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
---
M src/mainboard/asrock/h110m/devicetree.cb
1 file changed, 5 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/mainboard/asrock/h110m/devicetree.cb b/src/mainboard/asrock/h110m/devicetree.cb
index fa94dd9..d42d91e 100644
--- a/src/mainboard/asrock/h110m/devicetree.cb
+++ b/src/mainboard/asrock/h110m/devicetree.cb
@@ -180,6 +180,11 @@
    # SATA
    register "EnableSata" = "1"
    register "SataSalpSupport" = "1"
+	# 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.
    register "SataPortsEnable" = "{ \
    	[0]     = 1, \
    	[1]     = 1, \
@@ -190,8 +195,6 @@
    	[6]     = 0, \
    	[7]     = 0, \
    }"
-	# SATA4 and SATA5 are located in the lower right corner
-	# of the board, but there is no connector for this

# PCH UART, SPI, I2C
    register "SerialIoDevMode" = "{ \