video: tegra: dtv: Remove check for dma req list before cancelling
Laxman Dewangan [Fri, 6 Apr 2012 13:17:06 +0000 (18:17 +0530)]
Removing the checking whether dma req queue is empty or not
before canceling/stopping dma.
This function cannot guarantee correct result as there may be the
race between hw and sw status update.
If client wants to cancel dma req, it can directly call tegra_dma_cancel().

As the function tegra_dma_is_empty() does not return correct result,
it will be depreciated from the dma apis.

Change-Id: I3f5ebee8a7be345b02fcd8a69530c9b53207ef28
Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
Reviewed-on: http://git-master/r/95090
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Bitan Biswas <bbiswas@nvidia.com>

drivers/media/video/tegra/tegra_dtv.c

index 95270c4..8a53930 100644 (file)
@@ -390,9 +390,6 @@ static void __force_xfer_stop(struct dtv_stream *s)
                }
        }
 
-       /* just in case. dma should be cancelled before this */
-       if (!tegra_dma_is_empty(s->dma_chan))
-               pr_err("%s: DMA channel is not empty!\n", __func__);
        tegra_dma_cancel(s->dma_chan);
        s->xferring = false;