From patchwork Tue Feb 15 09:34:59 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arindam Nath X-Patchwork-Id: 558341 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.3) with ESMTP id p1F9a7Bj025874 for ; Tue, 15 Feb 2011 09:36:20 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752690Ab1BOJgU (ORCPT ); Tue, 15 Feb 2011 04:36:20 -0500 Received: from mail-iw0-f174.google.com ([209.85.214.174]:54223 "EHLO mail-iw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753787Ab1BOJgT (ORCPT ); Tue, 15 Feb 2011 04:36:19 -0500 Received: by iwn9 with SMTP id 9so5731723iwn.19 for ; Tue, 15 Feb 2011 01:36:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:sender:from:to:cc:subject:date:message-id :x-mailer:in-reply-to:references; bh=UDK+Mzva75V6v1T1VRLZODojgfm55sHcaMctLtt2dBQ=; b=oETxmFTkSkigHxk7gGRna8KnBlb7wlBRYOZ06cy92azg0zkkChMKDkRjHhk/JJ0xva mXk4U/rGOUtyk23OnFeKqahunOe3Havh+GGxm/knc/voEeJTVaCfWCCpzXzmuWwUSip0 TBlMR/MOB8kNvSN4gVnIFEAT978u2nrFLfwAI= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references; b=AXMLcFTa1xDQk7FlqrY90iue2JZXcLk1mga2mfTGhPuzkVGFRw8a8Nmb6FLoMcuOhM 1FkbWn+CIrQwnyq5KZ0Wtub+cb9jzFV48kGhjR5LhkhcJBpYoyAvHOquR+VW25XzPltK PVkNFd+tSIlg8pE0chDp6aWaDyeGgHgkTVG2s= Received: by 10.42.175.134 with SMTP id ba6mr6397704icb.260.1297762578656; Tue, 15 Feb 2011 01:36:18 -0800 (PST) Received: from localhost ([122.167.0.108]) by mx.google.com with ESMTPS id 8sm3444028iba.22.2011.02.15.01.36.11 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 15 Feb 2011 01:36:17 -0800 (PST) From: Arindam Nath To: cjb@laptop.org Cc: linux-mmc@vger.kernel.org, henry.su@amd.com, aaron.lu@amd.com, anath.amd@gmail.com, Arindam Nath Subject: [PATCH 01/12] mmc: sdhci: add support for auto CMD23 Date: Tue, 15 Feb 2011 15:04:59 +0530 Message-Id: <1297762510-2696-2-git-send-email-arindam.nath@amd.com> X-Mailer: git-send-email 1.7.1 In-Reply-To: <1297762510-2696-1-git-send-email-arindam.nath@amd.com> References: <1297762510-2696-1-git-send-email-arindam.nath@amd.com> Sender: linux-mmc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Tue, 15 Feb 2011 09:36:20 +0000 (UTC) diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c index d18c32b..b3f8a3c 100644 --- a/drivers/mmc/core/sd.c +++ b/drivers/mmc/core/sd.c @@ -188,6 +188,12 @@ static int mmc_decode_scr(struct mmc_card *card) scr->sda_vsn = UNSTUFF_BITS(resp, 56, 4); scr->bus_widths = UNSTUFF_BITS(resp, 48, 4); + if (scr->sda_vsn == SCR_SPEC_VER_2) { + /* Check if Physical Layer Spec v3.0 is supported*/ + scr->sda_spec3 = UNSTUFF_BITS(resp, 47, 1); + if (scr->sda_spec3) + scr->cmd_support = UNSTUFF_BITS(resp, 32, 2); + } if (UNSTUFF_BITS(resp, 55, 1)) card->erased_byte = 0xFF; diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 9e15f41..0b537cf 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -25,6 +25,7 @@ #include #include +#include #include "sdhci.h" @@ -46,6 +47,8 @@ static void sdhci_finish_data(struct sdhci_host *); static void sdhci_send_command(struct sdhci_host *, struct mmc_command *); static void sdhci_finish_command(struct sdhci_host *); +static struct mmc_command *abort_cmd; + static void sdhci_dumpregs(struct sdhci_host *host) { printk(KERN_DEBUG DRIVER_NAME ": =========== REGISTER DUMP (%s)===========\n", @@ -812,6 +815,30 @@ static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_data *data) sdhci_writew(host, data->blocks, SDHCI_BLOCK_COUNT); } +/* + * Does the Host Controller support Auto CMD23? + * + * There are four preconditions for Auto CMD23 to be supported: + * 1. Host Controller v3.00 or later + * 2. Card supports CMD23 + * 3. If DMA is used, it should be ADMA + * 4. Only when CMD18 or CMD25 is issued + */ +static int sdhci_host_auto_cmd23_supported(struct sdhci_host *host, + struct mmc_request *mrq) +{ + struct mmc_card *card = host->mmc->card; + + if ((host->version >= SDHCI_SPEC_300) && + card && (card->scr.cmd_support & SD_SCR_CMD23_SUPPORT) && + !(host->flags & SDHCI_USE_SDMA) && + ((mrq->cmd->opcode == MMC_READ_MULTIPLE_BLOCK) || + (mrq->cmd->opcode == MMC_WRITE_MULTIPLE_BLOCK))) + return 1; + + return 0; +} + static void sdhci_set_transfer_mode(struct sdhci_host *host, struct mmc_data *data) { @@ -824,11 +851,23 @@ static void sdhci_set_transfer_mode(struct sdhci_host *host, mode = SDHCI_TRNS_BLK_CNT_EN; if (data->blocks > 1) { + if (sdhci_host_auto_cmd23_supported(host, host->mrq)) { + /* + * Host Controller v3.00 can automatically send CMD23 + * before CMD18 or CMD25. For that, we need to enable + * Auto CMD23 in the Transfer Mode register and + * program the block count in Argument 2 register. + */ + mode |= SDHCI_TRNS_AUTO_CMD23; + sdhci_writel(host, data->blocks, SDHCI_ARGUMENT2); + } + if (host->quirks & SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12) mode |= SDHCI_TRNS_MULTI | SDHCI_TRNS_ACMD12; else mode |= SDHCI_TRNS_MULTI; } + if (data->flags & MMC_DATA_READ) mode |= SDHCI_TRNS_READ; if (host->flags & SDHCI_REQ_USE_DMA) @@ -1131,6 +1170,23 @@ static void sdhci_request(struct mmc_host *mmc, struct mmc_request *mrq) #ifndef SDHCI_USE_LEDS_CLASS sdhci_activate_led(host); #endif + /* + * Since the block count for CMD23 has already been specified in the + * Argument 2 register, we don't need CMD12 to stop multiple block + * transfers. + */ + if (sdhci_host_auto_cmd23_supported(host, mrq)) { + if (mrq->stop) { + /* + * Save the stop command here to be used during + * error recovery + */ + abort_cmd = mrq->data->stop; + mrq->data->stop = NULL; + mrq->stop = NULL; + } + } + if (host->quirks & SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12) { if (mrq->stop) { mrq->data->stop = NULL; @@ -1396,6 +1452,7 @@ static void sdhci_timeout_timer(unsigned long data) if (host->data) { host->data->error = -ETIMEDOUT; + host->data->stop = abort_cmd; sdhci_finish_data(host); } else { if (host->cmd) @@ -1534,9 +1591,10 @@ static void sdhci_data_irq(struct sdhci_host *host, u32 intmask) host->data->error = -EIO; } - if (host->data->error) + if (host->data->error) { + host->data->stop = abort_cmd; sdhci_finish_data(host); - else { + } else { if (intmask & (SDHCI_INT_DATA_AVAIL | SDHCI_INT_SPACE_AVAIL)) sdhci_transfer_pio(host); @@ -1783,8 +1841,10 @@ int sdhci_add_host(struct sdhci_host *host) host->flags &= ~SDHCI_USE_SDMA; } - if ((host->version >= SDHCI_SPEC_200) && (caps & SDHCI_CAN_DO_ADMA2)) + if ((host->version >= SDHCI_SPEC_200) && (caps & SDHCI_CAN_DO_ADMA2)) { + host->flags &= ~SDHCI_USE_SDMA; host->flags |= SDHCI_USE_ADMA; + } if ((host->quirks & SDHCI_QUIRK_BROKEN_ADMA) && (host->flags & SDHCI_USE_ADMA)) { diff --git a/drivers/mmc/host/sdhci.h b/drivers/mmc/host/sdhci.h index 6e0969e..223762c 100644 --- a/drivers/mmc/host/sdhci.h +++ b/drivers/mmc/host/sdhci.h @@ -25,6 +25,7 @@ */ #define SDHCI_DMA_ADDRESS 0x00 +#define SDHCI_ARGUMENT2 SDHCI_DMA_ADDRESS #define SDHCI_BLOCK_SIZE 0x04 #define SDHCI_MAKE_BLKSZ(dma, blksz) (((dma & 0x7) << 12) | (blksz & 0xFFF)) @@ -37,6 +38,7 @@ #define SDHCI_TRNS_DMA 0x01 #define SDHCI_TRNS_BLK_CNT_EN 0x02 #define SDHCI_TRNS_ACMD12 0x04 +#define SDHCI_TRNS_AUTO_CMD23 0x08 #define SDHCI_TRNS_READ 0x10 #define SDHCI_TRNS_MULTI 0x20 diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h index 8ce0827..22b0335 100644 --- a/include/linux/mmc/card.h +++ b/include/linux/mmc/card.h @@ -58,9 +58,13 @@ struct mmc_ext_csd { struct sd_scr { unsigned char sda_vsn; + unsigned char sda_spec3; unsigned char bus_widths; #define SD_SCR_BUS_WIDTH_1 (1<<0) #define SD_SCR_BUS_WIDTH_4 (1<<2) + unsigned char cmd_support; +#define SD_SCR_CMD20_SUPPORT (1<<0) +#define SD_SCR_CMD23_SUPPORT (1<<1) }; struct sd_ssr { diff --git a/include/linux/mmc/sd.h b/include/linux/mmc/sd.h index 3fd85e0..178363b 100644 --- a/include/linux/mmc/sd.h +++ b/include/linux/mmc/sd.h @@ -59,7 +59,7 @@ #define SCR_SPEC_VER_0 0 /* Implements system specification 1.0 - 1.01 */ #define SCR_SPEC_VER_1 1 /* Implements system specification 1.10 */ -#define SCR_SPEC_VER_2 2 /* Implements system specification 2.00 */ +#define SCR_SPEC_VER_2 2 /* Implements system specification 2.00 - 3.0x */ /* * SD bus widths