tests: drivers: audio: pdm_loopback: Check PDM state after stop start

nordic-segl · nordic-piks · commit 5d8d8a0f3898 · 2025-12-01T11:36:21.000+01:00
Add test that detects issue described in DLT-3748.

Check that PDM configuration is not lost by stopping
and then starting PDM interface.

Signed-off-by: Sebastian Głąb &lt;sebastian.glab@nordicsemi.no&gt;
diff --git a/tests/drivers/audio/pdm_loopback/src/main.c b/tests/drivers/audio/pdm_loopback/src/main.c
@@ -249,10 +249,10 @@ ZTEST(pdm_loopback, test_pdm_clk_frequency)
 {
 	int ret;
 	uint8_t gpiote_channel;
-	nrfx_gpiote_t *gpiote_instance =
-		&GPIOTE_NRFX_INST_BY_NODE(NRF_DT_GPIOTE_NODE(DT_NODELABEL(pulse_counter), gpios));
+	nrfx_gpiote_t gpiote_instance =
+		GPIOTE_NRFX_INST_BY_NODE(NRF_DT_GPIOTE_NODE(DT_NODELABEL(pulse_counter), gpios));
 
-	ret = nrfx_gpiote_channel_alloc(gpiote_instance, &gpiote_channel);
+	ret = nrfx_gpiote_channel_alloc(&gpiote_instance, &gpiote_channel);
 	zassert_true(ret == 0, "GPIOTE channel allocation failed, return code = %d", ret);
 
 	nrfx_gpiote_trigger_config_t trigger_cfg = {
@@ -267,10 +267,10 @@ ZTEST(pdm_loopback, test_pdm_clk_frequency)
 		.p_trigger_config = &trigger_cfg,
 	};
 
-	ret = nrfx_gpiote_input_configure(gpiote_instance, CLOCK_INPUT_PIN, &gpiote_cfg);
+	ret = nrfx_gpiote_input_configure(&gpiote_instance, CLOCK_INPUT_PIN, &gpiote_cfg);
 	zassert_true(ret == 0, "GPIOTE input configuration failed, return code = %d", ret);
 
-	nrfx_gpiote_trigger_enable(gpiote_instance, CLOCK_INPUT_PIN, false);
+	nrfx_gpiote_trigger_enable(&gpiote_instance, CLOCK_INPUT_PIN, false);
 
 	nrfx_timer_config_t timer_config = NRFX_TIMER_DEFAULT_CONFIG(
 					   NRFX_TIMER_BASE_FREQUENCY_GET(&timer_instance));
@@ -284,7 +284,7 @@ ZTEST(pdm_loopback, test_pdm_clk_frequency)
 	nrfx_timer_enable(&timer_instance);
 
 	nrfx_gppi_handle_t gppi_handle;
-	uint32_t eep = nrfx_gpiote_in_event_address_get(gpiote_instance, CLOCK_INPUT_PIN);
+	uint32_t eep = nrfx_gpiote_in_event_address_get(&gpiote_instance, CLOCK_INPUT_PIN);
 	uint32_t tep = nrfx_timer_task_address_get(&timer_instance, NRF_TIMER_TASK_COUNT);
 
 	ret = nrfx_gppi_conn_alloc(eep, tep, &gppi_handle);
@@ -300,6 +300,21 @@ ZTEST(pdm_loopback, test_pdm_clk_frequency)
 		       PDM_EXPECTED_FREQ * SAMPLING_RATIO / 30,
 		       "Captured incorrect frequency Hz. Captured pulses = %lu, expected = %lu",
 		       pulses, PDM_EXPECTED_FREQ * SAMPLING_RATIO);
+
+	/* Remove GPPI configuration. */
+	nrfx_gppi_conn_disable(gppi_handle);
+	nrfx_gppi_conn_free(eep, tep, gppi_handle);
+
+	/* Remove GPIOTE configuration. */
+	ret = nrfx_gpiote_pin_uninit(&gpiote_instance, CLOCK_INPUT_PIN);
+	zexpect_true(ret == 0, "nrfx_gpiote_pin_uninit() ret %d", ret);
+	nrfx_gpiote_trigger_disable(&gpiote_instance, CLOCK_INPUT_PIN);
+	ret = nrfx_gpiote_channel_free(&gpiote_instance, gpiote_channel);
+	zexpect_true(ret == 0, "nrfx_gpiote_channel_free() ret %d", ret);
+
+	/* Remove NRFX Timer configuration. */
+	nrfx_timer_disable(&timer_instance);
+	nrfx_timer_uninit(&timer_instance);
 }
 
 ZTEST_SUITE(pdm_loopback, NULL, device_setup, setup, teardown, NULL);
