src: audio: module_adapter: Do the params config right after init

The module_adapter_params() functions set the stream params based on the
base config for each module which is available right after module init.
So configure the params for IPC4 during module_new() and remove the call
to ipc4_pipeline_params(). In order to update the buffer params, add a
call to comp_verify_params() during module_adapter_prepare in order to
update the intermediate buffers. This should help with reducing the
time to trigger pipelines during start.

Add the params handling for the KPB and detect_test modules explicitly
because they don't use the module adapter.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>

Author: ranj063

src/audio/kpb.c

@@ -453,6 +453,8 @@ static void kpb_set_params(struct comp_dev *dev,
 {}
 #endif /* CONFIG_IPC_MAJOR_4 */
 
+static int kpb_params(struct comp_dev *dev, struct sof_ipc_stream_params *params);
+
 /*
  * \brief Create a key phrase buffer component.
  * \param[in] config - generic ipc component pointer.
@@ -534,6 +536,17 @@ static struct comp_dev *kpb_new(const struct comp_driver *drv,
 	dev->state = COMP_STATE_READY;
 	kpb_change_state(kpb, KPB_STATE_CREATED);
 
+#if CONFIG_IPC_MAJOR_4
+	struct sof_ipc_stream_params params;
+
+	/* set params based on base config for IPC4 */
+	ret = kpb_params(dev, &params);
+	if (ret < 0) {
+		rfree(dev);
+		return NULL;
+	}
+#endif
+
 	return dev;
 }
 
