Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/58965 )
Change subject: mb/google/dedede/var/bugzzy: Adjust I2C speed ......................................................................
mb/google/dedede/var/bugzzy: Adjust I2C speed
This change adjusts all I2C speed to lower then 400KHz. The rise_time_ns and fall_time_ns values for each port are capured by a scope.
BUG=None BRANCH=dedede TEST=built and verified adjusted I2C speed < 400KHz
Signed-off-by: Seunghwan Kim sh_.kim@samsung.corp-partner.google.com Change-Id: I9504608dd8d9a5f5a3848ef34691557942c21023 Reviewed-on: https://review.coreboot.org/c/coreboot/+/58965 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Menzel paulepanter@mailbox.org Reviewed-by: Karthik Ramasubramanian kramasub@google.com --- M src/mainboard/google/dedede/variants/bugzzy/overridetree.cb 1 file changed, 6 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
diff --git a/src/mainboard/google/dedede/variants/bugzzy/overridetree.cb b/src/mainboard/google/dedede/variants/bugzzy/overridetree.cb index 947fa47..e1318a8 100644 --- a/src/mainboard/google/dedede/variants/bugzzy/overridetree.cb +++ b/src/mainboard/google/dedede/variants/bugzzy/overridetree.cb @@ -63,12 +63,18 @@ }, .i2c[1] = { .speed = I2C_SPEED_FAST, + .rise_time_ns = 133, + .fall_time_ns = 29, }, .i2c[2] = { .speed = I2C_SPEED_FAST, + .rise_time_ns = 227, + .fall_time_ns = 9, }, .i2c[3] = { .speed = I2C_SPEED_FAST, + .rise_time_ns = 91, + .fall_time_ns = 2, }, .i2c[4] = { .speed = I2C_SPEED_FAST,