dmaengine: pl330: Really fix choppy sound because of wrong residue calculation 42/42442/1
authorKrzysztof Koz?owski <k.kozlowski.k@gmail.com>
Mon, 15 Jun 2015 14:00:09 +0000 (23:00 +0900)
committerInha Song <ideal.song@samsung.com>
Mon, 29 Jun 2015 08:02:01 +0000 (17:02 +0900)
commit41dec9884a82443da1b7814ae42efe65895ff3af
tree319cadc61667f6c094a7defbe27bea681872e810
parentad533b7c467aa4117cde97bdd9b86e98e35c8741
dmaengine: pl330: Really fix choppy sound because of wrong residue calculation

When pl330 driver was used during sound playback, after some time or
after a number of plays the sound became choppy or totally noisy. For
example on Odroid XU3 board the first four executions of aplay with
small WAVE worked fine, but fifth was unrecognizable with errors:
$ aplay /usr/share/sounds/alsa/Front_Right.wava
underrun!!! (at least 0.095 ms long)

Issue was caused by wrong residue reported by pl330 driver to
pcm_dmaengine for its cyclic dma transfers.

The pl330_tx_status(), residue reporting function, used a "last" flag in
a descriptor to indicate that there is no more data to send.

The pl330_tx_submit() iterated over descriptors trying to remove this
flag from them and then mark last descriptor as "last".  However when
iterating it actually removed the flag not from descriptors but always
from last of it (and then reset it). Thus effectively once some
descriptor was marked as last, then it stayed like this forever causing
residue to be reported too low.

Change-Id: I4283522a0306d91639548b50df446bb91912a0d5
Signed-off-by: Krzysztof Kozlowski <k.kozlowski.k@gmail.com>
Fixes: aee4d1fac887 ("dmaengine: pl330: improve pl330_tx_status() function")
Cc: <stable@vger.kernel.org>
Reported-by: gabriel@unseen.is
Suggested-by: Marek Szyprowski <m.szyprowski@samsung.com>
drivers/dma/pl330.c