diff --git a/tests/drivers/audio/pdm_prescaler/CMakeLists.txt b/tests/drivers/audio/pdm_prescaler/CMakeLists.txt
@@ -0,0 +1,12 @@
+#
+# Copyright (c) 2025 Nordic Semiconductor ASA
+#
+# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+#
+
+cmake_minimum_required(VERSION 3.20.0)
+
+find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
+project(pdm_prescaler)
+
+target_sources(app PRIVATE src/main.c)
diff --git a/tests/drivers/audio/pdm_prescaler/Kconfig b/tests/drivers/audio/pdm_prescaler/Kconfig
@@ -0,0 +1,28 @@
+#
+# Copyright (c) 2025 Nordic Semiconductor ASA
+#
+# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+#
+
+source "Kconfig.zephyr"
+
+config TEST_PDM_SAMPLING_RATE
+	int "PDM sample rate in Hz"
+	default 10000
+	help
+	  The test will use it to define frequency of PDM sampling and
+	  calculate the size of the buffer.
+
+config TEST_PDM_EXPECTED_FREQUENCY
+	int "Expected PDM_CLK frequency in Hz"
+	default 1000000
+	help
+	  The test will use it to confirm that the captured PDM_CLK
+	  frequency in correct.
+
+config TEST_PDM_SAMPLING_TIME
+	int "PDM sampling time for one block in ms"
+	default 100
+	help
+	  The test will use it to calculate the size of data block and
+	  determine the period of capturing timer.
diff --git a/tests/drivers/audio/pdm_prescaler/boards/nrf54l15dk_nrf54l15_cpuapp.overlay b/tests/drivers/audio/pdm_prescaler/boards/nrf54l15dk_nrf54l15_cpuapp.overlay
@@ -0,0 +1,39 @@
+/*
+ * Copyright (c) 2025 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+ */
+
+/ {
+	gpio_test {
+		compatible = "gpio-leds";
+
+		pulse_counter: pulse_counter {
+			gpios = <&gpio1 11 GPIO_ACTIVE_HIGH>;
+		};
+	};
+};
+
+&pinctrl {
+	pdm20_default_alt: pdm20_default_alt {
+		group1 {
+			psels = <NRF_PSEL(PDM_CLK, 1, 10)>,
+				<NRF_PSEL(PDM_DIN, 1, 12)>;
+		};
+	};
+};
+
+pdm_dev: &pdm20 {
+	status = "okay";
+	pinctrl-0 = <&pdm20_default_alt>;
+	pinctrl-names = "default";
+	clock-source = "PCLK32M";
+};
+
+&gpio1 {
+	status = "okay";
+};
+
+&timer00 {
+	status = "okay";
+};
diff --git a/tests/drivers/audio/pdm_prescaler/boards/nrf54l15dk_nrf54l15_cpuapp_ns.overlay b/tests/drivers/audio/pdm_prescaler/boards/nrf54l15dk_nrf54l15_cpuapp_ns.overlay
@@ -0,0 +1,39 @@
+/*
+ * Copyright (c) 2025 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+ */
+
+/ {
+	gpio_test {
+		compatible = "gpio-leds";
+
+		pulse_counter: pulse_counter {
+			gpios = <&gpio1 11 GPIO_ACTIVE_HIGH>;
+		};
+	};
+};
+
+&pinctrl {
+	pdm20_default_alt: pdm20_default_alt {
+		group1 {
+			psels = <NRF_PSEL(PDM_CLK, 1, 10)>,
+				<NRF_PSEL(PDM_DIN, 1, 12)>;
+		};
+	};
+};
+
+pdm_dev: &pdm20 {
+	status = "okay";
+	pinctrl-0 = <&pdm20_default_alt>;
+	pinctrl-names = "default";
+	clock-source = "PCLK32M";
+};
+
+&gpio1 {
+	status = "okay";
+};
+
+&timer00 {
+	status = "okay";
+};
diff --git a/tests/drivers/audio/pdm_prescaler/boards/nrf54lm20dk_nrf54lm20a_cpuapp.overlay b/tests/drivers/audio/pdm_prescaler/boards/nrf54lm20dk_nrf54lm20a_cpuapp.overlay
@@ -0,0 +1,43 @@
+/*
+ * Copyright (c) 2025 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+ */
+
+/* Test requires loopback between P1.23 and P1.24.
+ * For best performance, PDM_CLK shall be on 'Clock pin'.
+ */
+
+/ {
+	gpio_test {
+		compatible = "gpio-leds";
+
+		pulse_counter: pulse_counter {
+			gpios = <&gpio1 24 GPIO_ACTIVE_HIGH>;
+		};
+	};
+};
+
+&pinctrl {
+	pdm20_default_alt: pdm20_default_alt {
+		group1 {
+			psels = <NRF_PSEL(PDM_CLK, 1, 23)>,
+				<NRF_PSEL(PDM_DIN, 1, 0)>;
+		};
+	};
+};
+
+pdm_dev: &pdm20 {
+	status = "okay";
+	pinctrl-0 = <&pdm20_default_alt>;
+	pinctrl-names = "default";
+	clock-source = "PCLK32M";
+};
+
+&gpio1 {
+	status = "okay";
+};
+
+&timer00 {
+	status = "okay";
+};
diff --git a/tests/drivers/audio/pdm_prescaler/nrf54l_aclk.overlay b/tests/drivers/audio/pdm_prescaler/nrf54l_aclk.overlay
@@ -0,0 +1,9 @@
+/*
+ * Copyright (c) 2025 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+ */
+
+&pdm_dev {
+	clock-source = "ACLK";
+};
diff --git a/tests/drivers/audio/pdm_prescaler/prj.conf b/tests/drivers/audio/pdm_prescaler/prj.conf
@@ -0,0 +1,14 @@
+#
+# Copyright (c) 2025 Nordic Semiconductor ASA
+#
+# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+#
+
+CONFIG_ZTEST=y
+CONFIG_TEST_USERSPACE=y
+
+CONFIG_AUDIO=y
+CONFIG_AUDIO_DMIC=y
+CONFIG_GPIO=y
+CONFIG_NRFX_GPPI=y
+CONFIG_NRFX_TIMER=y
diff --git a/tests/drivers/audio/pdm_prescaler/src/main.c b/tests/drivers/audio/pdm_prescaler/src/main.c
diff --git a/tests/drivers/audio/pdm_prescaler/testcase.yaml b/tests/drivers/audio/pdm_prescaler/testcase.yaml
