Change in coreboot[master]: mb/google/poppy/variants/nautilus: set I2C speed to 400KHz

[Chris Wang (Code Review)]

Hello Chris Wang,

I'd like you to do a code review. Please visit

https://review.coreboot.org/22589

to review the following change.

Change subject: mb/google/poppy/variants/nautilus: set I2C speed to 400KHz
......................................................................

mb/google/poppy/variants/nautilus: set I2C speed to 400KHz

Add "speed_config" for each I2C port configuration to set speed to
400KHz.

BRANCH=master
BUG=none
TEST=compiled/verified

Change-Id: Icb48733b87cefc92577547b1eab661a8cbb12be6
Signed-off-by: Chris Wang 
---
M src/mainboard/google/poppy/variants/nautilus/devicetree.cb
1 file changed, 53 insertions(+), 8 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/22589/1

diff --git a/src/mainboard/google/poppy/variants/nautilus/devicetree.cb b/src/mainboard/google/poppy/variants/nautilus/devicetree.cb
index 6ee7a52..83feda5 100644
--- a/src/mainboard/google/poppy/variants/nautilus/devicetree.cb
+++ b/src/mainboard/google/poppy/variants/nautilus/devicetree.cb
@@ -78,6 +78,7 @@
    register "SlowSlewRateForIa" = "3"      # Fast/16
    register "SlowSlewRateForGt" = "3"      # Fast/16
    register "SlowSlewRateForSa" = "3"      # Fast/16
+
    # VR Settings Configuration for 4 Domains
    #+----------------+-------+-------+-------+-------+
    #| Domain/Setting |  SA   |  IA   | GTUS  |  GTS  |
@@ -175,6 +176,15 @@

# Touchscreen
    register "i2c_voltage[0]" = "I2C_VOLTAGE_3V3"
+	register "i2c[0]" = "{
+		.speed = I2C_SPEED_FAST,
+		.speed_config[0] = {
+			.speed = I2C_SPEED_FAST,
+			.scl_lcnt = 190,
+			.scl_hcnt = 100,
+			.sda_hold = 36,
+		},
+	}"

# H1
    register "i2c_voltage[1]" = "I2C_VOLTAGE_3V3"
@@ -182,20 +192,62 @@
    # for TPM communication before memory is up.
    register "i2c[1]" = "{
    	.early_init = 1,
+		.speed = I2C_SPEED_FAST,
+		.speed_config[0] = {
+			.speed = I2C_SPEED_FAST,
+			.scl_lcnt = 190,
+			.scl_hcnt = 100,
+			.sda_hold = 36,
+		},
    }"

# Trackpad
    register "i2c_voltage[2]" = "I2C_VOLTAGE_1V8"
+	register "i2c[2]" = "{
+		.speed = I2C_SPEED_FAST,
+		.speed_config[0] = {
+			.speed = I2C_SPEED_FAST,
+			.scl_lcnt = 190,
+			.scl_hcnt = 100,
+			.sda_hold = 36,
+		},
+	}"

# Pen
    register "i2c_voltage[3]" = "I2C_VOLTAGE_1V8"
+	register "i2c[3]" = "{
+		.speed = I2C_SPEED_FAST,
+		.speed_config[0] = {
+			.speed = I2C_SPEED_FAST,
+			.scl_lcnt = 190,
+			.scl_hcnt = 100,
+			.sda_hold = 36,
+		},
+	}"

# Camera
    register "i2c_voltage[4]" = "I2C_VOLTAGE_1V8"
+	register "i2c[4]" = "{
+		.speed = I2C_SPEED_FAST,
+		.speed_config[0] = {
+			.speed = I2C_SPEED_FAST,
+			.scl_lcnt = 190,
+			.scl_hcnt = 100,
+			.sda_hold = 36,
+		},
+	}"

# Audio
    register "i2c_voltage[5]" = "I2C_VOLTAGE_1V8"
-
+	register "i2c[5]" = "{
+		.speed = I2C_SPEED_FAST,
+		.speed_config[0] = {
+			.speed = I2C_SPEED_FAST,
+			.scl_lcnt = 190,
+			.scl_hcnt = 100,
+			.sda_hold = 36,
+		},
+	}"

# Must leave UART0 enabled or SD/eMMC will not work as PCI
    register "SerialIoDevMode" = "{
@@ -210,13 +262,6 @@
    	[PchSerialIoIndexUart0] = PchSerialIoPci,
    	[PchSerialIoIndexUart1] = PchSerialIoDisabled,
    	[PchSerialIoIndexUart2] = PchSerialIoSkipInit,
-	}"
-
-	# Enable I2C5 for audio codec at 400kHz
-		register "i2c[5]" = "{
-		.speed = I2C_SPEED_FAST,
-		.rise_time_ns = 104,
-		.fall_time_ns = 52,
    }"

register "speed_shift_enable" = "1"