[coreboot-gerrit] Change in coreboot[master]: mb/google/brya/baseboard/nissa: Configure I2C lcnt and hcnt

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63562 )

Change subject: mb/google/brya/baseboard/nissa: Configure I2C lcnt and hcnt
......................................................................

mb/google/brya/baseboard/nissa: Configure I2C lcnt and hcnt

Configure lcnt and hcnt directly to give the required frequency, tHIGH
and tLOW, instead of using rise and fall times. Aim for a frequency of
390 kHz to make sure it doesn't exceed 400 kHz on different boards.

BUG=b:227517802
TEST=Probe the clock line and check that it meets the requirements for
frequency, tHIGH and tLOW.

Change-Id: I4d4f877c1f0cd9aacd3fa152890b7ef82e059f78
Signed-off-by: Reka Norman 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63562
Tested-by: build bot (Jenkins) 
Reviewed-by: Kangheui Won 
Reviewed-by: Eric Lai 
---
M src/mainboard/google/brya/variants/baseboard/nissa/devicetree.cb
1 file changed, 20 insertions(+), 10 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Kangheui Won: Looks good to me, approved
  Eric Lai: Looks good to me, approved

diff --git a/src/mainboard/google/brya/variants/baseboard/nissa/devicetree.cb b/src/mainboard/google/brya/variants/baseboard/nissa/devicetree.cb
index c4c4fb3..dfc0a85 100644
--- a/src/mainboard/google/brya/variants/baseboard/nissa/devicetree.cb
+++ b/src/mainboard/google/brya/variants/baseboard/nissa/devicetree.cb
@@ -74,33 +74,43 @@
    	.i2c[0] = {
    		.early_init = 1,
    		.speed = I2C_SPEED_FAST,
-			.rise_time_ns = 650,
-			.fall_time_ns = 400,
    		.data_hold_time_ns = 50,
+			.speed_config[0] = {
+				.scl_lcnt = 157,
+				.scl_hcnt = 78,
+			}
    	},
    	.i2c[1] = {
    		.speed = I2C_SPEED_FAST,
-			.rise_time_ns = 650,
-			.fall_time_ns = 400,
    		.data_hold_time_ns = 50,
+			.speed_config[0] = {
+				.scl_lcnt = 157,
+				.scl_hcnt = 78,
+			}
    	},
    	.i2c[2] = {
    		.speed = I2C_SPEED_FAST,
-			.rise_time_ns = 650,
-			.fall_time_ns = 400,
    		.data_hold_time_ns = 50,
+			.speed_config[0] = {
+				.scl_lcnt = 157,
+				.scl_hcnt = 78,
+			}
    	},
    	.i2c[3] = {
    		.speed = I2C_SPEED_FAST,
-			.rise_time_ns = 650,
-			.fall_time_ns = 400,
    		.data_hold_time_ns = 50,
+			.speed_config[0] = {
+				.scl_lcnt = 157,
+				.scl_hcnt = 78,
+			}
    	},
    	.i2c[5] = {
    		.speed = I2C_SPEED_FAST,
-			.rise_time_ns = 650,
-			.fall_time_ns = 400,
    		.data_hold_time_ns = 50,
+			.speed_config[0] = {
+				.scl_lcnt = 157,
+				.scl_hcnt = 78,
+			}
    	},
    }"

2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.