From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jaehoon Chung Subject: Re: [PATCH] mmc: dw_mmc: Reset DMA before enabling IDMAC Date: Wed, 08 Oct 2014 08:59:46 +0900 Message-ID: <54347E72.8010203@samsung.com> References: <1412618020-22278-1-git-send-email-sonnyrao@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.samsung.com ([203.254.224.33]:19964 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754820AbaJGX7t (ORCPT ); Tue, 7 Oct 2014 19:59:49 -0400 In-reply-to: Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Doug Anderson , Sonny Rao Cc: "linux-mmc@vger.kernel.org" , Heiko Stuebner , Seungwon Jeon , Ulf Hansson , Eddie Cai , Addy Ke , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Acked-by: Jaehoon Chung On 10/07/2014 03:11 AM, Doug Anderson wrote: > Sonny, > > On Mon, Oct 6, 2014 at 10:53 AM, Sonny Rao wrote: >> We've already got a reset of DMA after it's done. Add one before we >> start DMA too. This fixes a data corruption on Rockchip SoCs which >> will get bad data when doing a DMA transfer after doing a PIO transfer. >> >> We tested this on an Exynos 5800 with HS200 and didn't notice any >> difference in sequential read throughput. >> >> Signed-off-by: Sonny Rao >> Signed-off-by: Doug Anderson >> --- >> drivers/mmc/host/dw_mmc.c | 4 ++++ >> 1 file changed, 4 insertions(+) > > Reviewed-by: Doug Anderson > Tested-by: Doug Anderson >