ASoC: Intel: Skylake: Fix DSP core ref count for init failure
authorSubhransu S. Prusty <subhransu.s.prusty@intel.com>
Tue, 22 Aug 2017 11:15:53 +0000 (16:45 +0530)
committerMark Brown <broonie@kernel.org>
Wed, 23 Aug 2017 10:55:18 +0000 (11:55 +0100)
During dsp init failure, the ref count is not incremented and dsp is
powered down. But as the skl driver calls put_core for the init failure it
decrements the dsp core ref count and ref count becomes unbalanced.

This results in dsp core powered up in further runtime suspend/resume
cycles and never powered down.

So increment the ref count before dsp core powerup and for any failure,
decrement in put_core will be balanced.

Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty@intel.com>
Acked-By: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/skylake/skl-sst-dsp.c

index c2ba5ff1632c79fa7bd2b07784cbe178568f8a0a..19ee1d4f3bdff5b2ff5973d702c9ddf714de38ea 100644 (file)
@@ -351,6 +351,8 @@ int skl_dsp_get_core(struct sst_dsp *ctx, unsigned int core_id)
                return -EINVAL;
        }
 
+       skl->cores.usage_count[core_id]++;
+
        if (skl->cores.state[core_id] == SKL_DSP_RESET) {
                ret = ctx->fw_ops.set_state_D0(ctx, core_id);
                if (ret < 0) {
@@ -359,8 +361,6 @@ int skl_dsp_get_core(struct sst_dsp *ctx, unsigned int core_id)
                }
        }
 
-       skl->cores.usage_count[core_id]++;
-
 out:
        dev_dbg(ctx->dev, "core id %d state %d usage_count %d\n",
                        core_id, skl->cores.state[core_id],