From patchwork Thu Aug 30 16:34:38 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hein_Tibosch X-Patchwork-Id: 1389151 Return-Path: X-Original-To: patchwork-linux-mmc@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 0D6BD3FDF5 for ; Thu, 30 Aug 2012 18:28:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752648Ab2H3S2k (ORCPT ); Thu, 30 Aug 2012 14:28:40 -0400 Received: from bosmailout10.eigbox.net ([66.96.186.10]:42616 "EHLO bosmailout10.eigbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752774Ab2H3S2j (ORCPT ); Thu, 30 Aug 2012 14:28:39 -0400 X-Greylist: delayed 2416 seconds by postgrey-1.27 at vger.kernel.org; Thu, 30 Aug 2012 14:28:39 EDT Received: from bosmailscan15.eigbox.net ([10.20.15.15]) by bosmailout10.eigbox.net with esmtp (Exim) id 1T78rN-00066O-DB for linux-mmc@vger.kernel.org; Thu, 30 Aug 2012 13:48:53 -0400 Received: from bosimpout01.eigbox.net ([10.20.55.1]) by bosmailscan15.eigbox.net with esmtp (Exim) id 1T78rN-00022E-5A; Thu, 30 Aug 2012 13:48:53 -0400 Received: from bosauthsmtp02.eigbox.net ([10.20.18.2]) by bosimpout01.eigbox.net with NO UCE id t5ot1j00302gpmq015otcE; Thu, 30 Aug 2012 13:48:53 -0400 X-Authority-Analysis: v=2.0 cv=aPZHX8Bm c=1 sm=1 a=EVDLJLhrg0sUKeqAW2r0xw==:17 a=bc2JKO6qiGsA:10 a=tW0haWE2Y5YA:10 a=yNKk50g6HJIA:10 a=8nJEP1OIZ-IA:10 a=yUBu062l_5EA:10 a=bJ0fqD8TFZgqkSadqForXVIPBlU=:19 a=jAfPOIUMAAAA:8 a=6yQVg0x1IOBCQPKcG9QA:9 a=wPNLvfGTeEIA:10 a=gE_ZUhsuebIA:10 a=9MP9vxlQrmnoeofDS6o88g==:117 X-EN-OrigOutIP: 10.20.18.2 X-EN-IMPSID: t5ot1j00302gpmq015otcE Received: from [114.79.60.254] (helo=[10.251.242.253]) by bosauthsmtp02.eigbox.net with esmtpa (Exim) id 1T78rM-0004tr-1d; Thu, 30 Aug 2012 13:48:53 -0400 Message-ID: <503F961E.5010905@yahoo.es> Date: Fri, 31 Aug 2012 00:34:38 +0800 From: Hein Tibosch User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/20120824 Thunderbird/15.0 MIME-Version: 1.0 To: Nicolas Ferre , "ludovic.desroches" CC: Hans-Christian Egtvedt , Havard Skinnemoen , "linux-mmc@vger.kernel.org" , linux-arm-kernel@lists.infradead.org, Chris Ball Subject: [PATCH v2 1/2] mmc: atmel-mci: DMA can be used with other controller X-EN-UserInfo: 3946c951b80c12a8be5482963a0b1232:e0ae43bc192b431f8b69f09a37527cbc X-EN-AuthUser: hein@htibosch.net X-EN-OrigIP: 114.79.60.254 X-EN-OrigHost: unknown Sender: linux-mmc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org After the latest changes to atmel-mci, it could not be used with DMA on the AVR32 platform. This patch will allow to use DMA again and it will avoid access to MCI register ATMCI_DMA. v2: Even if the IP version is lower than v3xx and doesn't have the DMA configuration register, DMA transfers can be used with a different controller than the Atmel AHB DMA one. For instance, some AVR chips use the Synopsys DesignWare AHB DMA controller. v2: in atmci_configure_dma: check both pdata and pdata->dma_slave Signed-off-by: Hein Tibosch Acked-by: Ludovic Desroches --- drivers/mmc/host/atmel-mci.c | 26 +++++++++++++------------- 1 files changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/mmc/host/atmel-mci.c b/drivers/mmc/host/atmel-mci.c index f2c115e..a6a5593 100644 --- a/drivers/mmc/host/atmel-mci.c +++ b/drivers/mmc/host/atmel-mci.c @@ -71,7 +71,7 @@ enum atmci_pdc_buf { }; struct atmel_mci_caps { - bool has_dma; + bool has_dma_conf_reg; bool has_pdc; bool has_cfg_reg; bool has_cstor_reg; @@ -411,7 +411,7 @@ static int atmci_regs_show(struct seq_file *s, void *v) atmci_show_status_reg(s, "SR", buf[ATMCI_SR / 4]); atmci_show_status_reg(s, "IMR", buf[ATMCI_IMR / 4]); - if (host->caps.has_dma) { + if (host->caps.has_dma_conf_reg) { u32 val; val = buf[ATMCI_DMA / 4]; @@ -767,7 +767,7 @@ static void atmci_dma_complete(void *arg) dev_vdbg(&host->pdev->dev, "DMA complete\n"); - if (host->caps.has_dma) + if (host->caps.has_dma_conf_reg) /* Disable DMA hardware handshaking on MCI */ atmci_writel(host, ATMCI_DMA, atmci_readl(host, ATMCI_DMA) & ~ATMCI_DMAEN); @@ -954,7 +954,9 @@ atmci_prepare_data_dma(struct atmel_mci *host, struct mmc_data *data) maxburst = atmci_convert_chksize(host->dma_conf.dst_maxburst); } - atmci_writel(host, ATMCI_DMA, ATMCI_DMA_CHKSIZE(maxburst) | ATMCI_DMAEN); + if (host->caps.has_dma_conf_reg) + atmci_writel(host, ATMCI_DMA, ATMCI_DMA_CHKSIZE(maxburst) | + ATMCI_DMAEN); sglen = dma_map_sg(chan->device->dev, data->sg, data->sg_len, direction); @@ -2161,7 +2163,10 @@ static bool atmci_configure_dma(struct atmel_mci *host) pdata = host->pdev->dev.platform_data; - if (pdata && find_slave_dev(pdata->dma_slave)) { + if (!pdata) + return false; + + if (pdata->dma_slave && find_slave_dev(pdata->dma_slave)) { dma_cap_mask_t mask; /* Try to grab a DMA channel */ @@ -2202,7 +2207,7 @@ static void __init atmci_get_cap(struct atmel_mci *host) dev_info(&host->pdev->dev, "version: 0x%x\n", version); - host->caps.has_dma = 0; + host->caps.has_dma_conf_reg = 0; host->caps.has_pdc = 1; host->caps.has_cfg_reg = 0; host->caps.has_cstor_reg = 0; @@ -2219,12 +2224,7 @@ static void __init atmci_get_cap(struct atmel_mci *host) host->caps.has_odd_clk_div = 1; case 0x400: case 0x300: -#ifdef CONFIG_AT_HDMAC - host->caps.has_dma = 1; -#else - dev_info(&host->pdev->dev, - "has dma capability but dma engine is not selected, then use pio\n"); -#endif + host->caps.has_dma_conf_reg = 1; host->caps.has_pdc = 0; host->caps.has_cfg_reg = 1; host->caps.has_cstor_reg = 1; @@ -2298,7 +2298,7 @@ static int __init atmci_probe(struct platform_device *pdev) /* Get MCI capabilities and set operations according to it */ atmci_get_cap(host); - if (host->caps.has_dma && atmci_configure_dma(host)) { + if (atmci_configure_dma(host)) { host->prepare_data = &atmci_prepare_data_dma; host->submit_data = &atmci_submit_data_dma; host->stop_transfer = &atmci_stop_transfer_dma;