diff options
| author | Takashi Iwai <tiwai@suse.de> | 2016-03-10 23:03:09 +0300 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2016-04-26 10:55:42 +0300 |
| commit | 58a8738cfcdec389b3764a636303f97b57f85193 (patch) | |
| tree | 9161a56c1a4e2f588aa4117575d6cd4d544b5153 | |
| parent | d2c5cf88d5282de258f4eb6ab40040b80a075cd8 (diff) | |
| download | linux-58a8738cfcdec389b3764a636303f97b57f85193.tar.xz | |
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>
| -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 */ |