tests: boards: i2c: Add testing with different i2c speeds

nordic-bami · kartben · commit 21f71dfab40e · 2025-04-30T16:27:43.000+02:00
Test with all available speeds on the given device.

Signed-off-by: Bartosz Miller &lt;bartosz.miller@nordicsemi.no&gt;
diff --git a/MAINTAINERS.yml b/MAINTAINERS.yml
@@ -4026,6 +4026,7 @@ nRF Platforms:
     - dts/bindings/*/nordic,*
     - tests/drivers/*/*nrf*/
     - snippets/nordic*/
+    - tests/boards/nrf/
   labels:
     - "platform: nRF"
 
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/bl54l15_dvk_nrf54l15_cpuapp.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/bl54l15_dvk_nrf54l15_cpuapp.overlay
@@ -46,13 +46,13 @@
 	};
 };
 
-&i2c21 {
+dut_twim: &i2c21 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
 	pinctrl-0 = <&i2c21_default_alt>;
 	pinctrl-1 = <&i2c21_sleep_alt>;
 	pinctrl-names = "default", "sleep";
-
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	sensor: sensor@54 {
 		reg = <0x54>;
 	};
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/bl54l15u_dvk_nrf54l15_cpuapp.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/bl54l15u_dvk_nrf54l15_cpuapp.overlay
@@ -46,13 +46,13 @@
 	};
 };
 
