From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laxman Dewangan Subject: Re: [PATCH V1] arm: tegra: dma: not required to move requestor when stopping. Date: Tue, 31 Jan 2012 16:32:48 +0530 Message-ID: <4F27CA58.2020809@nvidia.com> References: <1327392649-27605-1-git-send-email-ldewangan@nvidia.com> <74CDBE0F657A3D45AFBB94109FB122FF178CB81E5F@HQMAIL01.nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF178CB81E5F-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: "ccross-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org" , "olof-nZhT3qVonbNeoWH0uzbU5w@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 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? Thanks, Laxman