Hi Jiada Thank you for your patch
> Currently there is race condition between set of byte_pos and wrap > it around when new buffer starts. If .pointer is called in-between > it will result in inconsistent pointer position be returned > from .pointer callback. > > This patch increments buffer pointer atomically to avoid this issue. > > Signed-off-by: Jiada Wang <jiada_w...@mentor.com> > Reviewed-by: Takashi Sakamoto <takashi.sakam...@miraclelinux.com> > --- You using playback with PIO mode ? Because this function is no longer used on DMA mode Best regards --- Kuninori Morimoto