Commit 86331c5b authored by Takashi Iwai's avatar Takashi Iwai Committed by Ben Hutchings

ALSA: rme96: Fix unexpected volume reset after rate changes

commit a74a8216 upstream.

rme96 driver needs to reset DAC depending on the sample rate, and this
results in resetting to the max volume suddenly.  It's because of the
missing call of snd_rme96_apply_dac_volume().

However, calling this function right after the DAC reset still may not
work, and we need some delay before this call.  Since the DAC reset
and the procedure after that are performed in the spinlock, we delay
the DAC volume restore at the end after the spinlock.
Reported-and-tested-by: default avatarSylvain LABOISNE <maeda1@free.fr>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent a6d75fb2
...@@ -704,10 +704,11 @@ snd_rme96_playback_setrate(struct rme96 *rme96, ...@@ -704,10 +704,11 @@ snd_rme96_playback_setrate(struct rme96 *rme96,
{ {
/* change to/from double-speed: reset the DAC (if available) */ /* change to/from double-speed: reset the DAC (if available) */
snd_rme96_reset_dac(rme96); snd_rme96_reset_dac(rme96);
return 1; /* need to restore volume */
} else { } else {
writel(rme96->wcreg, rme96->iobase + RME96_IO_CONTROL_REGISTER); writel(rme96->wcreg, rme96->iobase + RME96_IO_CONTROL_REGISTER);
return 0;
} }
return 0;
} }
static int static int
...@@ -945,6 +946,7 @@ snd_rme96_playback_hw_params(struct snd_pcm_substream *substream, ...@@ -945,6 +946,7 @@ snd_rme96_playback_hw_params(struct snd_pcm_substream *substream,
struct rme96 *rme96 = snd_pcm_substream_chip(substream); struct rme96 *rme96 = snd_pcm_substream_chip(substream);
struct snd_pcm_runtime *runtime = substream->runtime; struct snd_pcm_runtime *runtime = substream->runtime;
int err, rate, dummy; int err, rate, dummy;
bool apply_dac_volume = false;
runtime->dma_area = (void __force *)(rme96->iobase + runtime->dma_area = (void __force *)(rme96->iobase +
RME96_IO_PLAY_BUFFER); RME96_IO_PLAY_BUFFER);
...@@ -958,24 +960,26 @@ snd_rme96_playback_hw_params(struct snd_pcm_substream *substream, ...@@ -958,24 +960,26 @@ snd_rme96_playback_hw_params(struct snd_pcm_substream *substream,
{ {
/* slave clock */ /* slave clock */
if ((int)params_rate(params) != rate) { if ((int)params_rate(params) != rate) {
spin_unlock_irq(&rme96->lock); err = -EIO;
return -EIO; goto error;
} }
} else if ((err = snd_rme96_playback_setrate(rme96, params_rate(params))) < 0) { } else {
spin_unlock_irq(&rme96->lock); err = snd_rme96_playback_setrate(rme96, params_rate(params));
return err; if (err < 0)
} goto error;
if ((err = snd_rme96_playback_setformat(rme96, params_format(params))) < 0) { apply_dac_volume = err > 0; /* need to restore volume later? */
spin_unlock_irq(&rme96->lock);
return err;
} }
err = snd_rme96_playback_setformat(rme96, params_format(params));
if (err < 0)
goto error;
snd_rme96_setframelog(rme96, params_channels(params), 1); snd_rme96_setframelog(rme96, params_channels(params), 1);
if (rme96->capture_periodsize != 0) { if (rme96->capture_periodsize != 0) {
if (params_period_size(params) << rme96->playback_frlog != if (params_period_size(params) << rme96->playback_frlog !=
rme96->capture_periodsize) rme96->capture_periodsize)
{ {
spin_unlock_irq(&rme96->lock); err = -EBUSY;
return -EBUSY; goto error;
} }
} }
rme96->playback_periodsize = rme96->playback_periodsize =
...@@ -986,9 +990,16 @@ snd_rme96_playback_hw_params(struct snd_pcm_substream *substream, ...@@ -986,9 +990,16 @@ snd_rme96_playback_hw_params(struct snd_pcm_substream *substream,
rme96->wcreg &= ~(RME96_WCR_PRO | RME96_WCR_DOLBY | RME96_WCR_EMP); rme96->wcreg &= ~(RME96_WCR_PRO | RME96_WCR_DOLBY | RME96_WCR_EMP);
writel(rme96->wcreg |= rme96->wcreg_spdif_stream, rme96->iobase + RME96_IO_CONTROL_REGISTER); writel(rme96->wcreg |= rme96->wcreg_spdif_stream, rme96->iobase + RME96_IO_CONTROL_REGISTER);
} }
err = 0;
error:
spin_unlock_irq(&rme96->lock); spin_unlock_irq(&rme96->lock);
if (apply_dac_volume) {
return 0; usleep_range(3000, 10000);
snd_rme96_apply_dac_volume(rme96);
}
return err;
} }
static int static int
......
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