diff options
| author | Koro Chen <koro.chen@mediatek.com> | 2015-12-03 10:53:28 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2015-12-08 20:06:45 +0300 |
| commit | 8d6f88ce961cf62137696627448cfd6038f07f41 (patch) | |
| tree | ba8aca5a3f6ad907cea24811bfe0515860ebef51 /sound/soc/mediatek/mtk-afe-common.h | |
| parent | b45e68df065a9babc43b4b7cd223c412d34b6658 (diff) | |
| download | linux-8d6f88ce961cf62137696627448cfd6038f07f41.tar.xz | |
ASoC: mediatek: Use current HW pointer for pointer callback
Previously we recorded "last interrupt position" and used it in
pointer callback. This is not correct implementation, and it causes
underruns when user space monitors buffer level to decide when to
send next data chunk in low latency application.
Remove position recording in IRQ handler and also hw_ptr in
struct mtk_afe_memif used to record that, and let pointer callback
reports current HW pointer instead.
Signed-off-by: Koro Chen <koro.chen@mediatek.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/mediatek/mtk-afe-common.h')
| -rw-r--r-- | sound/soc/mediatek/mtk-afe-common.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sound/soc/mediatek/mtk-afe-common.h b/sound/soc/mediatek/mtk-afe-common.h index cc4393cb1130..9b1af1a70874 100644 --- a/sound/soc/mediatek/mtk-afe-common.h +++ b/sound/soc/mediatek/mtk-afe-common.h @@ -92,7 +92,6 @@ struct mtk_afe_memif_data { struct mtk_afe_memif { unsigned int phys_buf_addr; int buffer_size; - unsigned int hw_ptr; /* Previous IRQ's HW ptr */ struct snd_pcm_substream *substream; const struct mtk_afe_memif_data *data; const struct mtk_afe_irq_data *irqdata; |