From 7458312a76554c4e90d145c047cc56e64716e9e3 Mon Sep 17 00:00:00 2001 From: Seppo Ingalsuo Date: Wed, 14 Jun 2023 13:41:47 +0300 Subject: [PATCH] Tools: Testbench: Remove unnecessary scheduling component find The pcm_dev is no more used in the code. It also causes a problem with "error: ipc get comp" with some topologies, where p->sched_id is missing. This fix helps testbech to parse a topology like pcm0p --> buf2.0 --> muxdemux --> buf1.0 --> ssp0.out ^ pcm1p --> buf3.0 -------| Signed-off-by: Seppo Ingalsuo --- tools/testbench/common_test.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/tools/testbench/common_test.c b/tools/testbench/common_test.c index 01a131773..50f7a2049 100644 --- a/tools/testbench/common_test.c +++ b/tools/testbench/common_test.c @@ -196,7 +196,6 @@ int tb_pipeline_reset(struct ipc *ipc, struct pipeline *p) int tb_pipeline_params(struct testbench_prm *tp, struct ipc *ipc, struct pipeline *p, struct tplg_context *ctx) { - struct ipc_comp_dev *pcm_dev; struct comp_dev *cd; struct sof_ipc_pcm_params params = {{0}}; char message[DEBUG_MSG_LEN]; @@ -244,13 +243,6 @@ int tb_pipeline_params(struct testbench_prm *tp, struct ipc *ipc, struct pipelin params.params.host_period_bytes = fs_period * params.params.channels * params.params.sample_container_bytes; - /* get scheduling component device for pipeline*/ - pcm_dev = ipc_get_comp_by_id(ipc, p->sched_id); - if (!pcm_dev) { - fprintf(stderr, "error: ipc get comp\n"); - return -EINVAL; - } - /* Get pipeline host component */ cd = tb_get_pipeline_host(p);