From mboxrd@z Thu Jan 1 00:00:00 1970 From: agust@denx.de (Anatolij Gustschin) Date: Fri, 4 Feb 2011 10:35:09 +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> Message-ID: <20110204103509.4465f23e@wker> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi all, On Thu, 3 Feb 2011 11:09:54 +0100 (CET) Guennadi Liakhovetski wrote: ... > Yes, the first interrupt is different, that's where I'm dropping / > postponing it. With your patch only N (equal to the number of buffers > used, I think) first interrupts toggle, then always only one buffer is > used: > > idmac_interrupt(): IDMAC irq 177, buf 0, current 0 > idmac_interrupt(): IDMAC irq 177, buf 1, current 1 > idmac_interrupt(): IDMAC irq 177, buf 0, current 0 > idmac_interrupt(): IDMAC irq 177, buf 1, current 1 > idmac_interrupt(): IDMAC irq 177, buf 0, current 0 > idmac_interrupt(): IDMAC irq 177, buf 1, current 1 > idmac_interrupt(): IDMAC irq 177, buf 0, current 0 > idmac_interrupt(): IDMAC irq 177, buf 0, current 0 > idmac_interrupt(): IDMAC irq 177, buf 0, current 0 > ... > > Verified with both capture.c and mplayer. Could you, please, verify > whether you get the same behaviour and what the problem could be? Currently I'm quite busy, but I'll look at it this week end. Thanks, Anatolij