From 024df27b0273897123f48d90e44350b7352a5191 Mon Sep 17 00:00:00 2001 From: Adrian Warecki Date: Thu, 4 Jul 2024 17:03:56 +0200 Subject: [PATCH] copier: Remove update source buffer format A data producing component is responsible for setting a sink buffer parameters. It is not necessary for the copier to override source buffer parameters. Remove the code responsible for it. Signed-off-by: Adrian Warecki --- src/audio/copier/copier_generic.c | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/src/audio/copier/copier_generic.c b/src/audio/copier/copier_generic.c index b270e55461a6..ed31fb7414e7 100644 --- a/src/audio/copier/copier_generic.c +++ b/src/audio/copier/copier_generic.c @@ -61,7 +61,7 @@ int apply_attenuation(struct comp_dev *dev, struct copier_data *cd, void copier_update_params(struct copier_data *cd, struct comp_dev *dev, struct sof_ipc_stream_params *params) { - struct comp_buffer *sink, *source; + struct comp_buffer *sink; struct list_item *sink_list; memset(params, 0, sizeof(*params)); @@ -90,19 +90,6 @@ void copier_update_params(struct copier_data *cd, struct comp_dev *dev, ipc4_update_buffer_format(sink, &cd->out_fmt[j]); } - /* - * force update the source buffer format to cover cases where the source module - * fails to set the sink buffer params - */ - if (!list_is_empty(&dev->bsource_list)) { - struct ipc4_audio_format *in_fmt; - - source = list_first_item(&dev->bsource_list, struct comp_buffer, sink_list); - - in_fmt = &cd->config.base.audio_fmt; - ipc4_update_buffer_format(source, in_fmt); - } - /* update params for the DMA buffer */ switch (dev->ipc_config.type) { case SOF_COMP_HOST: