tests: boards/intel_adsp/smoke: remove IPM test

This removes the IPM test from the intel_adsp smoke test.
The IPM driver is not being used by SOF and is there simply
for this test. The host IPC functionality is also being tested
via test_host_ipc. The additional usage of mailbox in the test
does not provide much value since the mailbox is shared memory
between the DSP and the host. We would have bigger problems if
the shared memory does not work properly.

Removing the test is in preparation to removing the CAVS host
IPM driver. That is also a preparation to rewrite the host IPC
driver to the IPC API, and there is no need to maintain
another host IPC driver that is not used by SOF.

Signed-off-by: Daniel Leung <[email protected]>

Author: dcpleung

tests/boards/intel_adsp/smoke/CMakeLists.txt

@@ -4,4 +4,4 @@ cmake_minimum_required(VERSION 3.20.0)
 find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
 project(intel_adsp)
 
-target_sources(app PRIVATE src/main.c src/smpboot.c src/hostipc.c src/cpus.c src/ipm.c)
+target_sources(app PRIVATE src/main.c src/smpboot.c src/hostipc.c src/cpus.c)

tests/boards/intel_adsp/smoke/prj.conf

@@ -2,5 +2,3 @@ CONFIG_ZTEST=y
 CONFIG_SMP=y
 CONFIG_SMP_BOOT_DELAY=y
 CONFIG_SCHED_CPU_MASK=y
-CONFIG_IPM=y
-CONFIG_IPM_CAVS_HOST=y

tests/boards/intel_adsp/smoke/src/ipm.c

@@ -6,8 +6,6 @@
 #include <stdlib.h>
 #include "tests.h"
 
-intel_adsp_ipc_handler_t ipm_handler;
-
 static bool clock_msg(const struct device *dev, void *arg,
 		      uint32_t data, uint32_t ext_data)
 {
@@ -66,15 +64,6 @@ ZTEST(intel_adsp, test_vecbase_lock)
 }
 #endif
 
-static void *intel_adsp_setup(void)
-{
-	struct intel_adsp_ipc_data *devdata = INTEL_ADSP_IPC_HOST_DEV->data;
-
-	ipm_handler = devdata->handle_message;
-
-	return NULL;
-}
-
 static void intel_adsp_teardown(void *data)
 {
 	/* Wait a bit so the python script on host is ready to receive
@@ -85,7 +74,6 @@ static void intel_adsp_teardown(void *data)
 	k_msleep(1000);
 }
 
-ZTEST_SUITE(intel_adsp, NULL, intel_adsp_setup, NULL, NULL,
-		intel_adsp_teardown);
+ZTEST_SUITE(intel_adsp, NULL, NULL, NULL, NULL, intel_adsp_teardown);
 
-ZTEST_SUITE(intel_adsp_boot, NULL, intel_adsp_setup, NULL, NULL, NULL);
+ZTEST_SUITE(intel_adsp_boot, NULL, NULL, NULL, NULL, NULL);

tests/boards/intel_adsp/smoke/src/main.c

@@ -20,10 +20,4 @@
 		zassert_true(i < 10000, "timeout waiting for %s", #expr); \
 	} while (0)
 
-/* Cached copy of the ipm_cavs_host driver's handler.  We save it at
- * the start of the test because we want to do unit testing on the
- * underlying adsp_ipc device, then recover it later.
- */
-extern intel_adsp_ipc_handler_t ipm_handler;
-
 #endif /* ZEPHYR_TESTS_INTEL_ADSP_TESTS_H */