On Fri, 05 May 2023 12:31:35 +0200, Oswald Buddenhagen wrote:
From: Jaroslav Kysela perex@perex.cz
The snd_pcm_playback_hw_avail() function uses runtime->status->hw_ptr. Unfortunately, in case when we call this function from snd_pcm_update_hw_ptr0(), this variable contains the previous hardware pointer. Use the new_hw_ptr argument to calculate hw_avail (filled samples by the user space) to correct the threshold comparison.
The new_hw_ptr argument may also be set to ULONG_MAX which means the initialization phase. In this case, use runtime->status->hw_ptr.
Suggested-by: Oswald Buddenhagen oswald.buddenhagen@gmx.de Signed-off-by: Jaroslav Kysela perex@perex.cz Reviewed-by: Oswald Buddenhagen oswald.buddenhagen@gmx.de
Here misses your Signed-off-by tag. Ditto for the patch 3. It's a legal requirement, and I can't apply patches without it.
Could you resubmit the series quickly with your SOB?
thanks,
Takashi