]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
ASoC: cirrus: i2s: IRQ-based stream watchdog
authorAlexander Sverdlin <alexander.sverdlin@gmail.com>
Sat, 28 Apr 2018 20:51:42 +0000 (22:51 +0200)
committerMark Brown <broonie@kernel.org>
Fri, 11 May 2018 02:27:33 +0000 (11:27 +0900)
commit98e1241c357b82e070294f486cef91a1263874fa
tree5761b94e4ab565df45ebd0e2031837b8580eb128
parent6ea8a84806ac91809e1759a200bf3c272b12aeb9
ASoC: cirrus: i2s: IRQ-based stream watchdog

I2S controller on EP93xx seems to have undocumented HW issue. According to
"EP93xx User’s Guide", controller can handle underflow and either transmit
last sample or zeroes in such case until FIFO is filled again. In reality
undeflow conditions seem to confuse internal state machine from time to
time and the whole stream gets shifted by one byte (as captured by logic
analyser on the I2S outputs). One could only hear noise instead of original
stream and this continues until the FIFO is disabled and enabled again.

Work this around by watching underflow interrupt and resetting I2S TX
channel + fill FIFO with zero samples until DMA catches up again. This is
a nasty workaround, but it works. Hence, Kconfig option to disable it in
case of problems.

Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/cirrus/Kconfig
sound/soc/cirrus/ep93xx-i2s.c