audio: dai: mark non-critical code as "cold"

Add __cold qualifiers to IPC context functions.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>

Author: lyakh

src/audio/dai-zephyr.c

@@ -76,8 +76,10 @@ static void dai_atomic_trigger(void *arg, enum notify_id type, void *data)
 }
 
 /* Assign DAI to a group */
-int dai_assign_group(struct dai_data *dd, struct comp_dev *dev, uint32_t group_id)
+__cold int dai_assign_group(struct dai_data *dd, struct comp_dev *dev, uint32_t group_id)
 {
+	assert_can_be_cold();
+
 	if (dd->group) {
 		if (dd->group->group_id != group_id) {
 			comp_err(dev, "DAI already in group %d, requested %d",
@@ -135,15 +137,17 @@ static int dai_trigger_op(struct dai *dai, int cmd, int direction)
 }
 
 /* called from src/ipc/ipc3/handler.c and src/ipc/ipc4/dai.c */
-int dai_set_config(struct dai *dai, struct ipc_config_dai *common_config,
-		   const void *spec_config)
+__cold int dai_set_config(struct dai *dai, struct ipc_config_dai *common_config,
+			  const void *spec_config)
 {
 	const struct device *dev = dai->dev;
 	const struct sof_ipc_dai_config *sof_cfg = spec_config;
 	struct dai_config cfg;
 	const void *cfg_params;
 	bool is_blob;
 
+	assert_can_be_cold();
+
 	cfg.dai_index = common_config->dai_index;
 	is_blob = common_config->is_config_blob;
 	cfg.format = sof_cfg->format;
@@ -467,11 +471,13 @@ dai_dma_multi_endpoint_cb(struct dai_data *dd, struct comp_dev *dev, uint32_t fr
 	return dma_status;
 }
 
-int dai_common_new(struct dai_data *dd, struct comp_dev *dev,
-		   const struct ipc_config_dai *dai_cfg)
+__cold int dai_common_new(struct dai_data *dd, struct comp_dev *dev,
+			  const struct ipc_config_dai *dai_cfg)
 {
 	uint32_t dir;
 
+	assert_can_be_cold();
+
 	dd->dai = dai_get(dai_cfg->type, dai_cfg->dai_index, DAI_CREAT);
 	if (!dd->dai) {
 		comp_err(dev, "dai_get() failed to create DAI.");
@@ -542,15 +548,17 @@ int dai_common_new(struct dai_data *dd, struct comp_dev *dev,
 	return 0;
 }
 
-static struct comp_dev *dai_new(const struct comp_driver *drv,
-				const struct comp_ipc_config *config,
-				const void *spec)
+__cold static struct comp_dev *dai_new(const struct comp_driver *drv,
+				       const struct comp_ipc_config *config,
+				       const void *spec)
 {
 	struct comp_dev *dev;
 	const struct ipc_config_dai *dai_cfg = spec;
 	struct dai_data *dd;
 	int ret;
 
+	assert_can_be_cold();
+
 	comp_cl_dbg(&comp_dai, "dai_new()");
 
 	dev = comp_alloc(drv, sizeof(*dev));
@@ -582,8 +590,10 @@ static struct comp_dev *dai_new(const struct comp_driver *drv,
 	return NULL;
 }
 
-void dai_common_free(struct dai_data *dd)
+__cold void dai_common_free(struct dai_data *dd)
 {
+	assert_can_be_cold();
+
 #ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
 	io_perf_monitor_release_slot(dd->io_perf_bytes_count);
 #endif
@@ -605,10 +615,12 @@ void dai_common_free(struct dai_data *dd)
 	rfree(dd->dai_spec_config);
 }
 
-static void dai_free(struct comp_dev *dev)
+__cold static void dai_free(struct comp_dev *dev)
 {
 	struct dai_data *dd = comp_get_drvdata(dev);
 
+	assert_can_be_cold();
+
 	if (dd->group)
 		notifier_unregister(dev, dd->group, NOTIFIER_ID_DAI_TRIGGER);
 
@@ -619,7 +631,7 @@ static void dai_free(struct comp_dev *dev)
 }
 
 int dai_common_get_hw_params(struct dai_data *dd, struct comp_dev *dev,
-			     struct sof_ipc_stream_params *params, int dir)
+				    struct sof_ipc_stream_params *params, int dir)
 {
 	struct dai_config cfg;
 	int ret;
@@ -645,12 +657,14 @@ int dai_common_get_hw_params(struct dai_data *dd, struct comp_dev *dev,
 	return ret;
 }
 
-static int dai_comp_get_hw_params(struct comp_dev *dev,
-				  struct sof_ipc_stream_params *params,
-				  int dir)
+__cold static int dai_comp_get_hw_params(struct comp_dev *dev,
+					 struct sof_ipc_stream_params *params,
+					 int dir)
 {
 	struct dai_data *dd = comp_get_drvdata(dev);
 
+	assert_can_be_cold();
+
 	return dai_common_get_hw_params(dd, dev, params, dir);
 }
 
@@ -1901,11 +1915,13 @@ static uint64_t dai_get_processed_data(struct comp_dev *dev, uint32_t stream_no,
 }
 
 #ifdef CONFIG_IPC_MAJOR_4
-int dai_zephyr_unbind(struct dai_data *dd, struct comp_dev *dev, void *data)
+__cold int dai_zephyr_unbind(struct dai_data *dd, struct comp_dev *dev, void *data)
 {
 	struct ipc4_module_bind_unbind *bu;
 	int buf_id;
 
+	assert_can_be_cold();
+
 	bu = (struct ipc4_module_bind_unbind *)data;
 	buf_id = IPC4_COMP_ID(bu->extension.r.src_queue, bu->extension.r.dst_queue);
 

src/ipc/ipc4/dai.c

@@ -12,6 +12,7 @@
 #include <rtos/idc.h>
 #include <rtos/alloc.h>
 #include <sof/lib/dai.h>
+#include <sof/lib/memory.h>
 #include <sof/lib/notifier.h>
 #include <sof/platform.h>
 #include <rtos/sof.h>
@@ -331,13 +332,15 @@ static int dai_init_llp_info(struct dai_data *dd, struct comp_dev *dev)
 	return 0;
 }
 
-int dai_config(struct dai_data *dd, struct comp_dev *dev, struct ipc_config_dai *common_config,
-	       const void *spec_config)
+__cold int dai_config(struct dai_data *dd, struct comp_dev *dev,
+		      struct ipc_config_dai *common_config, const void *spec_config)
 {
 	const struct ipc4_copier_module_cfg *copier_cfg = spec_config;
 	int size;
 	int ret;
 
+	assert_can_be_cold();
+
 	/* ignore if message not for this DAI id/type */
 	if (dd->ipc_config.dai_index != common_config->dai_index ||
 	    dd->ipc_config.type != common_config->type)

src/lib/dai.c

@@ -34,10 +34,12 @@ struct dai_group_list {
 
 static struct dai_group_list *groups[CONFIG_CORE_COUNT];
 
-static struct dai_group_list *dai_group_list_get(int core_id)
+__cold static struct dai_group_list *dai_group_list_get(int core_id)
 {
 	struct dai_group_list *group_list = groups[core_id];
 
+	assert_can_be_cold();
+
 	if (!group_list) {
 		group_list = rzalloc(SOF_MEM_ZONE_SYS, 0, SOF_MEM_CAPS_RAM,
 				     sizeof(*group_list));
@@ -49,12 +51,14 @@ static struct dai_group_list *dai_group_list_get(int core_id)
 	return group_list;
 }
 
-static struct dai_group *dai_group_find(uint32_t group_id)
+__cold static struct dai_group *dai_group_find(uint32_t group_id)
 {
 	struct list_item *dai_groups;
 	struct list_item *group_item;
 	struct dai_group *group = NULL;
 
+	assert_can_be_cold();
+
 	dai_groups = &dai_group_list_get(cpu_get_id())->list;
 
 	list_for_item(group_item, dai_groups) {
@@ -69,11 +73,13 @@ static struct dai_group *dai_group_find(uint32_t group_id)
 	return group;
 }
 
-static struct dai_group *dai_group_alloc()
+__cold static struct dai_group *dai_group_alloc(void)
 {
 	struct list_item *dai_groups = &dai_group_list_get(cpu_get_id())->list;
 	struct dai_group *group;
 
+	assert_can_be_cold();
+
 	group = rzalloc(SOF_MEM_ZONE_SYS, 0, SOF_MEM_CAPS_RAM,
 			sizeof(*group));
 
@@ -82,10 +88,12 @@ static struct dai_group *dai_group_alloc()
 	return group;
 }
 
-struct dai_group *dai_group_get(uint32_t group_id, uint32_t flags)
+__cold struct dai_group *dai_group_get(uint32_t group_id, uint32_t flags)
 {
 	struct dai_group *group;
 
+	assert_can_be_cold();
+
 	if (!group_id) {
 		tr_err(&dai_tr, "dai_group_get(): invalid group_id %u",
 		       group_id);
@@ -117,8 +125,10 @@ struct dai_group *dai_group_get(uint32_t group_id, uint32_t flags)
 	return group;
 }
 
-void dai_group_put(struct dai_group *group)
+__cold void dai_group_put(struct dai_group *group)
 {
+	assert_can_be_cold();
+
 	group->num_dais--;
 
 	/* Mark as unused if there are no more DAIs in this group */