aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVinod Koul <vinod.koul@intel.com>2016-07-26 08:36:45 -0400
committerMark Brown <broonie@kernel.org>2016-08-08 06:54:59 -0400
commite280823c23f9ccc5f3bbd0fb2fa458cd96c1a881 (patch)
treedbd0a49b9e5a9ff34aec5c16c4b4d4ea91bf08dd
parenta8e2c19efd5d27577661124b1033b344650e4899 (diff)
ASoC: Intel: Skylake: Parse UUIDs once
The firmware manifest contains UUIDs which needs to be passed only once. So use the newly introduced is_first_boot flag to distinguish and parse these only once. Signed-off-by: Vinod Koul <vinod.koul@intel.com> Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r--sound/soc/intel/skylake/skl-sst.c16
1 files changed, 9 insertions, 7 deletions
diff --git a/sound/soc/intel/skylake/skl-sst.c b/sound/soc/intel/skylake/skl-sst.c
index 6e9c634cf84f..064fc7ee3d88 100644
--- a/sound/soc/intel/skylake/skl-sst.c
+++ b/sound/soc/intel/skylake/skl-sst.c
@@ -88,13 +88,15 @@ static int skl_load_base_firmware(struct sst_dsp *ctx)
88 } 88 }
89 } 89 }
90 90
91 ret = snd_skl_parse_uuids(ctx, ctx->fw, SKL_ADSP_FW_BIN_HDR_OFFSET, 0); 91 /* prase uuids on first boot */
92 if (ret < 0) { 92 if (skl->is_first_boot) {
93 dev_err(ctx->dev, 93 ret = snd_skl_parse_uuids(ctx, ctx->fw, SKL_ADSP_FW_BIN_HDR_OFFSET, 0);
94 "UUID parsing err: %d\n", ret); 94 if (ret < 0) {
95 release_firmware(ctx->fw); 95 dev_err(ctx->dev, "UUID parsing err: %d\n", ret);
96 skl_dsp_disable_core(ctx, SKL_DSP_CORE0_MASK); 96 release_firmware(ctx->fw);
97 return ret; 97 skl_dsp_disable_core(ctx, SKL_DSP_CORE0_MASK);
98 return ret;
99 }
98 } 100 }
99 101
100 /* check for extended manifest */ 102 /* check for extended manifest */