@@ -786,12 +799,21 @@ static int kpb_params(struct comp_dev *dev,
 static int kpb_prepare(struct comp_dev *dev)
 {
 	struct comp_data *kpb = comp_get_drvdata(dev);
+	struct sof_ipc_stream_params params;
 	int ret = 0;
 	int i;
 	size_t hb_size_req = KPB_MAX_BUFFER_SIZE(kpb->config.sampling_width, kpb->config.channels);
 
 	comp_dbg(dev, "kpb_prepare()");
 
+	kpb_set_params(dev, &params);
+
+	ret = kbp_verify_params(dev, &params);
+	if (ret < 0) {
+		comp_err(dev, "kpb_prepare: pcm params verification failed");
+		return -EINVAL;
+	}
+
 	if (kpb->state == KPB_STATE_RESETTING ||
 	    kpb->state == KPB_STATE_RESET_FINISHING) {
 		comp_cl_err(&comp_kpb, "can not prepare KPB due to ongoing reset, state log %x",

src/audio/module_adapter/module_adapter.c

@@ -123,6 +123,19 @@ struct comp_dev *module_adapter_new(const struct comp_driver *drv,
 
 	dev->state = COMP_STATE_READY;
 
+#if CONFIG_IPC_MAJOR_4
+	struct sof_ipc_stream_params params = {{ 0 }};
+
+	/* set the stream params based on the module base cfg. It is OK to pass NULL params
+	 * here because it will be filled in based on the module base_cfg.
+	 */
+	ret = module_adapter_params(dev, &params);
+	if (ret) {
+		comp_err(dev, "module_adapter_new() %d: module params failed", ret);
+		goto err;
+	}
+#endif
+
 	comp_dbg(dev, "module_adapter_new() done");
 	return dev;
 err:
@@ -180,6 +193,12 @@ int module_adapter_prepare(struct comp_dev *dev)
 
 	comp_dbg(dev, "module_adapter_prepare() start");
 
+	ret = comp_verify_params(dev, mod->verify_params_flags, mod->stream_params);
+	if (ret < 0) {
+		comp_err(dev, "module_adapter_prepare(): comp_verify_params() failed.");
+		return ret;
+	}
+
 	/* Prepare module */
 	if (IS_PROCESSING_MODE_SINK_SOURCE(mod))
 		ret = module_adapter_sink_src_prepare(dev);
@@ -1185,9 +1204,10 @@ int module_adapter_reset(struct comp_dev *dev)
 		buffer_zero(buffer);
 	}
 
+#if CONFIG_IPC_MAJOR_3
 	rfree(mod->stream_params);
 	mod->stream_params = NULL;
-
+#endif
 	comp_dbg(dev, "module_adapter_reset(): done");
 
 	return comp_set_state(dev, COMP_TRIGGER_RESET);
@@ -1219,6 +1239,7 @@ void module_adapter_free(struct comp_dev *dev)
 
 #if CONFIG_IPC_MAJOR_4
 	rfree(mod->priv.cfg.input_pins);
+	rfree(mod->stream_params);
 #endif
 
 	rfree(mod);

src/ipc/ipc4/handler.c

@@ -155,46 +155,6 @@ __cold static int ipc4_delete_pipeline(struct ipc4_message_request *ipc4)
 	return ipc_pipeline_free(ipc, pipe->primary.r.instance_id);
 }
 
-static int ipc4_comp_params(struct comp_dev *current,
-			    struct comp_buffer *calling_buf,
-			    struct pipeline_walk_context *ctx, int dir)
-{
-	struct pipeline_data *ppl_data = ctx->comp_data;
-	int err;
-
-	/* don't do any params if current is running */
-	if (current->state == COMP_STATE_ACTIVE)
-		return 0;
-
-	/* Stay on the current pipeline */
-	if (current->pipeline != ((struct pipeline_data *)ctx->comp_data)->p)
-		return 0;
-
-	err = comp_params(current, &ppl_data->params->params);
-	if (err < 0 || err == PPL_STATUS_PATH_STOP)
-		return err;
-
-	return pipeline_for_each_comp(current, ctx, dir);
-}
-
-static int ipc4_pipeline_params(struct pipeline *p, struct comp_dev *host)
-{
-	struct sof_ipc_pcm_params hw_params = {{ 0 }};
-	struct pipeline_data data = {
-		.start = host,
-		.params = &hw_params,
-		.p = p,
-	};
-
-	struct pipeline_walk_context param_ctx = {
-		.comp_func = ipc4_comp_params,
-		.comp_data = &data,
-		.skip_incomplete = true,
-	};
-
-	return param_ctx.comp_func(host, NULL, &param_ctx, host->direction);
-}
-
 static int ipc4_pcm_params(struct ipc_comp_dev *pcm_dev)
 {
 	int err, reset_err;
@@ -205,15 +165,6 @@ static int ipc4_pcm_params(struct ipc_comp_dev *pcm_dev)
 		return -EINVAL;
 	}
 
-	/* configure pipeline audio params */
-	err = ipc4_pipeline_params(pcm_dev->cd->pipeline, pcm_dev->cd);
-	if (err < 0) {
-		ipc_cmd_err(&ipc_tr, "ipc: pipe %d comp %d params failed %d",
-			    pcm_dev->cd->pipeline->pipeline_id,
-			    pcm_dev->cd->pipeline->comp_id, err);
-		goto error;
-	}
-
 	/* prepare pipeline audio params */
 	err = pipeline_prepare(pcm_dev->cd->pipeline, pcm_dev->cd);
 	if (err < 0) {

src/samples/audio/detect_test.c

@@ -749,6 +749,7 @@ static struct comp_dev *test_keyword_new(const struct comp_driver *drv,
 	dev->direction = SOF_IPC_STREAM_CAPTURE;
 	dev->direction_set = true;
 	dev->state = COMP_STATE_READY;
+
 	return dev;
 
 cd_fail:
@@ -926,6 +927,7 @@ static int test_keyword_reset(struct comp_dev *dev)
 static int test_keyword_prepare(struct comp_dev *dev)
 {
 	struct comp_data *cd = comp_get_drvdata(dev);
+	struct sof_ipc_stream_params params;
 	uint16_t valid_bits = cd->sample_valid_bytes * 8;
 	uint16_t sample_width;
 	int ret;
@@ -938,6 +940,12 @@ static int test_keyword_prepare(struct comp_dev *dev)
 
 	comp_info(dev, "test_keyword_prepare()");
 
+	ret = test_keyword_params(dev, &params);
+	if (ret < 0) {
+		comp_err(dev, "test_keyword_prepare(): pcm params verification failed.");
+		return ret;
+	}
+
 	/*
 	 * FIXME: this condition is always "false" for IPC4 as audio format cannot be changed
 	 * without component re-creation. Does this "if" makes sense for IPC3?