-&i2c21 {
+dut_twim: &i2c21 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
 	pinctrl-0 = <&i2c21_default_alt>;
 	pinctrl-1 = <&i2c21_sleep_alt>;
 	pinctrl-names = "default", "sleep";
-
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	sensor: sensor@54 {
 		reg = <0x54>;
 	};
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/i2c_speed_fast.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/i2c_speed_fast.overlay
@@ -0,0 +1,8 @@
+/*
+ * Copyright 2025 Nordic Semiconductor ASA
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+&dut_twim {
+	clock-frequency = <I2C_BITRATE_FAST>;
+};
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/i2c_speed_fast_plus.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/i2c_speed_fast_plus.overlay
@@ -0,0 +1,8 @@
+/*
+ * Copyright 2025 Nordic Semiconductor ASA
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+&dut_twim {
+	clock-frequency = <I2C_BITRATE_FAST_PLUS>;
+};
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/nrf52840dk_nrf52840.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/nrf52840dk_nrf52840.overlay
@@ -39,22 +39,24 @@
 	};
 };
 
-&i2c0 {
+dut_twim: &i2c0 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
 	pinctrl-0 = <&i2c0_default_alt>;
 	pinctrl-1 = <&i2c0_sleep_alt>;
 	pinctrl-names = "default", "sleep";
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	sensor: sensor@54 {
 		reg = <0x54>;
 	};
 };
 
 
-&i2c1 {
+dut_twis: &i2c1 {
 	compatible = "nordic,nrf-twis";
 	status = "okay";
 	pinctrl-0 = <&i2c1_default_alt>;
 	pinctrl-1 = <&i2c1_sleep_alt>;
 	pinctrl-names = "default", "sleep";
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 };
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/nrf5340dk_nrf5340_cpuapp.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/nrf5340dk_nrf5340_cpuapp.overlay
@@ -39,12 +39,13 @@
 	};
 };
 
-&i2c1 {
+dut_twim: &i2c1 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
 	pinctrl-0 = <&i2c1_default_alt>;
 	pinctrl-1 = <&i2c1_sleep_alt>;
 	pinctrl-names = "default", "sleep";
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	sensor: sensor@54 {
 		reg = <0x54>;
 	};
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuapp.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuapp.overlay
@@ -39,13 +39,13 @@
 	};
 };
 
-&i2c130 {
+dut_twim: &i2c130 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
-	clock-frequency = <I2C_BITRATE_STANDARD>;
 	pinctrl-0 = <&i2c130_default_alt>;
 	pinctrl-1 = <&i2c130_sleep_alt>;
 	pinctrl-names = "default", "sleep";
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	memory-regions = <&cpuapp_dma_region>;
 	sensor: sensor@54 {
 		reg = <0x54>;
@@ -55,7 +55,6 @@
 &i2c131 {
 	compatible = "nordic,nrf-twis";
 	status = "okay";
-	clock-frequency = <I2C_BITRATE_STANDARD>;
 	pinctrl-0 = <&i2c131_default_alt>;
 	pinctrl-1 = <&i2c131_sleep_alt>;
 	pinctrl-names = "default", "sleep";
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.overlay
@@ -40,13 +40,13 @@
 	};
 };
 
-&i2c130 {
+dut_twim: &i2c130 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
-	clock-frequency = <I2C_BITRATE_STANDARD>;
 	pinctrl-0 = <&i2c130_default_alt>;
 	pinctrl-1 = <&i2c130_sleep_alt>;
 	pinctrl-names = "default", "sleep";
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	sensor: sensor@54 {
 		reg = <0x54>;
 	};
@@ -55,7 +55,6 @@
 &i2c131 {
 	compatible = "nordic,nrf-twis";
 	status = "okay";
-	clock-frequency = <I2C_BITRATE_STANDARD>;
 	pinctrl-0 = <&i2c131_default_alt>;
 	pinctrl-1 = <&i2c131_sleep_alt>;
 	pinctrl-names = "default", "sleep";
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/nrf54l15dk_nrf54l15_cpuapp.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/nrf54l15dk_nrf54l15_cpuapp.overlay
@@ -40,12 +40,13 @@
 	};
 };
 
-&i2c21 {
+dut_twim: &i2c21 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
 	pinctrl-0 = <&i2c21_default_alt>;
 	pinctrl-1 = <&i2c21_sleep_alt>;
 	pinctrl-names = "default", "sleep";
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	sensor: sensor@54 {
 		reg = <0x54>;
 	};
diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/nrf54l20pdk_nrf54l20_cpuapp.overlay b/tests/boards/nrf/i2c/i2c_slave/boards/nrf54l20pdk_nrf54l20_cpuapp.overlay
@@ -40,12 +40,13 @@
 	};
 };
 
-&i2c21 {
+dut_twim: &i2c21 {
 	compatible = "nordic,nrf-twim";
 	status = "okay";
 	pinctrl-0 = <&i2c21_default_alt>;
 	pinctrl-1 = <&i2c21_sleep_alt>;
 	pinctrl-names = "default", "sleep";
+	clock-frequency = <I2C_BITRATE_STANDARD>;
 	sensor: sensor@54 {
 		reg = <0x54>;
 	};
diff --git a/tests/boards/nrf/i2c/i2c_slave/testcase.yaml b/tests/boards/nrf/i2c/i2c_slave/testcase.yaml
@@ -1,12 +1,14 @@
+common:
+  depends_on: i2c
+  tags:
+    - drivers
+    - i2c
+  harness: ztest
+  harness_config:
+    fixture: i2c_loopback
+
 tests:
   boards.nrf.i2c.i2c_slave:
-    depends_on: i2c
-    tags:
-      - drivers
-      - i2c
-    harness: ztest
-    harness_config:
-      fixture: i2c_loopback
     platform_allow:
       - nrf52840dk/nrf52840
       - nrf5340dk/nrf5340/cpuapp
@@ -20,3 +22,30 @@ tests:
       - nrf54l15dk/nrf54l15/cpuapp
       - nrf54h20dk/nrf54h20/cpuapp
       - nrf54h20dk/nrf54h20/cpuppr
+  boards.nrf.i2c.i2c_slave.fast:
+    platform_allow:
+      - nrf52840dk/nrf52840
+      - nrf5340dk/nrf5340/cpuapp
+      - nrf54l15dk/nrf54l15/cpuapp
+      - nrf54l20pdk/nrf54l20/cpuapp
+      - nrf54h20dk/nrf54h20/cpuapp
+      - nrf54h20dk/nrf54h20/cpuppr
+    integration_platforms:
+      - nrf52840dk/nrf52840
+      - nrf5340dk/nrf5340/cpuapp
+      - nrf54l15dk/nrf54l15/cpuapp
+      - nrf54h20dk/nrf54h20/cpuapp
+      - nrf54h20dk/nrf54h20/cpuppr
+    extra_args:
+      - EXTRA_DTC_OVERLAY_FILE="boards/i2c_speed_fast.overlay"
+  boards.nrf.i2c.i2c_slave.fast_plus:
+    platform_allow:
+      - nrf5340dk/nrf5340/cpuapp
+      - nrf54h20dk/nrf54h20/cpuapp
+      - nrf54h20dk/nrf54h20/cpuppr
+    integration_platforms:
+      - nrf5340dk/nrf5340/cpuapp
+      - nrf54h20dk/nrf54h20/cpuapp
+      - nrf54h20dk/nrf54h20/cpuppr
+    extra_args:
+      - EXTRA_DTC_OVERLAY_FILE="boards/i2c_speed_fast_plus.overlay"
