diff options
author | Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> | 2017-10-31 01:40:32 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-10-31 12:06:52 +0100 |
commit | ce548931207c0d0059bd90171e2c458f897354d7 (patch) | |
tree | c8c5526be0b504c3191a3915ac02503cbd1944dd /sound/soc/sh/rcar/ssi.c | |
parent | ASoC: rsnd: use snd_pcm_running() in rsnd_io_is_working() (diff) | |
download | linux-ce548931207c0d0059bd90171e2c458f897354d7.tar.xz linux-ce548931207c0d0059bd90171e2c458f897354d7.zip |
ASoC: rsnd: Don't check SSISR::DIRQ when Capture
When stop case, it was Playback, it need to check all data were
completely sent. But in Capture case, it might not receive data
anymore. SSISR::DIRQ check is not need for Capture case.
Reported-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx@renesas.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/sh/rcar/ssi.c')
-rw-r--r-- | sound/soc/sh/rcar/ssi.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/sound/soc/sh/rcar/ssi.c b/sound/soc/sh/rcar/ssi.c index 61b4ce20ee2b..58e3420a1f05 100644 --- a/sound/soc/sh/rcar/ssi.c +++ b/sound/soc/sh/rcar/ssi.c @@ -598,15 +598,18 @@ static int rsnd_ssi_stop(struct rsnd_mod *mod, if (rsnd_ssi_is_parent(mod, io)) return 0; - /* - * disable all IRQ, - * and, wait all data was sent - */ cr = ssi->cr_own | ssi->cr_clk; - rsnd_mod_write(mod, SSICR, cr | EN); - rsnd_ssi_status_check(mod, DIRQ); + /* + * disable all IRQ, + * Playback: Wait all data was sent + * Capture: It might not receave data. Do nothing + */ + if (rsnd_io_is_play(io)) { + rsnd_mod_write(mod, SSICR, cr | EN); + rsnd_ssi_status_check(mod, DIRQ); + } /* * disable SSI, |