public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
* RE: [PATCH] 24xx dma patch
@ 2006-01-11  5:12 Menon, Nishanth
  0 siblings, 0 replies; 11+ messages in thread
From: Menon, Nishanth @ 2006-01-11  5:12 UTC (permalink / raw)
  To: kyungmin.park, Komal Shah; +Cc: linux-omap-open-source

> P.S., There's some problem when running qtopia on H4.
> At the end of mediaplay. we can't stop audio, exactly audio_sync why?
> But madplay is working well :)
I suspect we might be missing callback interrupts some place. The sync
waits for the buffers to be drained. If the draining never happened, it
implies that we have a mismatch at some point...
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 11+ messages in thread
* [PATCH] 24xx dma patch
@ 2006-01-06  7:33 Kyungmin Park
  2006-01-14  0:49 ` Tony Lindgren
  0 siblings, 1 reply; 11+ messages in thread
From: Kyungmin Park @ 2006-01-06  7:33 UTC (permalink / raw)
  To: linux-omap-open-source

Hi 

It reads DMA status too early and use callback function.
It's make dma error on 24xx when playing the music.

Regards,

Kyungmin Park

--

diff --git a/arch/arm/plat-omap/dma.c b/arch/arm/plat-omap/dma.c
--- a/arch/arm/plat-omap/dma.c
+++ b/arch/arm/plat-omap/dma.c
@@ -866,8 +866,10 @@ static int omap2_dma_handle_ch(int ch)
 	val = 1 << (ch);
 	omap_writel(val, OMAP_DMA4_IRQSTATUS_L0);
 
-	if (likely(dma_chan[ch].callback != NULL))
+	if (likely(dma_chan[ch].callback != NULL)) {
+		status = OMAP_DMA_CSR_REG(ch);
 		dma_chan[ch].callback(ch, status, dma_chan[ch].data);
+	}
 
 	return 0;
 }

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2006-01-15 19:44 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <0ISN00BRPWM5ID@mmp1.samsung.com>
2006-01-10 10:52 ` [PATCH] 24xx dma patch Komal Shah
2006-01-11  4:52   ` [PATCH] McBSP support on omap24xx Kyungmin Park
2006-01-14  1:27     ` Tony Lindgren
2006-01-14 13:34       ` Komal Shah
2006-01-15 19:44         ` Tony Lindgren
2006-01-11  4:59   ` [PATCH] 24xx dma patch Kyungmin Park
2006-01-14  1:02     ` Tony Lindgren
2006-01-14  1:04       ` Tony Lindgren
2006-01-11  5:12 Menon, Nishanth
  -- strict thread matches above, loose matches on Subject: below --
2006-01-06  7:33 Kyungmin Park
2006-01-14  0:49 ` Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox