diff options
author | Vinod Koul <vinod.koul@intel.com> | 2016-07-26 08:36:45 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-08-08 06:54:59 -0400 |
commit | e280823c23f9ccc5f3bbd0fb2fa458cd96c1a881 (patch) | |
tree | dbd0a49b9e5a9ff34aec5c16c4b4d4ea91bf08dd | |
parent | a8e2c19efd5d27577661124b1033b344650e4899 (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.c | 16 |
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 */ |