[coreboot-gerrit] [S] Change in coreboot[main]: mb/google/brox/var/lotso: Tune I2C frequency for 400 kHz

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83290?usp=email )

(

3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )Change subject: mb/google/brox/var/lotso: Tune I2C frequency for 400 kHz
......................................................................

mb/google/brox/var/lotso: Tune I2C frequency for 400 kHz

Before:
I2C0 - 401kHz
I2C4 - 405kHz

After:
I2C0 - 392kHz
I2C4 - 395kHz

HW: Change R8409/R8411 to 33ohm.

BUG=b:349743464,b:349735055
TEST=emerge-brox sys-boot/coreboot
     Test pass by EE

Change-Id: I985837b1b80e973f148529b446905580c0f95e98
Signed-off-by: Jian Tong 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/83290
Reviewed-by: Eric Lai 
Tested-by: build bot (Jenkins) 
Reviewed-by: Kun Liu 
---
M src/mainboard/google/brox/variants/lotso/overridetree.cb
1 file changed, 16 insertions(+), 0 deletions(-)

Approvals:
  Kun Liu: Looks good to me, but someone else must approve
  build bot (Jenkins): Verified
  Eric Lai: Looks good to me, approved

diff --git a/src/mainboard/google/brox/variants/lotso/overridetree.cb b/src/mainboard/google/brox/variants/lotso/overridetree.cb
index 274e22c..187840f 100644
--- a/src/mainboard/google/brox/variants/lotso/overridetree.cb
+++ b/src/mainboard/google/brox/variants/lotso/overridetree.cb
@@ -48,6 +48,22 @@
    register "tcss_ports[0]" = "TCSS_PORT_EMPTY"	# Disable TCP0
    register "tcss_ports[2]" = "TCSS_PORT_EMPTY" 	# Disable TCP2

+	register "common_soc_config" = "{
+		.i2c[0] = {
+			.speed = I2C_SPEED_FAST,
+			.rise_time_ns = 650,
+			.fall_time_ns = 350,
+			.data_hold_time_ns = 400,
+		},
+		.i2c[4] = {
+			.early_init = 1,
+			.speed = I2C_SPEED_FAST,
+			.rise_time_ns = 600,
+			.fall_time_ns = 110,
+			.data_hold_time_ns = 100,
+		},
+	}"
+
    register "serial_io_gspi_mode" = "{
    	[PchSerialIoIndexGSPI0] = PchSerialIoPci,
    	[PchSerialIoIndexGSPI1] = PchSerialIoPci,