ASoC: Intel: Skylake: Don't pause stopped pipeline while deleting

If pipeline is not STARTED, we do not need to pause pipeline while
deleting.

Signed-off-by: Dharageswari R <dharageswari.r@intel.com>
Signed-off-by: Jeeja KP <jeeja.kp@intel.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>

1 files changed, 11 insertions, 9 deletions

diff --git a/sound/soc/intel/skylake/skl-messages.c b/sound/soc/intel/skylake/skl-messages.c
index 07d2a73ff207..804091aa6e64 100644
--- a/sound/soc/intel/skylake/skl-messages.c
+++ b/sound/soc/intel/skylake/skl-messages.c
@@ -1046,7 +1046,7 @@ int skl_delete_pipe(struct skl_sst *ctx, struct skl_pipe *pipe)
 
         dev_dbg(ctx->dev, "%s: pipe = %d\n", __func__, pipe->ppl_id);
 
-        /* If pipe is not started, do not try to stop the pipe in FW. */
+        /* If pipe is started, do stop the pipe in FW. */
         if (pipe->state > SKL_PIPE_STARTED) {
                 ret = skl_set_pipe_state(ctx, pipe, PPL_PAUSED);
                 if (ret < 0) {
@@ -1055,18 +1055,20 @@ int skl_delete_pipe(struct skl_sst *ctx, struct skl_pipe *pipe)
                 }
 
                 pipe->state = SKL_PIPE_PAUSED;
-        } else {
-                /* If pipe was not created in FW, do not try to delete it */
-                if (pipe->state < SKL_PIPE_CREATED)
-                        return 0;
+        }
 
-                ret = skl_ipc_delete_pipeline(&ctx->ipc, pipe->ppl_id);
-                if (ret < 0)
-                        dev_err(ctx->dev, "Failed to delete pipeline\n");
+        /* If pipe was not created in FW, do not try to delete it */
+        if (pipe->state < SKL_PIPE_CREATED)
+                return 0;
 
-                pipe->state = SKL_PIPE_INVALID;
+        ret = skl_ipc_delete_pipeline(&ctx->ipc, pipe->ppl_id);
+        if (ret < 0) {
+                dev_err(ctx->dev, "Failed to delete pipeline\n");
+                return ret;
         }
 
+        pipe->state = SKL_PIPE_INVALID;
+
         return ret;
 }