Commit fc99d23f authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown

ASoC: rsnd: tidyup src->convert_rate reset timing

Current src->convert_rate will be set on .hw_param, and
be reset on .quit timing.
But, .hw_param will not be called again if user did Ctrl-Z + fg.
It should be reset on initial of .hw_param to keep its value.
Here, ctu.c already do this.
This patch solves this issue, other wise, MIXed sound will be
strange if user did like below.

	> aplay -D plughw:0,0 sound_44100.wav &
	> aplay -D plughw:0,1 sound_96000.wav
	> Ctrl-Z
	> fg # 96kHz will be played as 44.1kHz
Reported-by: default avatarHiroyuki Yokoyama <hiroyuki.yokoyama.vx@renesas.com>
Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tested-by: default avatarHiroyuki Yokoyama <hiroyuki.yokoyama.vx@renesas.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent b5aac5a9
...@@ -167,6 +167,7 @@ static int rsnd_src_hw_params(struct rsnd_mod *mod, ...@@ -167,6 +167,7 @@ static int rsnd_src_hw_params(struct rsnd_mod *mod,
* dpcm_fe_dai_hw_params() * dpcm_fe_dai_hw_params()
* dpcm_be_dai_hw_params() * dpcm_be_dai_hw_params()
*/ */
src->convert_rate = 0;
if (fe->dai_link->dynamic) { if (fe->dai_link->dynamic) {
int stream = substream->stream; int stream = substream->stream;
struct snd_soc_dpcm *dpcm; struct snd_soc_dpcm *dpcm;
...@@ -414,8 +415,6 @@ static int rsnd_src_quit(struct rsnd_mod *mod, ...@@ -414,8 +415,6 @@ static int rsnd_src_quit(struct rsnd_mod *mod,
rsnd_mod_power_off(mod); rsnd_mod_power_off(mod);
src->convert_rate = 0;
/* reset sync convert_rate */ /* reset sync convert_rate */
src->sync.val = 0; src->sync.val = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment