From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olof Johansson Subject: Re: [PATCH V1] arm: tegra: dma: not required to move requestor when stopping. Date: Tue, 31 Jan 2012 23:09:18 -0800 Message-ID: <20120201070918.GA7399@quad.lixom.net> References: <1327392649-27605-1-git-send-email-ldewangan@nvidia.com> <74CDBE0F657A3D45AFBB94109FB122FF178CB81E5F@HQMAIL01.nvidia.com> <4F27CA58.2020809@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <4F27CA58.2020809-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Laxman Dewangan Cc: Stephen Warren , "ccross-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org" , "linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org" , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-tegra@vger.kernel.org On Tue, Jan 31, 2012 at 04:32:48PM +0530, Laxman Dewangan wrote: > On Wednesday 25 January 2012 02:27 AM, Stephen Warren wrote: > >Laxman Dewangan wrote at Tuesday, January 24, 2012 1:11 AM: > >>It is not require to move the requestor of dma to INVALID > >>option before stopping dma. > >> > >>Signed-off-by: Laxman Dewangan > >Acked-by: Stephen Warren > >Tested-by: Stephen Warren > > > >I know that Laxman has checked with the HW engineers, and they see no > >reason to do this. I was originally worried about hung FIFOs preventing > >an in-progress DMA from completely without the removed code, but according > >to HW, that isn't a concern. > > > Olaf, > Can it be applied if it is OK to you? Yep, applied. -Olof