samples: port radio_test to ncs-bm

changed the implementation of the sample
to the subsystems and modules available in sdk-nrf-bm:
- use shell_print instead of printk everywhere
- use sample-specific clock_init() function
- cleaned-up prj.conf, disabled SD explicitly

Signed-off-by: Ivan Iushkov <ivan.iushkov@nordicsemi.no>

Author: ivaniushkov

samples/peripherals/radio_test/README.rst

@@ -25,113 +25,17 @@ The sample supports the following development kits:
 
       The following board variants do **not** have DFU capabilities:
 
-      .. list-table::
-         :header-rows: 1
-
-         * - Hardware platform
-           - PCA
-           - SoftDevice
-           - Board target
-         * - `nRF54L15 DK`_
-           - PCA10156
-           - S115
-           - bm_nrf54l15dk/nrf54l15/cpuapp/s115_softdevice
-         * - `nRF54L15 DK`_ (emulating nRF54L10)
-           - PCA10156
-           - S115
-           - bm_nrf54l15dk/nrf54l10/cpuapp/s115_softdevice
-         * - `nRF54L15 DK`_ (emulating nRF54L05)
-           - PCA10156
-           - S115
-           - bm_nrf54l15dk/nrf54l05/cpuapp/s115_softdevice
-         * - `nRF54LM20 DK`_
-           - PCA10184
-           - S115
-           - bm_nrf54lm20dk/nrf54lm20a/cpuapp/s115_softdevice
-         * - nRF54LS05 DK
-           - PCA10214
-           - S115
-           - bm_nrf54ls05dk/nrf54ls05b/cpuapp/s115_softdevice
-         * - `nRF54LV10 DK`_
-           - PCA10188
-           - S115
-           - bm_nrf54lv10dk/nrf54lv10/cpuapp/s115_softdevice
-         * - `nRF54L15 DK`_
-           - PCA10156
-           - S145
-           - bm_nrf54l15dk/nrf54l15/cpuapp/s145_softdevice
-         * - `nRF54L15 DK`_ (emulating nRF54L10)
-           - PCA10156
-           - S145
-           - bm_nrf54l15dk/nrf54l10/cpuapp/s145_softdevice
-         * - `nRF54L15 DK`_ (emulating nRF54L05)
-           - PCA10156
-           - S145
-           - bm_nrf54l15dk/nrf54l05/cpuapp/s145_softdevice
-         * - `nRF54LM20 DK`_
-           - PCA10184
-           - S145
-           - bm_nrf54lm20dk/nrf54lm20a/cpuapp/s145_softdevice
-         * - nRF54LS05 DK
-           - PCA10214
-           - S145
-           - bm_nrf54ls05dk/nrf54ls05b/cpuapp/s145_softdevice
-         * - `nRF54LV10 DK`_
-           - PCA10188
-           - S145
-           - bm_nrf54lv10dk/nrf54lv10/cpuapp/s145_softdevice
+      .. include:: /includes/supported_boards_all_non-mcuboot_variants_s115.txt
+
+      .. include:: /includes/supported_boards_all_non-mcuboot_variants_s145.txt
 
    .. group-tab:: MCUboot board variants
 
       The following board variants have DFU capabilities:
 
-      .. list-table::
-         :header-rows: 1
-
-         * - Hardware platform
-           - PCA
-           - SoftDevice
-           - Board target
-         * - `nRF54L15 DK`_
-           - PCA10156
-           - S115
-           - bm_nrf54l15dk/nrf54l15/cpuapp/s115_softdevice/mcuboot
-         * - `nRF54L15 DK`_ (emulating nRF54L10)
-           - PCA10156
-           - S115
-           - bm_nrf54l15dk/nrf54l10/cpuapp/s115_softdevice/mcuboot
-         * - `nRF54L15 DK`_ (emulating nRF54L05)
-           - PCA10156
-           - S115
-           - bm_nrf54l15dk/nrf54l05/cpuapp/s115_softdevice/mcuboot
-         * - `nRF54LM20 DK`_
-           - PCA10184
-           - S115
-           - bm_nrf54lm20dk/nrf54lm20a/cpuapp/s115_softdevice/mcuboot
-         * - `nRF54LV10 DK`_
-           - PCA10188
-           - S115
-           - bm_nrf54lv10dk/nrf54lv10/cpuapp/s115_softdevice/mcuboot
-         * - `nRF54L15 DK`_
-           - PCA10156
-           - S145
-           - bm_nrf54l15dk/nrf54l15/cpuapp/s145_softdevice/mcuboot
-         * - `nRF54L15 DK`_ (emulating nRF54L10)
-           - PCA10156
-           - S145
-           - bm_nrf54l15dk/nrf54l10/cpuapp/s145_softdevice/mcuboot
-         * - `nRF54L15 DK`_ (emulating nRF54L05)
-           - PCA10156
-           - S145
-           - bm_nrf54l15dk/nrf54l05/cpuapp/s145_softdevice/mcuboot
-         * - `nRF54LM20 DK`_
-           - PCA10184
-           - S145
-           - bm_nrf54lm20dk/nrf54lm20a/cpuapp/s145_softdevice/mcuboot
-         * - `nRF54LV10 DK`_
-           - PCA10188
-           - S145
-           - bm_nrf54lv10dk/nrf54lv10/cpuapp/s145_softdevice/mcuboot
+      .. include:: /includes/supported_boards_all_mcuboot_variants_s115.txt
+
+      .. include:: /includes/supported_boards_all_mcuboot_variants_s145.txt
 
 
 The sample also requires one of the following testing devices:
