ALSA: au88x0: Fix overlapped PCM pointer
authorTakashi Iwai <tiwai@suse.de>
Thu, 10 Mar 2016 20:03:09 +0000 (21:03 +0100)
committerTakashi Iwai <tiwai@suse.de>
Tue, 26 Apr 2016 07:55:42 +0000 (09:55 +0200)
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>
sound/pci/au88x0/au88x0_pcm.c

index a6d6d8d..df5741a 100644 (file)
@@ -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 */