From 9fd8adb167a3361509c7ec2523a71b1ba39c0433 Mon Sep 17 00:00:00 2001 From: Guennadi Liakhovetski Date: Fri, 30 May 2025 13:54:42 +0200 Subject: [PATCH] module-adapter: remove a superfluous check Remove a check for an impossible condition and make code clearer. The check became redundant after 097405872507 ("module_adapter: Don't propagate ENOSPC and ENODATA error codes"). Presence of redundant checks makes code confusing. Signed-off-by: Guennadi Liakhovetski --- src/audio/module_adapter/module_adapter.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/audio/module_adapter/module_adapter.c b/src/audio/module_adapter/module_adapter.c index f5a156bec24a..7311bbcf87e9 100644 --- a/src/audio/module_adapter/module_adapter.c +++ b/src/audio/module_adapter/module_adapter.c @@ -990,13 +990,9 @@ static int module_adapter_sink_source_copy(struct comp_dev *dev) ret = module_process_sink_src(mod, mod->sources, mod->num_of_sources, mod->sinks, mod->num_of_sinks); - if (ret) { - if (ret != -ENOSPC && ret != -ENODATA) - comp_err(dev, "module_adapter_sink_source_copy() process failed with error: %d", - ret); - else - ret = 0; - } + if (ret) + comp_err(dev, "module_adapter_sink_source_copy() process failed with error: %d", + ret); /* count number of processed data. To be removed in pipeline 2.0 */ for (i = 0; i < mod->num_of_sources; i++)