* [PATCH V1] arm: tegra: dma: not required to move requestor when stopping.
@ 2012-01-24 8:10 Laxman Dewangan
2012-01-24 20:57 ` Stephen Warren
0 siblings, 1 reply; 4+ messages in thread
From: Laxman Dewangan @ 2012-01-24 8:10 UTC (permalink / raw)
To: ccross, olof, swarren, linux
Cc: linux-tegra, linux-arm-kernel, linux-kernel, Laxman Dewangan
It is not require to move the requestor of dma to INVALID
option before stopping dma.
Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
---
arch/arm/mach-tegra/dma.c | 8 --------
1 files changed, 0 insertions(+), 8 deletions(-)
diff --git a/arch/arm/mach-tegra/dma.c b/arch/arm/mach-tegra/dma.c
index 998c55d..abea4f6 100644
--- a/arch/arm/mach-tegra/dma.c
+++ b/arch/arm/mach-tegra/dma.c
@@ -52,8 +52,6 @@
#define CSR_ONCE (1<<27)
#define CSR_FLOW (1<<21)
#define CSR_REQ_SEL_SHIFT 16
-#define CSR_REQ_SEL_MASK (0x1F<<CSR_REQ_SEL_SHIFT)
-#define CSR_REQ_SEL_INVALID (31<<CSR_REQ_SEL_SHIFT)
#define CSR_WCOUNT_SHIFT 2
#define CSR_WCOUNT_MASK 0xFFFC
@@ -183,18 +181,12 @@ static void tegra_dma_stop(struct tegra_dma_channel *ch)
static int tegra_dma_cancel(struct tegra_dma_channel *ch)
{
- u32 csr;
unsigned long irq_flags;
spin_lock_irqsave(&ch->lock, irq_flags);
while (!list_empty(&ch->list))
list_del(ch->list.next);
- csr = readl(ch->addr + APB_DMA_CHAN_CSR);
- csr &= ~CSR_REQ_SEL_MASK;
- csr |= CSR_REQ_SEL_INVALID;
- writel(csr, ch->addr + APB_DMA_CHAN_CSR);
-
tegra_dma_stop(ch);
spin_unlock_irqrestore(&ch->lock, irq_flags);
--
1.7.1.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* RE: [PATCH V1] arm: tegra: dma: not required to move requestor when stopping.
2012-01-24 8:10 [PATCH V1] arm: tegra: dma: not required to move requestor when stopping Laxman Dewangan
@ 2012-01-24 20:57 ` Stephen Warren
2012-01-31 11:02 ` Laxman Dewangan
0 siblings, 1 reply; 4+ messages in thread
From: Stephen Warren @ 2012-01-24 20:57 UTC (permalink / raw)
To: Laxman Dewangan, ccross, olof, linux
Cc: linux-tegra, linux-arm-kernel, linux-kernel
Laxman Dewangan wrote at Tuesday, January 24, 2012 1:11 AM:
> It is not require to move the requestor of dma to INVALID
> option before stopping dma.
>
> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
Acked-by: Stephen Warren <swarren@nvidia.com>
Tested-by: Stephen Warren <swarren@nvidia.com>
I know that Laxman has checked with the HW engineers, and they see no
reason to do this. I was originally worried about hung FIFOs preventing
an in-progress DMA from completely without the removed code, but according
to HW, that isn't a concern.
--
nvpublic
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH V1] arm: tegra: dma: not required to move requestor when stopping.
2012-01-24 20:57 ` Stephen Warren
@ 2012-01-31 11:02 ` Laxman Dewangan
2012-02-01 7:09 ` Olof Johansson
0 siblings, 1 reply; 4+ messages in thread
From: Laxman Dewangan @ 2012-01-31 11:02 UTC (permalink / raw)
To: Stephen Warren
Cc: ccross, olof, linux, linux-tegra, linux-arm-kernel, linux-kernel
On Wednesday 25 January 2012 02:27 AM, Stephen Warren wrote:
> Laxman Dewangan wrote at Tuesday, January 24, 2012 1:11 AM:
>> It is not require to move the requestor of dma to INVALID
>> option before stopping dma.
>>
>> Signed-off-by: Laxman Dewangan<ldewangan@nvidia.com>
> Acked-by: Stephen Warren<swarren@nvidia.com>
> Tested-by: Stephen Warren<swarren@nvidia.com>
>
> I know that Laxman has checked with the HW engineers, and they see no
> reason to do this. I was originally worried about hung FIFOs preventing
> an in-progress DMA from completely without the removed code, but according
> to HW, that isn't a concern.
>
Olaf,
Can it be applied if it is OK to you?
Thanks,
Laxman
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH V1] arm: tegra: dma: not required to move requestor when stopping.
2012-01-31 11:02 ` Laxman Dewangan
@ 2012-02-01 7:09 ` Olof Johansson
0 siblings, 0 replies; 4+ messages in thread
From: Olof Johansson @ 2012-02-01 7:09 UTC (permalink / raw)
To: Laxman Dewangan
Cc: Stephen Warren, ccross, linux, linux-tegra, linux-arm-kernel,
linux-kernel
On Tue, Jan 31, 2012 at 04:32:48PM +0530, Laxman Dewangan wrote:
> On Wednesday 25 January 2012 02:27 AM, Stephen Warren wrote:
> >Laxman Dewangan wrote at Tuesday, January 24, 2012 1:11 AM:
> >>It is not require to move the requestor of dma to INVALID
> >>option before stopping dma.
> >>
> >>Signed-off-by: Laxman Dewangan<ldewangan@nvidia.com>
> >Acked-by: Stephen Warren<swarren@nvidia.com>
> >Tested-by: Stephen Warren<swarren@nvidia.com>
> >
> >I know that Laxman has checked with the HW engineers, and they see no
> >reason to do this. I was originally worried about hung FIFOs preventing
> >an in-progress DMA from completely without the removed code, but according
> >to HW, that isn't a concern.
> >
> Olaf,
> Can it be applied if it is OK to you?
Yep, applied.
-Olof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-02-01 7:09 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-24 8:10 [PATCH V1] arm: tegra: dma: not required to move requestor when stopping Laxman Dewangan
2012-01-24 20:57 ` Stephen Warren
2012-01-31 11:02 ` Laxman Dewangan
2012-02-01 7:09 ` Olof Johansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).