@@ -149,7 +53,7 @@ Overview
 ********
 
 To run the tests, connect to the development kit through the serial port and send shell commands.
-Zephyr's :ref:`zephyr:shell_api` module is used to handle the commands.
+Shell subsystem is used to handle the commands.
 At any time during the tests, you can dynamically set the radio parameters, such as output power, bit rate, and channel.
 In sweep mode, you can set the time for which the radio scans each channel from one millisecond to 99 milliseconds, in steps of one millisecond.
 The sample also allows you to send a data pattern to another development kit.
@@ -274,30 +178,3 @@ Complete the following steps:
 #. Set the kit in the TX sweep mode using the ``start_tx_sweep`` command.
 #. Start the `RSSI Viewer app`_ and select the kit to communicate with.
 #. On the application chart, observe the TX sweep in the form of a wave that starts at 2420 MHz frequency and ends with 2480 MHz.
-
-Dependencies
-************
-
-This sample uses the following |NCS| libraries:
-
-  * :ref:`shell_ipc_readme`
-
-This sample has the following nrfx dependencies:
-
-  * :file:`nrfx/drivers/include/nrfx_timer.h`
-  * :file:`nrfx/hal/nrf_power.h`
-  * :file:`nrfx/hal/nrf_radio.h`
-
-In addition, it uses the following Zephyr libraries:
-
-* :ref:`zephyr:device_model_api`:
-
-   * :file:`drivers/clock_control.h`
-
-* :ref:`zephyr:kernel_api`:
-
-  * :file:`include/init.h`
-
-* :ref:`zephyr:shell_api`:
-
-  * :file:`include/shell/shell.h`

samples/peripherals/radio_test/prj.conf

@@ -4,15 +4,27 @@
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
-CONFIG_CONSOLE_HANDLER=y
+# Logging
+CONFIG_LOG=y
+CONFIG_LOG_BACKEND_BM_UARTE=y
+
+# Shell
 CONFIG_SHELL=y
+CONFIG_SHELL_BACKEND_BM_UARTE=y
 
-CONFIG_DYNAMIC_INTERRUPTS=y
+# Optional shell features
+CONFIG_SHELL_HELP=y
+CONFIG_SHELL_TAB=y
+CONFIG_SHELL_TAB_AUTOCOMPLETION=y
+CONFIG_SHELL_HISTORY=y
 
+# HW used by radio_test
 CONFIG_NRFX_TIMER=y
 CONFIG_NRFX_GPPI=y
 
-CONFIG_ENTROPY_GENERATOR=y
-CONFIG_NRF_SECURITY=y
+# Use fake PRNG to reduce HW dependencies
+CONFIG_TEST_RANDOM_GENERATOR=y
 
-CONFIG_SHELL_BACKEND_SERIAL_RX_RING_BUFFER_SIZE=1024
+# Disable SoftDevice.
+# The radio_test sample needs to controlr radio and other relevant periperhals on its own.
+CONFIG_SOFTDEVICE=n

samples/peripherals/radio_test/src/main.c

@@ -4,56 +4,30 @@
  * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
  */
 
