From cd598dd8fa760386bd247e1dbfce25cf6882aa8b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Stasiak?= <michal.stasiak@nordicsemi.no>
Date: Wed, 4 Dec 2024 12:02:59 +0100
Subject: [PATCH] tests: boards: nrf: i2c: add support for nRF54H20 cpuppr
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

i2c_slave tests can now be run on nRF54H20 cpuppr target.

Signed-off-by: Michał Stasiak <michal.stasiak@nordicsemi.no>
---
 .../boards/nrf54h20dk_nrf54h20_cpuppr.conf    |  1 +
 .../boards/nrf54h20dk_nrf54h20_cpuppr.overlay | 62 +++++++++++++++++++
 .../boards/nrf54h20dk_nrf54h20_cpuapp.overlay | 51 +++++++++++++++
 .../i2c_slave/sysbuild/vpr_launcher/prj.conf  |  1 +
 tests/boards/nrf/i2c/i2c_slave/testcase.yaml  |  2 +
 5 files changed, 117 insertions(+)
 create mode 100644 tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.conf
 create mode 100644 tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.overlay
 create mode 100644 tests/boards/nrf/i2c/i2c_slave/sysbuild/vpr_launcher/boards/nrf54h20dk_nrf54h20_cpuapp.overlay
 create mode 100644 tests/boards/nrf/i2c/i2c_slave/sysbuild/vpr_launcher/prj.conf

diff --git a/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.conf b/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.conf
new file mode 100644
index 000000000000..157e0a11f728
--- /dev/null
+++ b/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.conf
@@ -0,0 +1 @@
+CONFIG_NRFX_TWIS131=y
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
new file mode 100644
index 000000000000..e9698b2b2bc5
--- /dev/null
+++ b/tests/boards/nrf/i2c/i2c_slave/boards/nrf54h20dk_nrf54h20_cpuppr.overlay
@@ -0,0 +1,62 @@
+/ {
+	aliases {
+		i2c-slave = &i2c131;
+	};
+};
+
+&pinctrl {
+	i2c130_default_alt: i2c130_default_alt {
+		group1 {
+			psels = <NRF_PSEL(TWIM_SDA, 2, 8)>,
+				<NRF_PSEL(TWIM_SCL, 1, 2)>;
+		};
+	};
+
+	i2c130_sleep_alt: i2c130_sleep_alt {
+		group1 {
+			psels = <NRF_PSEL(TWIM_SDA, 2, 8)>,
+				<NRF_PSEL(TWIM_SCL, 1, 2)>;
+			low-power-enable;
+		};
+	};
+
+	i2c131_default_alt: i2c131_default_alt {
+		group1 {
+/* Temporary workaround as it is currently not possible
+ * to configure pins for TWIS with pinctrl.
+ */
+			psels = <NRF_PSEL(TWIM_SDA, 2, 9)>,
+				<NRF_PSEL(TWIM_SCL, 1, 3)>;
+			bias-pull-up;
+		};
+	};
+
+	i2c131_sleep_alt: i2c131_sleep_alt {
+		group1 {
+			psels = <NRF_PSEL(TWIM_SDA, 2, 9)>,
+				<NRF_PSEL(TWIM_SCL, 1, 3)>;
+			low-power-enable;
+		};
+	};
+};
+
+&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";
+	sensor: sensor@54 {
+		reg = <0x54>;
+	};
+};
+
+&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/sysbuild/vpr_launcher/boards/nrf54h20dk_nrf54h20_cpuapp.overlay b/tests/boards/nrf/i2c/i2c_slave/sysbuild/vpr_launcher/boards/nrf54h20dk_nrf54h20_cpuapp.overlay
new file mode 100644
index 000000000000..1d7700959367
--- /dev/null
+++ b/tests/boards/nrf/i2c/i2c_slave/sysbuild/vpr_launcher/boards/nrf54h20dk_nrf54h20_cpuapp.overlay
@@ -0,0 +1,51 @@
+&pinctrl {
+	i2c130_default_alt: i2c130_default_alt {
+		group1 {
+			psels = <NRF_PSEL(TWIM_SDA, 2, 8)>,
+				<NRF_PSEL(TWIM_SCL, 1, 2)>;
+		};
+	};
+
+	i2c130_sleep_alt: i2c130_sleep_alt {
+		group1 {
+			psels = <NRF_PSEL(TWIM_SDA, 2, 8)>,
+				<NRF_PSEL(TWIM_SCL, 1, 2)>;
+			low-power-enable;
+		};
+	};
+
+	i2c131_default_alt: i2c131_default_alt {
+		group1 {
+/* Temporary workaround as it is currently not possible
+ * to configure pins for TWIS with pinctrl.
+ */
+			psels = <NRF_PSEL(TWIM_SDA, 2, 9)>,
+				<NRF_PSEL(TWIM_SCL, 1, 3)>;
+			bias-pull-up;
+		};
+	};
+
+	i2c131_sleep_alt: i2c131_sleep_alt {
+		group1 {
+			psels = <NRF_PSEL(TWIM_SDA, 2, 9)>,
+				<NRF_PSEL(TWIM_SCL, 1, 3)>;
+			low-power-enable;
+		};
+	};
+};
+
+&i2c130 {
+	status = "reserved";
+	pinctrl-0 = <&i2c130_default_alt>;
+	pinctrl-1 = <&i2c130_sleep_alt>;
+	pinctrl-names = "default", "sleep";
+	interrupt-parent = <&cpuppr_clic>;
+};
+
+&i2c131 {
+	status = "reserved";
+	pinctrl-0 = <&i2c131_default_alt>;
+	pinctrl-1 = <&i2c131_sleep_alt>;
+	pinctrl-names = "default", "sleep";
+	interrupt-parent = <&cpuppr_clic>;
+};
diff --git a/tests/boards/nrf/i2c/i2c_slave/sysbuild/vpr_launcher/prj.conf b/tests/boards/nrf/i2c/i2c_slave/sysbuild/vpr_launcher/prj.conf
new file mode 100644
index 000000000000..b2a4ba591044
--- /dev/null
+++ b/tests/boards/nrf/i2c/i2c_slave/sysbuild/vpr_launcher/prj.conf
@@ -0,0 +1 @@
+# nothing here
diff --git a/tests/boards/nrf/i2c/i2c_slave/testcase.yaml b/tests/boards/nrf/i2c/i2c_slave/testcase.yaml
index ad3b34058c74..6a63242ee254 100644
--- a/tests/boards/nrf/i2c/i2c_slave/testcase.yaml
+++ b/tests/boards/nrf/i2c/i2c_slave/testcase.yaml
@@ -10,8 +10,10 @@ tests:
       - nrf5340dk/nrf5340/cpuapp
       - nrf54l15dk/nrf54l15/cpuapp
       - nrf54h20dk/nrf54h20/cpuapp
+      - nrf54h20dk/nrf54h20/cpuppr
     integration_platforms:
       - nrf52840dk/nrf52840
       - nrf5340dk/nrf5340/cpuapp
       - nrf54l15dk/nrf54l15/cpuapp
       - nrf54h20dk/nrf54h20/cpuapp
+      - nrf54h20dk/nrf54h20/cpuppr