Commit 9cf3049e authored by Jeeja KP's avatar Jeeja KP Committed by Mark Brown

ASoC: Intel: Skylake: Fix return of skl_get_queue_index

In unbind modules, the skl_get_queue_index() can return error
if the pin is dynamic and module is not bound yet. So instead
of returning error this check should return success as modules
is not yet bound. This will let the module be bound when connected
pipes are enabled and will bind this as well.

So change the return value to 0
Signed-off-by: default avatarJeeja KP <jeeja.kp@intel.com>
Signed-off-by: default avatarVinod Koul <vinod.koul@intel.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 9ba8ffef
...@@ -688,14 +688,14 @@ int skl_unbind_modules(struct skl_sst *ctx, ...@@ -688,14 +688,14 @@ int skl_unbind_modules(struct skl_sst *ctx,
/* get src queue index */ /* get src queue index */
src_index = skl_get_queue_index(src_mcfg->m_out_pin, dst_id, out_max); src_index = skl_get_queue_index(src_mcfg->m_out_pin, dst_id, out_max);
if (src_index < 0) if (src_index < 0)
return -EINVAL; return 0;
msg.src_queue = src_index; msg.src_queue = src_index;
/* get dst queue index */ /* get dst queue index */
dst_index = skl_get_queue_index(dst_mcfg->m_in_pin, src_id, in_max); dst_index = skl_get_queue_index(dst_mcfg->m_in_pin, src_id, in_max);
if (dst_index < 0) if (dst_index < 0)
return -EINVAL; return 0;
msg.dst_queue = dst_index; msg.dst_queue = dst_index;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment