[coreboot-gerrit] Change in coreboot[master]: mainboard/google/coral: Add USB2 phy setting override for Sante/Lava/...

[Pan Sheng-Liang (Code Review)]

Pan Sheng-Liang has uploaded this change for review. ( https://review.coreboot.org/23880


Change subject: mainboard/google/coral: Add USB2 phy setting override for Sante/Lava/Blue/Bruce/Astronaut
......................................................................

mainboard/google/coral: Add USB2 phy setting override for Sante/Lava/Blue/Bruce/Astronaut

Due to there are some chances USB devices can not be detected.
USB2 port#1 and #4 PHY register need to be overridden.

port#1:
PERPORTPETXISET = 4
PERPORTTXISET = 4
IUSBTXEMPHASISEN= 1
PERPORTTXPEHALF= 0

port#4:
PERPORTPETXISET = 7
PERPORTTXISET = 7
IUSBTXEMPHASISEN= 1
PERPORTTXPEHALF= 0

BUG=b:72623892
BRANCH=master
TEST=emerge-coral coreboot chromeos-bootimage

Change-Id: I045c14ef87a5920666dc06c2b53f0fed0b702d0e
Signed-off-by: Pan Sheng-Liang <Sheng-Liang.Pan at quantatw.com>
---
M src/mainboard/google/reef/variants/coral/mainboard.c
1 file changed, 68 insertions(+), 14 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/23880/1

diff --git a/src/mainboard/google/reef/variants/coral/mainboard.c b/src/mainboard/google/reef/variants/coral/mainboard.c
index 853276e..97c4c0e 100644
--- a/src/mainboard/google/reef/variants/coral/mainboard.c
+++ b/src/mainboard/google/reef/variants/coral/mainboard.c
@@ -21,12 +21,21 @@
 #include <soc/intel/common/vbt.h>
 
 enum {
-        SKU_0_ASTRONAUT = 0,
-        SKU_1_ASTRONAUT = 1,
+	SKU_0_ASTRONAUT = 0,
+	SKU_1_ASTRONAUT = 1,
 	SKU_2_SANTA = 2,
 	SKU_3_SANTA = 3,
-        SKU_61_ASTRONAUT = 61,
-        SKU_62_ASTRONAUT = 62,
+	SKU_4_LAVA = 4,
+	SKU_5_LAVA = 5,
+	SKU_6_BLUE = 6,
+	SKU_7_BLUE = 7,
+	SKU_8_BRUCE = 8,
+	SKU_9_LAVA = 9,
+	SKU_10_LAVA = 10,
+	SKU_11_BRUCE = 11,
+	SKU_12_BLUE = 12,
+	SKU_61_ASTRONAUT = 61,
+	SKU_62_ASTRONAUT = 62,
 	SKU_160_NASHER = 160,
 	SKU_161_NASHER = 161,
 	SKU_162_NASHER = 162,
@@ -66,18 +75,63 @@
 	switch (sku_id) {
         case SKU_0_ASTRONAUT:
         case SKU_1_ASTRONAUT:
-		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 7;
-		cfg->usb2eye[1].Usb20PerPortTxiSet = 2;
-		break;
-	case SKU_2_SANTA:
-	case SKU_3_SANTA:
-		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 7;
-		cfg->usb2eye[1].Usb20PerPortTxiSet = 2;
-		break;
         case SKU_61_ASTRONAUT:
         case SKU_62_ASTRONAUT:
-		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 7;
-		cfg->usb2eye[1].Usb20PerPortTxiSet = 5;
+		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 4;
+		cfg->usb2eye[1].Usb20PerPortTxiSet = 4;
+		cfg->usb2eye[1].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[1].Usb20PerPortTxPeHalf = 0;
+		cfg->usb2eye[4].Usb20PerPortPeTxiSet = 7;
+		cfg->usb2eye[4].Usb20PerPortTxiSet = 7;
+		cfg->usb2eye[4].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[4].Usb20PerPortTxPeHalf = 0;
+                break;
+	case SKU_2_SANTA:
+	case SKU_3_SANTA:
+		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 4;
+		cfg->usb2eye[1].Usb20PerPortTxiSet = 4;
+		cfg->usb2eye[1].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[1].Usb20PerPortTxPeHalf = 0;
+		cfg->usb2eye[4].Usb20PerPortPeTxiSet = 7;
+		cfg->usb2eye[4].Usb20PerPortTxiSet = 7;
+		cfg->usb2eye[4].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[4].Usb20PerPortTxPeHalf = 0;
+                break;
+	case SKU_4_LAVA:
+	case SKU_5_LAVA:
+	case SKU_9_LAVA:
+	case SKU_10_LAVA:
+		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 4;
+		cfg->usb2eye[1].Usb20PerPortTxiSet = 4;
+		cfg->usb2eye[1].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[1].Usb20PerPortTxPeHalf = 0;
+		cfg->usb2eye[4].Usb20PerPortPeTxiSet = 7;
+		cfg->usb2eye[4].Usb20PerPortTxiSet = 7;
+		cfg->usb2eye[4].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[4].Usb20PerPortTxPeHalf = 0;
+                break;
+	case SKU_6_BLUE:
+	case SKU_7_BLUE:
+	case SKU_12_BLUE:
+		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 4;
+		cfg->usb2eye[1].Usb20PerPortTxiSet = 4;
+		cfg->usb2eye[1].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[1].Usb20PerPortTxPeHalf = 0;
+		cfg->usb2eye[4].Usb20PerPortPeTxiSet = 7;
+		cfg->usb2eye[4].Usb20PerPortTxiSet = 7;
+		cfg->usb2eye[4].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[4].Usb20PerPortTxPeHalf = 0;
+                break;
+	case SKU_8_BRUCE:
+	case SKU_11_BRUCE:
+		cfg->usb2eye[1].Usb20PerPortPeTxiSet = 4;
+		cfg->usb2eye[1].Usb20PerPortTxiSet = 4;
+		cfg->usb2eye[1].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[1].Usb20PerPortTxPeHalf = 0;
+		cfg->usb2eye[4].Usb20PerPortPeTxiSet = 7;
+		cfg->usb2eye[4].Usb20PerPortTxiSet = 7;
+		cfg->usb2eye[4].Usb20IUsbTxEmphasisEn = 1;
+		cfg->usb2eye[4].Usb20PerPortTxPeHalf = 0;
                 break;
 	default:
 		break;

-- 
To view, visit https://review.coreboot.org/23880
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I045c14ef87a5920666dc06c2b53f0fed0b702d0e
Gerrit-Change-Number: 23880
Gerrit-PatchSet: 1
Gerrit-Owner: Pan Sheng-Liang <sl.pan.quantw at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180227/5c36a525/attachment-0001.html>