Test(cmocka, crossover): move source/sink stubs to a shared mock unit

Signed-off-by: Piotr Hoppe <piotr.hoppe@intel.com>

Author: piotrhoppeintel

test/cmocka/src/audio/crossover/CMakeLists.txt

@@ -13,14 +13,16 @@ cmocka_test(crossover_lr4_test
 target_link_libraries(crossover_lr4_test PRIVATE m)
 
 # crossover_split_test: tests the split_2way / split_3way / split_4way routing
-# topology by compiling crossover_generic.c directly.  The FORMAT macros are
-# left undefined so that only the split functions (no audio_stream dependency)
-# are exercised; audio_stream_copy is stubbed inside crossover_split_test.c.
+# topology by compiling crossover_generic.c directly.  The per-format processing
+# functions in crossover_generic.c reference the source/sink API; those symbols
+# are satisfied by the trivial stubs in crossover_test_mocks.c (shareable by any
+# test that links crossover_generic.c).
 
 add_compile_options(-DUNIT_TEST)
 
 cmocka_test(crossover_split_test
 	crossover_split_test.c
+	crossover_test_mocks.c
 	${PROJECT_SOURCE_DIR}/src/audio/crossover/crossover_generic.c
 	${PROJECT_SOURCE_DIR}/src/math/iir_df1.c
 	${PROJECT_SOURCE_DIR}/src/math/iir_df1_generic.c

test/cmocka/src/audio/crossover/crossover_split_test.c

@@ -45,19 +45,9 @@
 #include <stdio.h>
 #include <cmocka.h>
 
-/* audio_stream_copy is referenced by crossover_default_pass() which is
- * compiled as part of crossover_generic.c.  The split functions under test
- * never call it, but the linker needs a definition.
+/* The source/sink API stubs required to link crossover_generic.c live in
+ * crossover_test_mocks.c, added to this test's sources via CMake.
  */
-#include <sof/audio/audio_stream.h>
-
-int audio_stream_copy(const struct audio_stream *source, uint32_t ioffset,
-		      struct audio_stream *sink, uint32_t ooffset, uint32_t samples)
-{
-	(void)source; (void)ioffset; (void)sink; (void)ooffset; (void)samples;
-	return 0;
-}
-
 #include <sof/audio/format.h>
 #include <sof/math/iir_df1.h>
 #include <module/crossover/crossover_common.h>

test/cmocka/src/audio/crossover/crossover_test_mocks.c

@@ -0,0 +1,78 @@
+// SPDX-License-Identifier: BSD-3-Clause
+//
+// Copyright(c) 2025 Intel Corporation. All rights reserved.
+//
+// Author: Piotr Hoppe <piotr.hoppe@intel.com>
+
+/**
+ * @file crossover_test_mocks.c
+ * @brief Trivial source/sink API stubs for the crossover unit tests.
+ *
+ * The crossover_generic.c translation unit also contains the per-format
+ * processing functions (crossover_s16/s24/s32_default and the passthrough),
+ * which reference the source/sink API. The split functions under test never
+ * call them, but the linker still needs the referenced symbols defined.
+ *
+ * The stubs are provided by this separate translation unit so that any number
+ * of test executables can link them via CMake without risking multiple
+ * definitions. The prototypes come from the real source/sink API headers.
+ */
+
+#include <stdbool.h>
+#include <stddef.h>
+#include <stdint.h>
+#include <sof/audio/source_api.h>
+#include <sof/audio/sink_api.h>
+#include <sof/audio/sink_source_utils.h>
+
+size_t source_get_frame_bytes(struct sof_source *source)
+{
+	(void)source;
+	return 0;
+}
+
+int source_get_data_s16(struct sof_source *source, size_t req_size, int16_t const **data_ptr,
+			int16_t const **buffer_start, int *buffer_samples)
+{
+	(void)source; (void)req_size; (void)data_ptr; (void)buffer_start; (void)buffer_samples;
+	return 0;
+}
+
+int source_get_data_s32(struct sof_source *source, size_t req_size, int32_t const **data_ptr,
+			int32_t const **buffer_start, int *buffer_samples)
+{
+	(void)source; (void)req_size; (void)data_ptr; (void)buffer_start; (void)buffer_samples;
+	return 0;
+}
+
+int source_release_data(struct sof_source *source, size_t free_size)
+{
+	(void)source; (void)free_size;
+	return 0;
+}
+
+int sink_get_buffer_s16(struct sof_sink *sink, size_t req_size, int16_t **data_ptr,
+			int16_t **buffer_start, int *buffer_samples)
+{
+	(void)sink; (void)req_size; (void)data_ptr; (void)buffer_start; (void)buffer_samples;
+	return 0;
+}
+
+int sink_get_buffer_s32(struct sof_sink *sink, size_t req_size, int32_t **data_ptr,
+			int32_t **buffer_start, int *buffer_samples)
+{
+	(void)sink; (void)req_size; (void)data_ptr; (void)buffer_start; (void)buffer_samples;
+	return 0;
+}
+
+int sink_commit_buffer(struct sof_sink *sink, size_t commit_size)
+{
+	(void)sink; (void)commit_size;
+	return 0;
+}
+
+int source_to_sink_copy(struct sof_source *source, struct sof_sink *sink, bool free, size_t size)
+{
+	(void)source; (void)sink; (void)free; (void)size;
+	return 0;
+}