From mboxrd@z Thu Jan 1 00:00:00 1970 From: shawn.guo@freescale.com (Shawn Guo) Date: Tue, 19 Jul 2011 16:45:37 +0800 Subject: [PATCH 1/1] ARM: mxs-dma: reset after disable channel In-Reply-To: <20110719082146.GA1951@pengutronix.de> References: <1311048596-19932-1-git-send-email-b29396@freescale.com> <20110719040215.GF3838@S2100-06.ap.freescale.net> <20110719082146.GA1951@pengutronix.de> Message-ID: <20110719084536.GH3838@S2100-06.ap.freescale.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jul 19, 2011 at 10:21:46AM +0200, Wolfram Sang wrote: > On Tue, Jul 19, 2011 at 12:02:16PM +0800, Shawn Guo wrote: > > On Tue, Jul 19, 2011 at 12:09:56PM +0800, Dong Aisheng wrote: > > > We met some channels in abnormal state after disable. > > > Reset it to get a clean state. > > > > > > Signed-off-by: Dong Aisheng > > > Cc: Vinod Koul > > > Cc: Shawn Guo > > > --- > > > > Acked-by: Shawn Guo > > Just to make sure: Did you investigate where the "abnormal" state came > from and if that could be fixed? > I suppose this is a question for Aisheng, as he is the one who actually saw the "abnormal" state. But since mxs_dma_disable_chan() is being called in mxs_dma_control(DMA_TERMINATE_ALL) for case audio, it's should be pretty safe reset the channel here. -- Regards, Shawn