[coreboot-gerrit] Change in coreboot[master]: mb/lenovo/s230u: fix sata port map for the msata port

Arthur Heymans (Code Review) gerrit at coreboot.org
Wed May 10 11:22:54 CEST 2017


Arthur Heymans has submitted this change and it was merged. ( https://review.coreboot.org/19523 )

Change subject: mb/lenovo/s230u: fix sata port map for the msata port
......................................................................


mb/lenovo/s230u: fix sata port map for the msata port

s230u seems only have two sata ports: one for the 2.5in hdd and one for
msata. map 0x11 (port 0 & 4) enables hdd but not msata, and map 0x5
(port 0 & 2) enables both.

Change-Id: I1e9e96f0d0849b1e8c4e02aa4f686ceb5e10b3ab
Signed-off-by: Bill XIE <persmule at gmail.com>
Reviewed-on: https://review.coreboot.org/19523
Tested-by: build bot (Jenkins) <no-reply at coreboot.org>
Reviewed-by: Alexander Couzens <lynxis at fe80.eu>
---
M src/mainboard/lenovo/s230u/devicetree.cb
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Alexander Couzens: Looks good to me, approved
  build bot (Jenkins): Verified



diff --git a/src/mainboard/lenovo/s230u/devicetree.cb b/src/mainboard/lenovo/s230u/devicetree.cb
index ff4c6c4..b9b38dd 100644
--- a/src/mainboard/lenovo/s230u/devicetree.cb
+++ b/src/mainboard/lenovo/s230u/devicetree.cb
@@ -52,7 +52,7 @@
 			register "pcie_hotplug_map" = "{ 0, 1, 0, 1, 0, 0, 0, 0 }"
 			register "pcie_port_coalesce" = "1"
 			register "sata_interface_speed_support" = "0x3"
-			register "sata_port_map" = "0x11"
+			register "sata_port_map" = "0x5"
 			register "superspeed_capable_ports" = "0x0000000f"
 			register "xhci_overcurrent_mapping" = "0x00000c03"
 			register "xhci_switchable_ports" = "0x0000000f"

-- 
To view, visit https://review.coreboot.org/19523
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1e9e96f0d0849b1e8c4e02aa4f686ceb5e10b3ab
Gerrit-PatchSet: 12
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Bill XIE <persmule at gmail.com>
Gerrit-Reviewer: Alexander Couzens <lynxis at fe80.eu>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>



More information about the coreboot-gerrit mailing list