From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@deeprootsystems.com (Kevin Hilman) Date: Mon, 24 May 2010 10:27:07 -0700 Subject: [PATCH 3/6] davinci: edma: clear events in edma_start() In-Reply-To: <4BFA6A22.1000006@windriver.com> (stanley miao's message of "Mon\, 24 May 2010 19\:59\:30 +0800") References: <1268841163-5868-1-git-send-email-khilman@deeprootsystems.com> <1268841163-5868-4-git-send-email-khilman@deeprootsystems.com> <4BFA6A22.1000006@windriver.com> Message-ID: <871vd16wic.fsf@deeprootsystems.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org [please don't top post, moved your reply below original...] "stanley.miao" writes: > Kevin Hilman wrote: >> From: Brian Niebuhr >> >> This patch fixes an issue where a DMA channel can erroneously process an >> event generated by a previous transfer. A failure case is where DMA is >> being used for SPI transmit and receive channels on OMAP L138. In this >> case there is a single bit that controls all event generation from the >> SPI peripheral. Therefore it is possible that between when edma_stop() >> has been called for the transmit channel on a previous transfer and >> edma_start() is called for the transmit channel on a subsequent transfer, >> that a transmit event has been generated. >> >> The fix is to clear events in edma_start(). This prevents false events >> from being processed when events are enabled for that channel. >> > > This patch causes that the sound can not work normally on OMAP_L138. > Can you describe "can not work normally"? Does that mean simply does not work, or works with pops & clicks etc.? Just to clarify... how did you isolate it to this patch. If you revert just this patch on current davinci git, do you have working sound as you expect again? Kevin