-#include <zephyr/sys/printk.h>
-#include <zephyr/drivers/clock_control.h>
-#include <zephyr/drivers/clock_control/nrf_clock_control.h>
-#include <zephyr/pm/device_runtime.h>
-#if defined(NRF54L15_XXAA)
-#include <hal/nrf_clock.h>
-#endif /* defined(NRF54L15_XXAA) */
+#include <zephyr/shell/shell.h>
+#include <bm/shell/backend_bm_uarte.h>
 #include <nrfx.h>
+#include <hal/nrf_clock.h>
 /* TODO: DRGN-27733 Remove the alternative condition for nRF54LS05B
  * when the errata has been applied to this chip and the errata
  * check can be used instead.
  */
 #if NRF54L_ERRATA_20_PRESENT || defined(NRF54LS05B_ENGA_XXAA)
 #include <hal/nrf_power.h>
 #endif /* NRF54L_ERRATA_20_PRESENT */
-#if defined(NRF54LM20A_XXAA)
-#include <hal/nrf_clock.h>
-#endif /* defined(NRF54LM20A_XXAA) */
 
-/* Empty trim value */
-#define TRIM_VALUE_EMPTY 0xFFFFFFFF
+#include "radio_test.h"
+
+const struct shell *sh;
 
 static void clock_init(void)
 {
-	int err;
-	int res;
-	struct onoff_manager *clk_mgr;
-	struct onoff_client clk_cli;
-
-	clk_mgr = z_nrf_clock_control_get_onoff(CLOCK_CONTROL_NRF_SUBSYS_HF);
-	if (!clk_mgr) {
-		printk("Unable to get the Clock manager\n");
-		return;
+	nrf_clock_event_clear(NRF_CLOCK, NRF_CLOCK_EVENT_HFCLKSTARTED);
+	nrf_clock_task_trigger(NRF_CLOCK, NRF_CLOCK_TASK_HFCLKSTART);
+	while (!nrf_clock_event_check(NRF_CLOCK, NRF_CLOCK_EVENT_HFCLKSTARTED)) {
+		/* spin until HFXO is up */
 	}
-
-	sys_notify_init_spinwait(&clk_cli.notify);
-
-	err = onoff_request(clk_mgr, &clk_cli);
-	if (err < 0) {
-		printk("Clock request failed: %d\n", err);
-		return;
-	}
-
-	do {
-		err = sys_notify_fetch_result(&clk_cli.notify, &res);
-		if (!err && res) {
-			printk("Clock could not be started: %d\n", res);
-			return;
-		}
-	} while (err);
+	nrf_clock_event_clear(NRF_CLOCK, NRF_CLOCK_EVENT_HFCLKSTARTED);
 
 #if NRF54L_ERRATA_20_PRESENT
 	if (nrf54l_errata_20()) {
@@ -70,15 +44,53 @@ static void clock_init(void)
 	/* MLTPAN-39 */
 	nrf_clock_task_trigger(NRF_CLOCK, NRF_CLOCK_TASK_PLLSTART);
 #endif /* defined(NRF54LM20A_XXAA) */
-
-	printk("Clock has started\n");
 }
 
 int main(void)
 {
-	printk("Starting Radio Test sample\n");
+	sh = shell_backend_bm_uarte_get_ptr();
+	const struct shell_backend_config_flags cfg_flags =
+		SHELL_DEFAULT_BACKEND_CONFIG_FLAGS;
+
+	shell_init(sh, NULL, cfg_flags, false, 0);
+	shell_start(sh);
+
+	shell_print(sh, "Starting Radio Test sample");
 
 	clock_init();
 
+	struct radio_test_config *p_radio_test_conf = radio_cmd_get_test_config();
+	int err = radio_test_init(p_radio_test_conf);
+
+	if (err)
+	{
+		shell_error(sh, "radio_test_init() failed: %d", err);
+	}
+
+	while (true) {
+		/*
+		 * Process all received data from the shell backend, calling registered command
+		 * handlers like sample_terminate_cmd if the command and arguments match.
+		 */
+		shell_process(sh);
+
+		/*
+		 * Ensure no events are missed while we check if there is pending RX data to
+		 * be processed.
+		 */
+		unsigned int key = irq_lock();
+
+		radio_test_process_rx_timeout();
+
+		if (shell_backend_bm_uarte_rx_ready()) {
+			/* Process pending RX data */
+			irq_unlock(key);
+			continue;
+		}
+
+		/* Idle until next event */
+		k_cpu_atomic_idle(key);
+	}
+
 	return 0;
 }