diff options
author | Takashi Iwai <tiwai@suse.de> | 2016-03-10 21:03:09 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2016-04-26 09:55:42 +0200 |
commit | 58a8738cfcdec389b3764a636303f97b57f85193 (patch) | |
tree | 9161a56c1a4e2f588aa4117575d6cd4d544b5153 /sound/pci/au88x0 | |
parent | ALSA: hrtimer: Handle start/stop more properly (diff) | |
download | linux-58a8738cfcdec389b3764a636303f97b57f85193.tar.xz linux-58a8738cfcdec389b3764a636303f97b57f85193.zip |
ALSA: au88x0: Fix overlapped PCM pointer
au88x0 hardware seems returning the current pointer at the buffer
boundary instead of going back to zero. This results in spewing
warnings from PCM core.
This patch corrects the return value from the pointer callback within
the proper value range, just returning zero if the position is equal
or above the buffer size.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/au88x0')
-rw-r--r-- | sound/pci/au88x0/au88x0_pcm.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sound/pci/au88x0/au88x0_pcm.c b/sound/pci/au88x0/au88x0_pcm.c index a6d6d8d0867a..df5741a78fd2 100644 --- a/sound/pci/au88x0/au88x0_pcm.c +++ b/sound/pci/au88x0/au88x0_pcm.c @@ -432,7 +432,10 @@ static snd_pcm_uframes_t snd_vortex_pcm_pointer(struct snd_pcm_substream *substr #endif //printk(KERN_INFO "vortex: pointer = 0x%x\n", current_ptr); spin_unlock(&chip->lock); - return (bytes_to_frames(substream->runtime, current_ptr)); + current_ptr = bytes_to_frames(substream->runtime, current_ptr); + if (current_ptr >= substream->runtime->buffer_size) + current_ptr = 0; + return current_ptr; } /* operators */ |