Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/58238 )
Change subject: mb/google/dedede/var/corori: Configure I2C times for Touchpad ......................................................................
mb/google/dedede/var/corori: Configure I2C times for Touchpad
Configure I2C high / low time in the device tree to ensure Touchpad I2C CLK runs accurately between 380 kHz and 400 kHz.
Measured I2C frequency just as below after tuning: Touch Pad CLK: 389.2 KHz
BUG=b:202787528 TEST=Build and check after tuning I2C clock is between 380 kHz and 400 kHz
Change-Id: I0f9d062fc611de0062a39849aee1174268391682 Signed-off-by: Ian Feng ian_feng@compal.corp-partner.google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/58238 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Henry Sun henrysun@google.com Reviewed-by: Karthik Ramasubramanian kramasub@google.com Reviewed-by: Paul Menzel paulepanter@mailbox.org --- M src/mainboard/google/dedede/variants/corori/overridetree.cb 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Karthik Ramasubramanian: Looks good to me, approved Henry Sun: Looks good to me, but someone else must approve
diff --git a/src/mainboard/google/dedede/variants/corori/overridetree.cb b/src/mainboard/google/dedede/variants/corori/overridetree.cb index d322000..f7899a6 100644 --- a/src/mainboard/google/dedede/variants/corori/overridetree.cb +++ b/src/mainboard/google/dedede/variants/corori/overridetree.cb @@ -39,6 +39,8 @@ register "common_soc_config" = "{ .i2c[0] = { .speed = I2C_SPEED_FAST, + .rise_time_ns = 255, + .fall_time_ns = 14, }, .i2c[4] = { .speed = I2C_SPEED_FAST,