From 5aac660e0281494932bd7375891a3e21685edbe3 Mon Sep 17 00:00:00 2001 From: Joe Cheng Date: Wed, 25 Oct 2023 14:49:13 +0800 Subject: [PATCH] dts : add more error logs in set_configuration() Add more error logs to identify potential problem in set_configuration() Co-developed-by: Mac Chiang Signed-off-by: Mac Chiang Signed-off-by: Joe Cheng --- src/audio/module_adapter/module/dts/dts.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/audio/module_adapter/module/dts/dts.c b/src/audio/module_adapter/module/dts/dts.c index 8664c9cb1..78ca727b0 100644 --- a/src/audio/module_adapter/module/dts/dts.c +++ b/src/audio/module_adapter/module/dts/dts.c @@ -427,17 +427,24 @@ dts_codec_set_configuration(struct processing_module *mod, uint32_t config_id, ret = module_set_configuration(mod, config_id, pos, data_offset_size, fragment, fragment_size, response, response_size); - if (ret < 0) + if (ret < 0) { + comp_err(dev, "dts_codec_set_configuration(): error %x from module_set_configuration()", + ret); return ret; + } /* return if more fragments are expected */ - if (pos != MODULE_CFG_FRAGMENT_LAST && pos != MODULE_CFG_FRAGMENT_SINGLE) + if (pos != MODULE_CFG_FRAGMENT_LAST && pos != MODULE_CFG_FRAGMENT_SINGLE) { + comp_err(dev, "dts_codec_set_configuration(): pos %d error", pos); return 0; + } #if CONFIG_IPC_MAJOR_3 // return if the module is not prepared - if (md->state < MODULE_INITIALIZED) + if (md->state < MODULE_INITIALIZED) { + comp_err(dev, "dts_codec_set_configuration(): state %d error", md->state); return 0; + } #endif /* whole configuration received, apply it now */