[do not merge] Audio: Up_down_mixer: Traces add

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

Author: singalsu

src/audio/up_down_mixer/up_down_mixer.c

@@ -47,6 +47,8 @@ static int set_downmix_coefficients(struct processing_module *mod,
 	struct comp_dev *dev = mod->dev;
 	int ret;
 
+	comp_info(dev, "set_downmix_coefficients()");
+
 	if (cd->downmix_coefficients) {
 		ret = memcpy_s(&custom_coeffs, sizeof(custom_coeffs), downmix_coefficients,
 			       sizeof(int32_t) * UP_DOWN_MIX_COEFFS_LENGTH);
@@ -110,6 +112,8 @@ static int set_downmix_coefficients(struct processing_module *mod,
 static up_down_mixer_routine select_mix_out_stereo(struct comp_dev *dev,
 						   const struct ipc4_audio_format *format)
 {
+	comp_info(dev, "select_mix_out_stereo()");
+
 	if (format->depth == IPC4_DEPTH_16BIT) {
 		switch (format->ch_cfg) {
 		case IPC4_CHANNEL_CONFIG_MONO:
@@ -175,6 +179,8 @@ static up_down_mixer_routine select_mix_out_stereo(struct comp_dev *dev,
 static up_down_mixer_routine select_mix_out_mono(struct comp_dev *dev,
 						 const struct ipc4_audio_format *format)
 {
+	comp_info(dev, "select_mix_out_mono(");
+
 	if (format->depth == IPC4_DEPTH_16BIT) {
 		switch (format->ch_cfg) {
 		case IPC4_CHANNEL_CONFIG_STEREO:
@@ -226,6 +232,8 @@ static up_down_mixer_routine select_mix_out_mono(struct comp_dev *dev,
 static up_down_mixer_routine select_mix_out_5_1(struct comp_dev *dev,
 						const struct ipc4_audio_format *format)
 {
+	comp_info(dev, "select_mix_out_5_1()");
+
 	if (format->depth == IPC4_DEPTH_16BIT) {
 		switch (format->ch_cfg) {
 		case IPC4_CHANNEL_CONFIG_MONO:
@@ -265,6 +273,8 @@ static int init_mix(struct processing_module *mod,
 	struct up_down_mixer_data *cd = module_get_private_data(mod);
 	struct comp_dev *dev = mod->dev;
 
+	comp_info(dev, "init_mix()");
+
 	if (!format)
 		return -EINVAL;
 
@@ -326,6 +336,8 @@ static int up_down_mixer_free(struct processing_module *mod)
 {
 	struct up_down_mixer_data *cd = module_get_private_data(mod);
 
+	comp_info(mod->dev, "up_down_mixer_free()");
+
 	rfree(cd->buf_in);
 	rfree(cd->buf_out);
 	rfree(cd);
@@ -372,14 +384,10 @@ static int up_down_mixer_init(struct processing_module *mod)
 		  dev->ipc_config.ipc_config_size, min_size);
 
 	if (dev->ipc_config.ipc_config_size > min_size) {
+		comp_info(dev, "init configuration found");
 		cd->has_init_mix_configuration = true;
-		comp_info(dev, "init data contains up_down_mixer configuration");
-		comp_info(dev, "out_channel_config = %d", up_down_mixer->out_channel_config);
-		comp_info(dev, "coefficients_select = %d", up_down_mixer->coefficients_select);
-		comp_info(dev, "channel_map = %d", up_down_mixer->channel_map);
-		for (i = 0; i < UP_DOWN_MIX_COEFFS_LENGTH; i++)
-			comp_info(dev, "coef[%d] = %d", i, up_down_mixer->coefficients[i]);
 	} else {
+		comp_info(dev, "no configuration in init, using default");
 		cd->has_init_mix_configuration = false;
 #if FORCE_REQUEST_MONO
 		default_config.out_channel_config = 0;
@@ -390,31 +398,51 @@ static int up_down_mixer_init(struct processing_module *mod)
 #else
 		default_config.out_channel_config = up_down_mixer_init->base_cfg.audio_fmt.ch_cfg;
 		default_config.coefficients_select = DEFAULT_COEFFICIENTS;
+		for (i = 0; i < UP_DOWN_MIX_COEFFS_LENGTH; i++)
+			default_config.coefficients[i] = 0;
 #endif
 		up_down_mixer = &default_config;
-		comp_info(dev, "no up_down_mixer configuration in init, using default");
 	}
 
+	comp_info(dev, "init data: %d, %d, 0x%08x", up_down_mixer->out_channel_config,
+		  up_down_mixer->coefficients_select, up_down_mixer->channel_map);
+#if UP_DOWN_MIX_COEFFS_LENGTH == 8
+	comp_info(dev, "coef[0..3]: %d, %d, %d, %d",
+		  up_down_mixer->coefficients[0], up_down_mixer->coefficients[1],
+		  up_down_mixer->coefficients[2], up_down_mixer->coefficients[3]);
+	comp_info(dev, "coef[4..7]: %d, %d, %d, %d",
+		  up_down_mixer->coefficients[4], up_down_mixer->coefficients[5],
+		  up_down_mixer->coefficients[6], up_down_mixer->coefficients[7]);
+#endif
+
 	switch (up_down_mixer->coefficients_select) {
 	case DEFAULT_COEFFICIENTS:
 		cd->out_channel_map = create_channel_map(up_down_mixer->out_channel_config);
+		comp_info(dev, "c1 out_channel map = %d", cd->out_channel_map);
 		ret = init_mix(mod, &mod->priv.cfg.base_cfg.audio_fmt,
 			       up_down_mixer->out_channel_config, NULL);
+		comp_info(dev, "c1 init_mix() done");
 		break;
 	case CUSTOM_COEFFICIENTS:
 		cd->out_channel_map = create_channel_map(up_down_mixer->out_channel_config);
+		comp_info(dev, "c2 out_channel map = %d", cd->out_channel_map);
 		ret = init_mix(mod, &mod->priv.cfg.base_cfg.audio_fmt,
 			       up_down_mixer->out_channel_config, up_down_mixer->coefficients);
+		comp_info(dev, "c2 init_mix() done");
 		break;
 	case DEFAULT_COEFFICIENTS_WITH_CHANNEL_MAP:
 		cd->out_channel_map = up_down_mixer->channel_map;
+		comp_info(dev, "c3 out_channel map = %d", cd->out_channel_map);
 		ret = init_mix(mod, &mod->priv.cfg.base_cfg.audio_fmt,
 			       up_down_mixer->out_channel_config, NULL);
+		comp_info(dev, "c3 init_mix() done");
 		break;
 	case CUSTOM_COEFFICIENTS_WITH_CHANNEL_MAP:
 		cd->out_channel_map = up_down_mixer->channel_map;
+		comp_info(dev, "c4 out_channel map = %d", cd->out_channel_map);
 		ret = init_mix(mod, &mod->priv.cfg.base_cfg.audio_fmt,
 			       up_down_mixer->out_channel_config, up_down_mixer->coefficients);
+		comp_info(dev, "c4 init_mix() done");
 		break;
 	default:
 		comp_err(dev, "up_down_mixer_init(): unsupported coefficient type");
@@ -427,9 +455,11 @@ static int up_down_mixer_init(struct processing_module *mod)
 		goto err;
 	}
 
+	comp_info(dev, "up_down_mixer_init() ready");
 	return 0;
 
 err:
+	comp_err(dev, "up_down_mixer_init() error");
 	up_down_mixer_free(mod);
 	return ret;
 }