diff options
author | Mans Rullgard <mans@mansr.com> | 2016-02-03 08:18:20 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-02-05 08:22:49 -0500 |
commit | 436e056c4ba368f13a5709a5a4a7f26fc238a5a6 (patch) | |
tree | 9a274499c8b25b2d751b09936fbde4105477a92a /sound/soc/mxs | |
parent | 92e963f50fc74041b5e9e744c330dca48e04f08d (diff) |
ASoC: mxs-saif: check BUSY bit in hw_params() only if not mclk_in_use
If something else, typically a codec, has enabled mclk, the BUSY
bit may be set when hw_params() is called without this being an
error. This check thus causes intermittent failures to configure
the sound device when used in such a manner. Fix this by making
the test conditional on !saif->mclk_in_use.
Signed-off-by: Mans Rullgard <mans@mansr.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/mxs')
-rw-r--r-- | sound/soc/mxs/mxs-saif.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/mxs/mxs-saif.c b/sound/soc/mxs/mxs-saif.c index c866ade28ad0..cd04eb7ee753 100644 --- a/sound/soc/mxs/mxs-saif.c +++ b/sound/soc/mxs/mxs-saif.c | |||
@@ -408,7 +408,7 @@ static int mxs_saif_hw_params(struct snd_pcm_substream *substream, | |||
408 | } | 408 | } |
409 | 409 | ||
410 | stat = __raw_readl(saif->base + SAIF_STAT); | 410 | stat = __raw_readl(saif->base + SAIF_STAT); |
411 | if (stat & BM_SAIF_STAT_BUSY) { | 411 | if (!saif->mclk_in_use && (stat & BM_SAIF_STAT_BUSY)) { |
412 | dev_err(cpu_dai->dev, "error: busy\n"); | 412 | dev_err(cpu_dai->dev, "error: busy\n"); |
413 | return -EBUSY; | 413 | return -EBUSY; |
414 | } | 414 | } |