From mboxrd@z Thu Jan 1 00:00:00 1970 From: agust@denx.de (Anatolij Gustschin) Date: Sat, 5 Feb 2011 21:04:57 +0100 Subject: [PATCH 2/2 v2] dma: ipu_idmac: do not lose valid received data in the irq handler In-Reply-To: References: <1296031789-1721-3-git-send-email-agust@denx.de> <1296476549-10421-1-git-send-email-agust@denx.de> <20110205143505.0b300a3a@wker> Message-ID: <20110205210457.7218ecdc@wker> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sat, 5 Feb 2011 17:36:37 +0100 (CET) Guennadi Liakhovetski wrote: ... > > > Verified with both capture.c and mplayer. Could you, please, verify > > > whether you get the same behaviour and what the problem could be? > > > > Now I did some further testing with idmac patch applied and with > > added debug print in the IDMAC interrupt handler. There is no problem. > > Testing with capture.c (4 buffers used as default) shows that buffer > > numbers toggle correctly for all 100 captured frames: > > Hm, interesting, I'll have to look at my testing in more detail then > (once back from FOSDEM). Could you maybe try mplayer too? I can't try mplayer since I don't have mplayer setup for this. But looking at the mplayer source I don't see why it should behave differently. Depending on mode mplayer queues 2 or 6 buffers. Testing with my test app with 6 queued buffers shows no issues, here the buffer numbers toggle correctly, too. Anatolij