Skip to contentSkip to conversation

MMC: add sdhci generic framework

PATCH New Lei Wen 6 participants 21 messages Jun 16, 2011 – Oct 15, 2013 MMC, SD and SDIO

lore
Share & tools
lore

Lei Wen

author
#0 lore raw

PATCH v2 1/2 MMC: add sdhci generic framework

Nowdays, there are plenty of mmc driver in uboot adopt the sd standard host design, aka as sdhci. It is better to centralize the common logic together to better maintenance.

---
 drivers/mmc/Makefile |    1 +
 drivers/mmc/sdhci.c  |  433 ++++++++++++++++++++++++++++++++++++++++++++++++++
 include/sdhci.h      |  325 +++++++++++++++++++++++++++++++++++++
 3 files changed, 759 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/sdhci.c
 create mode 100644 include/sdhci.h
drivers/mmc/Makefile +1 -0
index a8fe17a..50b5117 100644
@@ -38,6 +38,7 @@ COBJS-$(CONFIG_OMAP3_MMC) += omap3_mmc.o
 COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
 COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
 COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
+COBJS-$(CONFIG_SDHCI) += sdhci.o
 
 COBJS	:= $(COBJS-y)
 SRCS	:= $(COBJS:.o=.c)
drivers/mmc/sdhci.c new file +433 -0

433 lines not on this page · raw message

include/sdhci.h new file +325 -0

325 lines not on this page · raw message

Signature · 1 line
1.7.0.4
author
#1 lore raw

PATCH v2 1/2 MMC: add sdhci generic framework

Nowdays, there are plenty of mmc driver in uboot adopt the sd standard host design, aka as sdhci. It is better to centralize the common logic together to better maintenance.

---
V1:
add sdhci generic framework

V2:
No Change

 drivers/mmc/Makefile |    1 +
 drivers/mmc/sdhci.c  |  433 ++++++++++++++++++++++++++++++++++++++++++++++++++
 include/sdhci.h      |  325 +++++++++++++++++++++++++++++++++++++
 3 files changed, 759 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/sdhci.c
 create mode 100644 include/sdhci.h
drivers/mmc/Makefile +1 -0
index a8fe17a..50b5117 100644
@@ -38,6 +38,7 @@ COBJS-$(CONFIG_OMAP3_MMC) += omap3_mmc.o
 COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
 COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
 COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
+COBJS-$(CONFIG_SDHCI) += sdhci.o
 
 COBJS	:= $(COBJS-y)
 SRCS	:= $(COBJS:.o=.c)
drivers/mmc/sdhci.c new file +433 -0

433 lines not on this page · raw message

include/sdhci.h new file +325 -0

325 lines not on this page · raw message

Signature · 1 line
1.7.0.4
#2 lore raw

PATCH v2 1/2 MMC: add sdhci generic framework

In reply to Lei Wen

Hi All,

Any comment to this patch set, could it be merged in this open window?

Thanks,
Lei

Quoted 803 lines · Lei Wen

Nowdays, there are plenty of mmc driver in uboot adopt the sd standard host design, aka as sdhci. It is better to centralize the common logic together to better maintenance.

---

?drivers/mmc/Makefile | ? ?1 +
?drivers/mmc/sdhci.c ?| ?433 ++++++++++++++++++++++++++++++++++++++++++++++++++
?include/sdhci.h ? ? ?| ?325 +++++++++++++++++++++++++++++++++++++

?3 files changed, 759 insertions(+), 0 deletions(-) ?create mode 100644 drivers/mmc/sdhci.c ?create mode 100644 include/sdhci.h

drivers/mmc/Makefile +0 -0
index a8fe17a..50b5117 100644
@@ -38,6 +38,7 @@ COBJS-$(CONFIG_OMAP3_MMC) += omap3_mmc.o

?COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
?COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
?COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
+COBJS-$(CONFIG_SDHCI) += sdhci.o

?COBJS ?:= $(COBJS-y)
?SRCS ? := $(COBJS:.o=.c)

drivers/mmc/sdhci.c new file +433 -0
new file mode 100644
index 0000000..9ebd33d
@@ -0,0 +1,433 @@
+/*
+ * Copyright 2011, Marvell Semiconductor Inc.
+ * Lei Wen <[email protected]>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. ?See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ *
+ * Back ported to the 8xx platform (from the 8260 platform) by
+ * Murray.Jensen at cmst.csiro.au, 27-Jan-01.
+ */
+
+#include <common.h>
+#include <malloc.h>
+#include <mmc.h>
+#include <sdhci.h>
+
+void *aligned_buffer;
+
+static void sdhci_reset(struct sdhci_host *host, u8 mask)
+{
+ ? ? ? unsigned long timeout;
+
+ ? ? ? /* Wait max 100 ms */
+ ? ? ? timeout = 100;
+ ? ? ? sdhci_writeb(host, mask, SDHCI_SOFTWARE_RESET);
+ ? ? ? while (sdhci_readb(host, SDHCI_SOFTWARE_RESET) & mask) {
+ ? ? ? ? ? ? ? if (timeout == 0) {
+ ? ? ? ? ? ? ? ? ? ? ? printf("Reset 0x%x never completed.\n", (int)mask);
+ ? ? ? ? ? ? ? ? ? ? ? return;
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? timeout--;
+ ? ? ? ? ? ? ? udelay(1000);
+ ? ? ? }
+}
+
+static void sdhci_cmd_done(struct sdhci_host *host, struct mmc_cmd *cmd)
+{
+ ? ? ? int i;
+ ? ? ? if (cmd->resp_type & MMC_RSP_136) {
+ ? ? ? ? ? ? ? /* CRC is stripped so we need to do some shifting. */
+ ? ? ? ? ? ? ? for (i = 0; i < 4; i++) {
+ ? ? ? ? ? ? ? ? ? ? ? cmd->response[i] = sdhci_readl(host,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? SDHCI_RESPONSE + (3-i)*4) << 8;
+ ? ? ? ? ? ? ? ? ? ? ? if (i != 3)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? cmd->response[i] |= sdhci_readb(host,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? SDHCI_RESPONSE + (3-i)*4-1);
+ ? ? ? ? ? ? ? }
+ ? ? ? } else {
+ ? ? ? ? ? ? ? cmd->response[0] = sdhci_readl(host, SDHCI_RESPONSE);
+ ? ? ? }
+}
+
+static void sdhci_transfer_pio(struct sdhci_host *host, struct mmc_data *data)
+{
+ ? ? ? int i;
+ ? ? ? char *offs;
+ ? ? ? for (i = 0; i < data->blocksize; i += 4) {
+ ? ? ? ? ? ? ? offs = data->dest + i;
+ ? ? ? ? ? ? ? if (data->flags == MMC_DATA_READ)
+ ? ? ? ? ? ? ? ? ? ? ? *(u32 *)offs = sdhci_readl(host, SDHCI_BUFFER);
+ ? ? ? ? ? ? ? else
+ ? ? ? ? ? ? ? ? ? ? ? sdhci_writel(host, *(u32 *)offs, SDHCI_BUFFER);
+ ? ? ? }
+}
+
+static int sdhci_transfer_data(struct sdhci_host *host, struct mmc_data *data,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned int start_addr)
+{
+ ? ? ? unsigned int stat, rdy, mask, block = 0;
+
+ ? ? ? rdy = SDHCI_INT_SPACE_AVAIL | SDHCI_INT_DATA_AVAIL;
+ ? ? ? mask = SDHCI_DATA_AVAILABLE | SDHCI_SPACE_AVAILABLE;
+ ? ? ? do {
+ ? ? ? ? ? ? ? stat = sdhci_readl(host, SDHCI_INT_STATUS);
+ ? ? ? ? ? ? ? if (stat & SDHCI_INT_ERROR) {
+ ? ? ? ? ? ? ? ? ? ? ? printf("Error detected in status(0x%X)!\n", stat);
+ ? ? ? ? ? ? ? ? ? ? ? return -1;
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? if (stat & rdy) {
+ ? ? ? ? ? ? ? ? ? ? ? if (!(sdhci_readl(host, SDHCI_PRESENT_STATE) & mask))
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? continue;
+ ? ? ? ? ? ? ? ? ? ? ? sdhci_writel(host, rdy, SDHCI_INT_STATUS);
+ ? ? ? ? ? ? ? ? ? ? ? sdhci_transfer_pio(host, data);
+ ? ? ? ? ? ? ? ? ? ? ? data->dest += data->blocksize;
+ ? ? ? ? ? ? ? ? ? ? ? if (++block >= data->blocks)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? }
+#ifdef CONFIG_MMC_SDMA
+ ? ? ? ? ? ? ? if (stat & SDHCI_INT_DMA_END) {
+ ? ? ? ? ? ? ? ? ? ? ? sdhci_writel(host, SDHCI_INT_DMA_END, SDHCI_INT_STATUS);
+ ? ? ? ? ? ? ? ? ? ? ? start_addr &= SDHCI_DEFAULT_BOUNDARY_SIZE - 1;
+ ? ? ? ? ? ? ? ? ? ? ? start_addr += SDHCI_DEFAULT_BOUNDARY_SIZE;
+ ? ? ? ? ? ? ? ? ? ? ? sdhci_writel(host, start_addr, SDHCI_DMA_ADDRESS);
+ ? ? ? ? ? ? ? }
+#endif
+ ? ? ? } while (!(stat & SDHCI_INT_DATA_END));
+ ? ? ? return 0;
+}
+
+int sdhci_send_command(struct mmc *mmc, struct mmc_cmd *cmd,
+ ? ? ? ? ? ? ? ? ? ? ?struct mmc_data *data)
+{
+ ? ? ? struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+ ? ? ? unsigned int stat = 0;
+ ? ? ? int ret = 0;
+ ? ? ? int trans_bytes = 0, is_aligned = 1;
+ ? ? ? u32 mask, flags, mode;
+ ? ? ? unsigned int timeout, start_addr = 0;
+
+ ? ? ? /* Wait max 10 ms */
+ ? ? ? timeout = 10;
+
+ ? ? ? sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_STATUS);
+ ? ? ? mask = SDHCI_CMD_INHIBIT | SDHCI_DATA_INHIBIT;
+
+ ? ? ? /* We shouldn't wait for data inihibit for stop commands, even
+ ? ? ? ? ?though they might use busy signaling */
+ ? ? ? if (cmd->cmdidx == MMC_CMD_STOP_TRANSMISSION)
+ ? ? ? ? ? ? ? mask &= ~SDHCI_DATA_INHIBIT;
+
+ ? ? ? while (sdhci_readl(host, SDHCI_PRESENT_STATE) & mask) {
+ ? ? ? ? ? ? ? if (timeout == 0) {
+ ? ? ? ? ? ? ? ? ? ? ? printf("Controller never released inhibit bit(s).\n");
+ ? ? ? ? ? ? ? ? ? ? ? return COMM_ERR;
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? timeout--;
+ ? ? ? ? ? ? ? udelay(1000);
+ ? ? ? }
+
+ ? ? ? mask = SDHCI_INT_RESPONSE;
+ ? ? ? if (!(cmd->resp_type & MMC_RSP_PRESENT))
+ ? ? ? ? ? ? ? flags = SDHCI_CMD_RESP_NONE;
+ ? ? ? else if (cmd->resp_type & MMC_RSP_136)
+ ? ? ? ? ? ? ? flags = SDHCI_CMD_RESP_LONG;
+ ? ? ? else if (cmd->resp_type & MMC_RSP_BUSY) {
+ ? ? ? ? ? ? ? flags = SDHCI_CMD_RESP_SHORT_BUSY;
+ ? ? ? ? ? ? ? mask |= SDHCI_INT_DATA_END;
+ ? ? ? } else
+ ? ? ? ? ? ? ? flags = SDHCI_CMD_RESP_SHORT;
+
+ ? ? ? if (cmd->resp_type & MMC_RSP_CRC)
+ ? ? ? ? ? ? ? flags |= SDHCI_CMD_CRC;
+ ? ? ? if (cmd->resp_type & MMC_RSP_OPCODE)
+ ? ? ? ? ? ? ? flags |= SDHCI_CMD_INDEX;
+ ? ? ? if (data)
+ ? ? ? ? ? ? ? flags |= SDHCI_CMD_DATA;
+
+ ? ? ? /*Set Transfer mode regarding to data flag*/
+ ? ? ? if (data != 0) {
+ ? ? ? ? ? ? ? sdhci_writeb(host, 0xe, SDHCI_TIMEOUT_CONTROL);
+ ? ? ? ? ? ? ? mode = SDHCI_TRNS_BLK_CNT_EN;
+ ? ? ? ? ? ? ? trans_bytes = data->blocks * data->blocksize;
+ ? ? ? ? ? ? ? if (data->blocks > 1)
+ ? ? ? ? ? ? ? ? ? ? ? mode |= SDHCI_TRNS_MULTI;
+
+ ? ? ? ? ? ? ? if (data->flags == MMC_DATA_READ)
+ ? ? ? ? ? ? ? ? ? ? ? mode |= SDHCI_TRNS_READ;
+
+#ifdef CONFIG_MMC_SDMA
+ ? ? ? ? ? ? ? if (data->flags == MMC_DATA_READ)
+ ? ? ? ? ? ? ? ? ? ? ? start_addr = (unsigned int)data->dest;
+ ? ? ? ? ? ? ? else
+ ? ? ? ? ? ? ? ? ? ? ? start_addr = (unsigned int)data->src;
+ ? ? ? ? ? ? ? if ((host->quirks & SDHCI_QUIRK_32BIT_DMA_ADDR) &&
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? (start_addr & 0x7) != 0x0) {
+ ? ? ? ? ? ? ? ? ? ? ? is_aligned = 0;
+ ? ? ? ? ? ? ? ? ? ? ? start_addr = (unsigned int)aligned_buffer;
+ ? ? ? ? ? ? ? ? ? ? ? if (data->flags != MMC_DATA_READ)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? memcpy(aligned_buffer, data->src, trans_bytes);
+ ? ? ? ? ? ? ? }
+
+ ? ? ? ? ? ? ? sdhci_writel(host, start_addr, SDHCI_DMA_ADDRESS);
+ ? ? ? ? ? ? ? mode |= SDHCI_TRNS_DMA;
+#endif
+ ? ? ? ? ? ? ? sdhci_writew(host, SDHCI_MAKE_BLKSZ(SDHCI_DEFAULT_BOUNDARY_ARG,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? data->blocksize),
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? SDHCI_BLOCK_SIZE);
+ ? ? ? ? ? ? ? sdhci_writew(host, data->blocks, SDHCI_BLOCK_COUNT);
+ ? ? ? ? ? ? ? sdhci_writew(host, mode, SDHCI_TRANSFER_MODE);
+ ? ? ? }
+
+ ? ? ? sdhci_writel(host, cmd->cmdarg, SDHCI_ARGUMENT);
+#ifdef CONFIG_MMC_SDMA
+ ? ? ? flush_cache(0, ~0);
+#endif
+ ? ? ? sdhci_writew(host, SDHCI_MAKE_CMD(cmd->cmdidx, flags), SDHCI_COMMAND);
+ ? ? ? do {
+ ? ? ? ? ? ? ? stat = sdhci_readl(host, SDHCI_INT_STATUS);
+ ? ? ? ? ? ? ? if (stat & SDHCI_INT_ERROR)
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? } while ((stat & mask) != mask);
+
+ ? ? ? if ((stat & (SDHCI_INT_ERROR | mask)) == mask) {
+ ? ? ? ? ? ? ? sdhci_cmd_done(host, cmd);
+ ? ? ? ? ? ? ? sdhci_writel(host, mask, SDHCI_INT_STATUS);
+ ? ? ? } else
+ ? ? ? ? ? ? ? ret = -1;
+
+ ? ? ? if (!ret && data)
+ ? ? ? ? ? ? ? ret = sdhci_transfer_data(host, data, start_addr);
+
+ ? ? ? stat = sdhci_readl(host, SDHCI_INT_STATUS);
+ ? ? ? sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_STATUS);
+ ? ? ? if (!ret) {
+ ? ? ? ? ? ? ? if ((host->quirks & SDHCI_QUIRK_32BIT_DMA_ADDR) &&
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? !is_aligned && (data->flags == MMC_DATA_READ))
+ ? ? ? ? ? ? ? ? ? ? ? memcpy(data->dest, aligned_buffer, trans_bytes);
+ ? ? ? ? ? ? ? return 0;
+ ? ? ? }
+
+ ? ? ? sdhci_reset(host, SDHCI_RESET_CMD);
+ ? ? ? sdhci_reset(host, SDHCI_RESET_DATA);
+ ? ? ? if (stat & SDHCI_INT_TIMEOUT)
+ ? ? ? ? ? ? ? return TIMEOUT;
+ ? ? ? else
+ ? ? ? ? ? ? ? return COMM_ERR;
+}
+
+static int sdhci_set_clock(struct mmc *mmc, unsigned int clock)
+{
+ ? ? ? struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+ ? ? ? unsigned int div, clk, timeout;
+
+ ? ? ? sdhci_writew(host, 0, SDHCI_CLOCK_CONTROL);
+
+ ? ? ? if (clock == 0)
+ ? ? ? ? ? ? ? return 0;
+
+ ? ? ? if (host->version >= SDHCI_SPEC_300) {
+ ? ? ? ? ? ? ? /* Version 3.00 divisors must be a multiple of 2. */
+ ? ? ? ? ? ? ? if (mmc->f_max <= clock)
+ ? ? ? ? ? ? ? ? ? ? ? div = 1;
+ ? ? ? ? ? ? ? else {
+ ? ? ? ? ? ? ? ? ? ? ? for (div = 2; div < SDHCI_MAX_DIV_SPEC_300; div += 2) {
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? if ((mmc->f_max / div) <= clock)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? }
+ ? ? ? } else {
+ ? ? ? ? ? ? ? /* Version 2.00 divisors must be a power of 2. */
+ ? ? ? ? ? ? ? for (div = 1; div < SDHCI_MAX_DIV_SPEC_200; div *= 2) {
+ ? ? ? ? ? ? ? ? ? ? ? if ((mmc->f_max / div) <= clock)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? }
+ ? ? ? }
+ ? ? ? div >>= 1;
+
+ ? ? ? clk = (div & SDHCI_DIV_MASK) << SDHCI_DIVIDER_SHIFT;
+ ? ? ? clk |= ((div & SDHCI_DIV_HI_MASK) >> SDHCI_DIV_MASK_LEN)
+ ? ? ? ? ? ? ? << SDHCI_DIVIDER_HI_SHIFT;
+ ? ? ? clk |= SDHCI_CLOCK_INT_EN;
+ ? ? ? sdhci_writew(host, clk, SDHCI_CLOCK_CONTROL);
+
+ ? ? ? /* Wait max 20 ms */
+ ? ? ? timeout = 20;
+ ? ? ? while (!((clk = sdhci_readw(host, SDHCI_CLOCK_CONTROL))
+ ? ? ? ? ? ? ? & SDHCI_CLOCK_INT_STABLE)) {
+ ? ? ? ? ? ? ? if (timeout == 0) {
+ ? ? ? ? ? ? ? ? ? ? ? printf("Internal clock never stabilised.\n");
+ ? ? ? ? ? ? ? ? ? ? ? return -1;
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? timeout--;
+ ? ? ? ? ? ? ? udelay(1000);
+ ? ? ? }
+
+ ? ? ? clk |= SDHCI_CLOCK_CARD_EN;
+ ? ? ? sdhci_writew(host, clk, SDHCI_CLOCK_CONTROL);
+ ? ? ? return 0;
+}
+
+static void sdhci_set_power(struct sdhci_host *host, unsigned short power)
+{
+ ? ? ? u8 pwr = 0;
+
+ ? ? ? if (power != (unsigned short)-1) {
+ ? ? ? ? ? ? ? switch (1 << power) {
+ ? ? ? ? ? ? ? case MMC_VDD_165_195:
+ ? ? ? ? ? ? ? ? ? ? ? pwr = SDHCI_POWER_180;
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case MMC_VDD_29_30:
+ ? ? ? ? ? ? ? case MMC_VDD_30_31:
+ ? ? ? ? ? ? ? ? ? ? ? pwr = SDHCI_POWER_300;
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case MMC_VDD_32_33:
+ ? ? ? ? ? ? ? case MMC_VDD_33_34:
+ ? ? ? ? ? ? ? ? ? ? ? pwr = SDHCI_POWER_330;
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? }
+ ? ? ? }
+
+ ? ? ? if (pwr == 0) {
+ ? ? ? ? ? ? ? sdhci_writeb(host, 0, SDHCI_POWER_CONTROL);
+ ? ? ? ? ? ? ? return;
+ ? ? ? }
+
+ ? ? ? pwr |= SDHCI_POWER_ON;
+
+ ? ? ? sdhci_writeb(host, pwr, SDHCI_POWER_CONTROL);
+}
+
+void sdhci_set_ios(struct mmc *mmc)
+{
+ ? ? ? u32 ctrl;
+ ? ? ? struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+
+ ? ? ? if (mmc->clock != host->clock)
+ ? ? ? ? ? ? ? sdhci_set_clock(mmc, mmc->clock);
+
+ ? ? ? /* Set bus width */
+ ? ? ? ctrl = sdhci_readb(host, SDHCI_HOST_CONTROL);
+ ? ? ? if (mmc->bus_width == 8) {
+ ? ? ? ? ? ? ? ctrl &= ~SDHCI_CTRL_4BITBUS;
+ ? ? ? ? ? ? ? if (host->version >= SDHCI_SPEC_300)
+ ? ? ? ? ? ? ? ? ? ? ? ctrl |= SDHCI_CTRL_8BITBUS;
+ ? ? ? } else {
+ ? ? ? ? ? ? ? if (host->version >= SDHCI_SPEC_300)
+ ? ? ? ? ? ? ? ? ? ? ? ctrl &= ~SDHCI_CTRL_8BITBUS;
+ ? ? ? ? ? ? ? if (mmc->bus_width == 4)
+ ? ? ? ? ? ? ? ? ? ? ? ctrl |= SDHCI_CTRL_4BITBUS;
+ ? ? ? ? ? ? ? else
+ ? ? ? ? ? ? ? ? ? ? ? ctrl &= ~SDHCI_CTRL_4BITBUS;
+ ? ? ? }
+
+ ? ? ? if (mmc->clock > 26000000)
+ ? ? ? ? ? ? ? ctrl |= SDHCI_CTRL_HISPD;
+ ? ? ? else
+ ? ? ? ? ? ? ? ctrl &= ~SDHCI_CTRL_HISPD;
+
+ ? ? ? sdhci_writeb(host, ctrl, SDHCI_HOST_CONTROL);
+}
+
+int sdhci_init(struct mmc *mmc)
+{
+ ? ? ? struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+
+ ? ? ? if ((host->quirks & SDHCI_QUIRK_32BIT_DMA_ADDR) && !aligned_buffer) {
+ ? ? ? ? ? ? ? aligned_buffer = memalign(8, 512*1024);
+ ? ? ? ? ? ? ? if (!aligned_buffer) {
+ ? ? ? ? ? ? ? ? ? ? ? printf("Aligned buffer alloc failed!!!");
+ ? ? ? ? ? ? ? ? ? ? ? return -1;
+ ? ? ? ? ? ? ? }
+ ? ? ? }
+
+ ? ? ? /* Eable all state */
+ ? ? ? sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_ENABLE);
+ ? ? ? sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_SIGNAL_ENABLE);
+
+ ? ? ? sdhci_set_power(host, fls(mmc->voltages) - 1);
+
+ ? ? ? return 0;
+}
+
+int add_sdhci(struct sdhci_host *host, u32 max_clk, u32 min_clk)
+{
+ ? ? ? struct mmc *mmc;
+ ? ? ? unsigned int caps;
+
+ ? ? ? mmc = malloc(sizeof(struct mmc));
+ ? ? ? if (!mmc) {
+ ? ? ? ? ? ? ? printf("mmc malloc fail!\n");
+ ? ? ? ? ? ? ? return -1;
+ ? ? ? }
+
+ ? ? ? mmc->priv = host;
+
+ ? ? ? sprintf(mmc->name, "%s", host->name);
+ ? ? ? mmc->send_cmd = sdhci_send_command;
+ ? ? ? mmc->set_ios = sdhci_set_ios;
+ ? ? ? mmc->init = sdhci_init;
+
+ ? ? ? caps = sdhci_readl(host, SDHCI_CAPABILITIES);
+#ifdef CONFIG_MMC_SDMA
+ ? ? ? if (!(caps & SDHCI_CAN_DO_SDMA)) {
+ ? ? ? ? ? ? ? printf("Your controller don't support sdma!!\n");
+ ? ? ? ? ? ? ? return -1;
+ ? ? ? }
+#endif
+
+ ? ? ? if (max_clk)
+ ? ? ? ? ? ? ? mmc->f_max = max_clk;
+ ? ? ? else {
+ ? ? ? ? ? ? ? if (host->version >= SDHCI_SPEC_300)
+ ? ? ? ? ? ? ? ? ? ? ? mmc->f_max = (caps & SDHCI_CLOCK_V3_BASE_MASK)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? >> SDHCI_CLOCK_BASE_SHIFT;
+ ? ? ? ? ? ? ? else
+ ? ? ? ? ? ? ? ? ? ? ? mmc->f_max = (caps & SDHCI_CLOCK_BASE_MASK)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? >> SDHCI_CLOCK_BASE_SHIFT;
+ ? ? ? ? ? ? ? mmc->f_max *= 1000000;
+ ? ? ? }
+ ? ? ? if (mmc->f_max == 0) {
+ ? ? ? ? ? ? ? printf("Hardware doesn't specify base clock frequency\n");
+ ? ? ? ? ? ? ? return -1;
+ ? ? ? }
+ ? ? ? if (min_clk)
+ ? ? ? ? ? ? ? mmc->f_min = min_clk;
+ ? ? ? else {
+ ? ? ? ? ? ? ? if (host->version >= SDHCI_SPEC_300)
+ ? ? ? ? ? ? ? ? ? ? ? mmc->f_min = mmc->f_max / SDHCI_MAX_DIV_SPEC_300;
+ ? ? ? ? ? ? ? else
+ ? ? ? ? ? ? ? ? ? ? ? mmc->f_min = mmc->f_max / SDHCI_MAX_DIV_SPEC_200;
+ ? ? ? }
+
+ ? ? ? mmc->voltages = 0;
+ ? ? ? if (caps & SDHCI_CAN_VDD_330)
+ ? ? ? ? ? ? ? mmc->voltages |= MMC_VDD_32_33 | MMC_VDD_33_34;
+ ? ? ? if (caps & SDHCI_CAN_VDD_300)
+ ? ? ? ? ? ? ? mmc->voltages |= MMC_VDD_29_30 | MMC_VDD_30_31;
+ ? ? ? if (caps & SDHCI_CAN_VDD_180)
+ ? ? ? ? ? ? ? mmc->voltages |= MMC_VDD_165_195;
+ ? ? ? mmc->host_caps = MMC_MODE_HS | MMC_MODE_HS_52MHz | MMC_MODE_4BIT;
+ ? ? ? if (caps & SDHCI_CAN_DO_8BIT)
+ ? ? ? ? ? ? ? mmc->host_caps |= MMC_MODE_8BIT;
+
+ ? ? ? sdhci_reset(host, SDHCI_RESET_ALL);
+ ? ? ? mmc_register(mmc);
+
+ ? ? ? return 0;
+}
include/sdhci.h new file +325 -1
new file mode 100644
index 0000000..6d52ce9
@@ -0,0 +1,325 @@
+/*
+ * Copyright 2011, Marvell Semiconductor Inc.
+ * Lei Wen <[email protected]>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. ?See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ *
+ * Back ported to the 8xx platform (from the 8260 platform) by
+ * Murray.Jensen at cmst.csiro.au, 27-Jan-01.
+ */
+#ifndef __SDHCI_HW_H
+#define __SDHCI_HW_H
+
+#include <asm/io.h>
+/*
+ * Controller registers
+ */
+
+#define SDHCI_DMA_ADDRESS ? ? ?0x00
+
+#define SDHCI_BLOCK_SIZE ? ? ? 0x04
+#define ?SDHCI_MAKE_BLKSZ(dma, blksz) (((dma & 0x7) << 12) | (blksz & 0xFFF))
+
+#define SDHCI_BLOCK_COUNT ? ? ?0x06
+
+#define SDHCI_ARGUMENT ? ? ? ? 0x08
+
+#define SDHCI_TRANSFER_MODE ? ?0x0C
+#define ?SDHCI_TRNS_DMA ? ? ? ? ? ? ? ?0x01
+#define ?SDHCI_TRNS_BLK_CNT_EN 0x02
+#define ?SDHCI_TRNS_ACMD12 ? ? 0x04
+#define ?SDHCI_TRNS_READ ? ? ? 0x10
+#define ?SDHCI_TRNS_MULTI ? ? ?0x20
+
+#define SDHCI_COMMAND ? ? ? ? ?0x0E
+#define ?SDHCI_CMD_RESP_MASK ? 0x03
+#define ?SDHCI_CMD_CRC ? ? ? ? 0x08
+#define ?SDHCI_CMD_INDEX ? ? ? 0x10
+#define ?SDHCI_CMD_DATA ? ? ? ? ? ? ? ?0x20
+#define ?SDHCI_CMD_ABORTCMD ? ?0xC0
+
+#define ?SDHCI_CMD_RESP_NONE ? 0x00
+#define ?SDHCI_CMD_RESP_LONG ? 0x01
+#define ?SDHCI_CMD_RESP_SHORT ?0x02
+#define ?SDHCI_CMD_RESP_SHORT_BUSY 0x03
+
+#define SDHCI_MAKE_CMD(c, f) (((c & 0xff) << 8) | (f & 0xff))
+#define SDHCI_GET_CMD(c) ((c>>8) & 0x3f)
+
+#define SDHCI_RESPONSE ? ? ? ? 0x10
+
+#define SDHCI_BUFFER ? ? ? ? ? 0x20
+
+#define SDHCI_PRESENT_STATE ? ?0x24
+#define ?SDHCI_CMD_INHIBIT ? ? 0x00000001
+#define ?SDHCI_DATA_INHIBIT ? ?0x00000002
+#define ?SDHCI_DOING_WRITE ? ? 0x00000100
+#define ?SDHCI_DOING_READ ? ? ?0x00000200
+#define ?SDHCI_SPACE_AVAILABLE 0x00000400
+#define ?SDHCI_DATA_AVAILABLE ?0x00000800
+#define ?SDHCI_CARD_PRESENT ? ?0x00010000
+#define ?SDHCI_WRITE_PROTECT ? 0x00080000
+
+#define SDHCI_HOST_CONTROL ? ? 0x28
+#define ?SDHCI_CTRL_LED ? ? ? ? ? ? ? ?0x01
+#define ?SDHCI_CTRL_4BITBUS ? ?0x02
+#define ?SDHCI_CTRL_HISPD ? ? ?0x04
+#define ?SDHCI_CTRL_DMA_MASK ? 0x18
+#define ? SDHCI_CTRL_SDMA ? ? ?0x00
+#define ? SDHCI_CTRL_ADMA1 ? ? 0x08
+#define ? SDHCI_CTRL_ADMA32 ? ?0x10
+#define ? SDHCI_CTRL_ADMA64 ? ?0x18
+#define ? SDHCI_CTRL_8BITBUS ? 0x20
+
+#define SDHCI_POWER_CONTROL ? ?0x29
+#define ?SDHCI_POWER_ON ? ? ? ? ? ? ? ?0x01
+#define ?SDHCI_POWER_180 ? ? ? 0x0A
+#define ?SDHCI_POWER_300 ? ? ? 0x0C
+#define ?SDHCI_POWER_330 ? ? ? 0x0E
+
+#define SDHCI_BLOCK_GAP_CONTROL ? ? ? ?0x2A
+
+#define SDHCI_WAKE_UP_CONTROL ?0x2B
+#define ?SDHCI_WAKE_ON_INT ? ? 0x01
+#define ?SDHCI_WAKE_ON_INSERT ?0x02
+#define ?SDHCI_WAKE_ON_REMOVE ?0x04
+
+#define SDHCI_CLOCK_CONTROL ? ?0x2C
+#define ?SDHCI_DIVIDER_SHIFT ? 8
+#define ?SDHCI_DIVIDER_HI_SHIFT ? ? ? ?6
+#define ?SDHCI_DIV_MASK ? ? ? ?0xFF
+#define ?SDHCI_DIV_MASK_LEN ? ?8
+#define ?SDHCI_DIV_HI_MASK ? ? 0x300
+#define ?SDHCI_CLOCK_CARD_EN ? 0x0004
+#define ?SDHCI_CLOCK_INT_STABLE ? ? ? ?0x0002
+#define ?SDHCI_CLOCK_INT_EN ? ?0x0001
+
+#define SDHCI_TIMEOUT_CONTROL ?0x2E
+
+#define SDHCI_SOFTWARE_RESET ? 0x2F
+#define ?SDHCI_RESET_ALL ? ? ? 0x01
+#define ?SDHCI_RESET_CMD ? ? ? 0x02
+#define ?SDHCI_RESET_DATA ? ? ?0x04
+
+#define SDHCI_INT_STATUS ? ? ? 0x30
+#define SDHCI_INT_ENABLE ? ? ? 0x34
+#define SDHCI_SIGNAL_ENABLE ? ?0x38
+#define ?SDHCI_INT_RESPONSE ? ?0x00000001
+#define ?SDHCI_INT_DATA_END ? ?0x00000002
+#define ?SDHCI_INT_DMA_END ? ? 0x00000008
+#define ?SDHCI_INT_SPACE_AVAIL 0x00000010
+#define ?SDHCI_INT_DATA_AVAIL ?0x00000020
+#define ?SDHCI_INT_CARD_INSERT 0x00000040
+#define ?SDHCI_INT_CARD_REMOVE 0x00000080
+#define ?SDHCI_INT_CARD_INT ? ?0x00000100
+#define ?SDHCI_INT_ERROR ? ? ? 0x00008000
+#define ?SDHCI_INT_TIMEOUT ? ? 0x00010000
+#define ?SDHCI_INT_CRC ? ? ? ? 0x00020000
+#define ?SDHCI_INT_END_BIT ? ? 0x00040000
+#define ?SDHCI_INT_INDEX ? ? ? 0x00080000
+#define ?SDHCI_INT_DATA_TIMEOUT ? ? ? ?0x00100000
+#define ?SDHCI_INT_DATA_CRC ? ?0x00200000
+#define ?SDHCI_INT_DATA_END_BIT ? ? ? ?0x00400000
+#define ?SDHCI_INT_BUS_POWER ? 0x00800000
+#define ?SDHCI_INT_ACMD12ERR ? 0x01000000
+#define ?SDHCI_INT_ADMA_ERROR ?0x02000000
+
+#define ?SDHCI_INT_NORMAL_MASK 0x00007FFF
+#define ?SDHCI_INT_ERROR_MASK ?0xFFFF8000
+
+#define ?SDHCI_INT_CMD_MASK ? ?(SDHCI_INT_RESPONSE | SDHCI_INT_TIMEOUT | \
+ ? ? ? ? ? ? ? SDHCI_INT_CRC | SDHCI_INT_END_BIT | SDHCI_INT_INDEX)
+#define ?SDHCI_INT_DATA_MASK ? (SDHCI_INT_DATA_END | SDHCI_INT_DMA_END | \
+ ? ? ? ? ? ? ? SDHCI_INT_DATA_AVAIL | SDHCI_INT_SPACE_AVAIL | \
+ ? ? ? ? ? ? ? SDHCI_INT_DATA_TIMEOUT | SDHCI_INT_DATA_CRC | \
+ ? ? ? ? ? ? ? SDHCI_INT_DATA_END_BIT | SDHCI_INT_ADMA_ERROR)
+#define SDHCI_INT_ALL_MASK ? ? ((unsigned int)-1)
+
+#define SDHCI_ACMD12_ERR ? ? ? 0x3C
+
+/* 3E-3F reserved */
+
+#define SDHCI_CAPABILITIES ? ? 0x40
+#define ?SDHCI_TIMEOUT_CLK_MASK ? ? ? ?0x0000003F
+#define ?SDHCI_TIMEOUT_CLK_SHIFT 0
+#define ?SDHCI_TIMEOUT_CLK_UNIT ? ? ? ?0x00000080
+#define ?SDHCI_CLOCK_BASE_MASK 0x00003F00
+#define ?SDHCI_CLOCK_V3_BASE_MASK ? ? ?0x0000FF00
+#define ?SDHCI_CLOCK_BASE_SHIFT ? ? ? ?8
+#define ?SDHCI_MAX_BLOCK_MASK ?0x00030000
+#define ?SDHCI_MAX_BLOCK_SHIFT ?16
+#define ?SDHCI_CAN_DO_8BIT ? ? 0x00040000
+#define ?SDHCI_CAN_DO_ADMA2 ? ?0x00080000
+#define ?SDHCI_CAN_DO_ADMA1 ? ?0x00100000
+#define ?SDHCI_CAN_DO_HISPD ? ?0x00200000
+#define ?SDHCI_CAN_DO_SDMA ? ? 0x00400000
+#define ?SDHCI_CAN_VDD_330 ? ? 0x01000000
+#define ?SDHCI_CAN_VDD_300 ? ? 0x02000000
+#define ?SDHCI_CAN_VDD_180 ? ? 0x04000000
+#define ?SDHCI_CAN_64BIT ? ? ? 0x10000000
+
+#define SDHCI_CAPABILITIES_1 ? 0x44
+
+#define SDHCI_MAX_CURRENT ? ? ?0x48
+
+/* 4C-4F reserved for more max current */
+
+#define SDHCI_SET_ACMD12_ERROR 0x50
+#define SDHCI_SET_INT_ERROR ? ?0x52
+
+#define SDHCI_ADMA_ERROR ? ? ? 0x54
+
+/* 55-57 reserved */
+
+#define SDHCI_ADMA_ADDRESS ? ? 0x58
+
+/* 60-FB reserved */
+
+#define SDHCI_SLOT_INT_STATUS ?0xFC
+
+#define SDHCI_HOST_VERSION ? ? 0xFE
+#define ?SDHCI_VENDOR_VER_MASK 0xFF00
+#define ?SDHCI_VENDOR_VER_SHIFT ? ? ? ?8
+#define ?SDHCI_SPEC_VER_MASK ? 0x00FF
+#define ?SDHCI_SPEC_VER_SHIFT ?0
+#define ? SDHCI_SPEC_100 ? ? ? 0
+#define ? SDHCI_SPEC_200 ? ? ? 1
+#define ? SDHCI_SPEC_300 ? ? ? 2
+
+/*
+ * End of controller registers.
+ */
+
+#define SDHCI_MAX_DIV_SPEC_200 256
+#define SDHCI_MAX_DIV_SPEC_300 2046
+
+/*
+ * quirks
+ */
+#define SDHCI_QUIRK_32BIT_DMA_ADDR ? ? (1 << 0)
+
+/*
+ * Host SDMA buffer boundary. Valid values from 4K to 512K in powers of 2.
+ */
+#define SDHCI_DEFAULT_BOUNDARY_SIZE ? ?(512 * 1024)
+#define SDHCI_DEFAULT_BOUNDARY_ARG ? ? (7)
+struct sdhci_ops {
+#ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS
+ ? ? ? u32 ? ? ? ? ? ? (*read_l)(struct sdhci_host *host, int reg);
+ ? ? ? u16 ? ? ? ? ? ? (*read_w)(struct sdhci_host *host, int reg);
+ ? ? ? u8 ? ? ? ? ? ? ?(*read_b)(struct sdhci_host *host, int reg);
+ ? ? ? void ? ? ? ? ? ?(*write_l)(struct sdhci_host *host, u32 val, int reg);
+ ? ? ? void ? ? ? ? ? ?(*write_w)(struct sdhci_host *host, u16 val, int reg);
+ ? ? ? void ? ? ? ? ? ?(*write_b)(struct sdhci_host *host, u8 val, int reg);
+#endif
+};
+
+struct sdhci_host {
+ ? ? ? char *name;
+ ? ? ? void *ioaddr;
+ ? ? ? unsigned int quirks;
+ ? ? ? unsigned int version;
+ ? ? ? unsigned int clock;
+ ? ? ? const struct sdhci_ops *ops;
+};
+
+#ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS
+
+static inline void sdhci_writel(struct sdhci_host *host, u32 val, int reg)
+{
+ ? ? ? if (unlikely(host->ops->write_l))
+ ? ? ? ? ? ? ? host->ops->write_l(host, val, reg);
+ ? ? ? else
+ ? ? ? ? ? ? ? writel(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writew(struct sdhci_host *host, u16 val, int reg)
+{
+ ? ? ? if (unlikely(host->ops->write_w))
+ ? ? ? ? ? ? ? host->ops->write_w(host, val, reg);
+ ? ? ? else
+ ? ? ? ? ? ? ? writew(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writeb(struct sdhci_host *host, u8 val, int reg)
+{
+ ? ? ? if (unlikely(host->ops->write_b))
+ ? ? ? ? ? ? ? host->ops->write_b(host, val, reg);
+ ? ? ? else
+ ? ? ? ? ? ? ? writeb(val, host->ioaddr + reg);
+}
+
+static inline u32 sdhci_readl(struct sdhci_host *host, int reg)
+{
+ ? ? ? if (unlikely(host->ops->read_l))
+ ? ? ? ? ? ? ? return host->ops->read_l(host, reg);
+ ? ? ? else
+ ? ? ? ? ? ? ? return readl(host->ioaddr + reg);
+}
+
+static inline u16 sdhci_readw(struct sdhci_host *host, int reg)
+{
+ ? ? ? if (unlikely(host->ops->read_w))
+ ? ? ? ? ? ? ? return host->ops->read_w(host, reg);
+ ? ? ? else
+ ? ? ? ? ? ? ? return readw(host->ioaddr + reg);
+}
+
+static inline u8 sdhci_readb(struct sdhci_host *host, int reg)
+{
+ ? ? ? if (unlikely(host->ops->read_b))
+ ? ? ? ? ? ? ? return host->ops->read_b(host, reg);
+ ? ? ? else
+ ? ? ? ? ? ? ? return readb(host->ioaddr + reg);
+}
+
+#else
+
+static inline void sdhci_writel(struct sdhci_host *host, u32 val, int reg)
+{
+ ? ? ? writel(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writew(struct sdhci_host *host, u16 val, int reg)
+{
+ ? ? ? writew(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writeb(struct sdhci_host *host, u8 val, int reg)
+{
+ ? ? ? writeb(val, host->ioaddr + reg);
+}
+static inline u32 sdhci_readl(struct sdhci_host *host, int reg)
+{
+ ? ? ? return readl(host->ioaddr + reg);
+}
+
+static inline u16 sdhci_readw(struct sdhci_host *host, int reg)
+{
+ ? ? ? return readw(host->ioaddr + reg);
+}
+
+static inline u8 sdhci_readb(struct sdhci_host *host, int reg)
+{
+ ? ? ? return readb(host->ioaddr + reg);
+}
+#endif
+
+int add_sdhci(struct sdhci_host *host, u32 max_clk, u32 min_clk);
+#endif /* __SDHCI_HW_H */
--

1.7.0.4

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

#3 lore raw

PATCH 1/2 MMC: add sdhci generic framework

Hello,

Is this driver adaptable for sheevaplug ?

Le 29/06/2011 05:46, Lei Wen a ?crit :

Quoted 815 lines

Hi All,

Any comment to this patch set, could it be merged in this open window?

Thanks,
Lei

Quoted 798 lines · Lei Wen

Nowdays, there are plenty of mmc driver in uboot adopt the sd standard host design, aka as sdhci. It is better to centralize the common logic together to better maintenance.

---
 drivers/mmc/Makefile |    1 +
 drivers/mmc/sdhci.c  |  433 ++++++++++++++++++++++++++++++++++++++++++++++++++
 include/sdhci.h      |  325 +++++++++++++++++++++++++++++++++++++
 3 files changed, 759 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/sdhci.c
 create mode 100644 include/sdhci.h
drivers/mmc/Makefile +1 -0
index a8fe17a..50b5117 100644
@@ -38,6 +38,7 @@ COBJS-$(CONFIG_OMAP3_MMC) += omap3_mmc.o
 COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
 COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
 COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
+COBJS-$(CONFIG_SDHCI) += sdhci.o
 
 COBJS  := $(COBJS-y)
 SRCS   := $(COBJS:.o=.c)
drivers/mmc/sdhci.c new file +433 -0
new file mode 100644
index 0000000..9ebd33d
@@ -0,0 +1,433 @@
+/*
+ * Copyright 2011, Marvell Semiconductor Inc.
+ * Lei Wen <[email protected]>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ *
+ * Back ported to the 8xx platform (from the 8260 platform) by
+ * Murray.Jensen at cmst.csiro.au, 27-Jan-01.
+ */
+
+#include <common.h>
+#include <malloc.h>
+#include <mmc.h>
+#include <sdhci.h>
+
+void *aligned_buffer;
+
+static void sdhci_reset(struct sdhci_host *host, u8 mask)
+{
+       unsigned long timeout;
+
+       /* Wait max 100 ms */
+       timeout = 100;
+       sdhci_writeb(host, mask, SDHCI_SOFTWARE_RESET);
+       while (sdhci_readb(host, SDHCI_SOFTWARE_RESET) & mask) {
+               if (timeout == 0) {
+                       printf("Reset 0x%x never completed.\n", (int)mask);
+                       return;
+               }
+               timeout--;
+               udelay(1000);
+       }
+}
+
+static void sdhci_cmd_done(struct sdhci_host *host, struct mmc_cmd *cmd)
+{
+       int i;
+       if (cmd->resp_type & MMC_RSP_136) {
+               /* CRC is stripped so we need to do some shifting. */
+               for (i = 0; i < 4; i++) {
+                       cmd->response[i] = sdhci_readl(host,
+                                       SDHCI_RESPONSE + (3-i)*4) << 8;
+                       if (i != 3)
+                               cmd->response[i] |= sdhci_readb(host,
+                                               SDHCI_RESPONSE + (3-i)*4-1);
+               }
+       } else {
+               cmd->response[0] = sdhci_readl(host, SDHCI_RESPONSE);
+       }
+}
+
+static void sdhci_transfer_pio(struct sdhci_host *host, struct mmc_data *data)
+{
+       int i;
+       char *offs;
+       for (i = 0; i < data->blocksize; i += 4) {
+               offs = data->dest + i;
+               if (data->flags == MMC_DATA_READ)
+                       *(u32 *)offs = sdhci_readl(host, SDHCI_BUFFER);
+               else
+                       sdhci_writel(host, *(u32 *)offs, SDHCI_BUFFER);
+       }
+}
+
+static int sdhci_transfer_data(struct sdhci_host *host, struct mmc_data *data,
+                               unsigned int start_addr)
+{
+       unsigned int stat, rdy, mask, block = 0;
+
+       rdy = SDHCI_INT_SPACE_AVAIL | SDHCI_INT_DATA_AVAIL;
+       mask = SDHCI_DATA_AVAILABLE | SDHCI_SPACE_AVAILABLE;
+       do {
+               stat = sdhci_readl(host, SDHCI_INT_STATUS);
+               if (stat & SDHCI_INT_ERROR) {
+                       printf("Error detected in status(0x%X)!\n", stat);
+                       return -1;
+               }
+               if (stat & rdy) {
+                       if (!(sdhci_readl(host, SDHCI_PRESENT_STATE) & mask))
+                               continue;
+                       sdhci_writel(host, rdy, SDHCI_INT_STATUS);
+                       sdhci_transfer_pio(host, data);
+                       data->dest += data->blocksize;
+                       if (++block >= data->blocks)
+                               break;
+               }
+#ifdef CONFIG_MMC_SDMA
+               if (stat & SDHCI_INT_DMA_END) {
+                       sdhci_writel(host, SDHCI_INT_DMA_END, SDHCI_INT_STATUS);
+                       start_addr &= SDHCI_DEFAULT_BOUNDARY_SIZE - 1;
+                       start_addr += SDHCI_DEFAULT_BOUNDARY_SIZE;
+                       sdhci_writel(host, start_addr, SDHCI_DMA_ADDRESS);
+               }
+#endif
+       } while (!(stat & SDHCI_INT_DATA_END));
+       return 0;
+}
+
+int sdhci_send_command(struct mmc *mmc, struct mmc_cmd *cmd,
+                      struct mmc_data *data)
+{
+       struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+       unsigned int stat = 0;
+       int ret = 0;
+       int trans_bytes = 0, is_aligned = 1;
+       u32 mask, flags, mode;
+       unsigned int timeout, start_addr = 0;
+
+       /* Wait max 10 ms */
+       timeout = 10;
+
+       sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_STATUS);
+       mask = SDHCI_CMD_INHIBIT | SDHCI_DATA_INHIBIT;
+
+       /* We shouldn't wait for data inihibit for stop commands, even
+          though they might use busy signaling */
+       if (cmd->cmdidx == MMC_CMD_STOP_TRANSMISSION)
+               mask &= ~SDHCI_DATA_INHIBIT;
+
+       while (sdhci_readl(host, SDHCI_PRESENT_STATE) & mask) {
+               if (timeout == 0) {
+                       printf("Controller never released inhibit bit(s).\n");
+                       return COMM_ERR;
+               }
+               timeout--;
+               udelay(1000);
+       }
+
+       mask = SDHCI_INT_RESPONSE;
+       if (!(cmd->resp_type & MMC_RSP_PRESENT))
+               flags = SDHCI_CMD_RESP_NONE;
+       else if (cmd->resp_type & MMC_RSP_136)
+               flags = SDHCI_CMD_RESP_LONG;
+       else if (cmd->resp_type & MMC_RSP_BUSY) {
+               flags = SDHCI_CMD_RESP_SHORT_BUSY;
+               mask |= SDHCI_INT_DATA_END;
+       } else
+               flags = SDHCI_CMD_RESP_SHORT;
+
+       if (cmd->resp_type & MMC_RSP_CRC)
+               flags |= SDHCI_CMD_CRC;
+       if (cmd->resp_type & MMC_RSP_OPCODE)
+               flags |= SDHCI_CMD_INDEX;
+       if (data)
+               flags |= SDHCI_CMD_DATA;
+
+       /*Set Transfer mode regarding to data flag*/
+       if (data != 0) {
+               sdhci_writeb(host, 0xe, SDHCI_TIMEOUT_CONTROL);
+               mode = SDHCI_TRNS_BLK_CNT_EN;
+               trans_bytes = data->blocks * data->blocksize;
+               if (data->blocks > 1)
+                       mode |= SDHCI_TRNS_MULTI;
+
+               if (data->flags == MMC_DATA_READ)
+                       mode |= SDHCI_TRNS_READ;
+
+#ifdef CONFIG_MMC_SDMA
+               if (data->flags == MMC_DATA_READ)
+                       start_addr = (unsigned int)data->dest;
+               else
+                       start_addr = (unsigned int)data->src;
+               if ((host->quirks & SDHCI_QUIRK_32BIT_DMA_ADDR) &&
+                               (start_addr & 0x7) != 0x0) {
+                       is_aligned = 0;
+                       start_addr = (unsigned int)aligned_buffer;
+                       if (data->flags != MMC_DATA_READ)
+                               memcpy(aligned_buffer, data->src, trans_bytes);
+               }
+
+               sdhci_writel(host, start_addr, SDHCI_DMA_ADDRESS);
+               mode |= SDHCI_TRNS_DMA;
+#endif
+               sdhci_writew(host, SDHCI_MAKE_BLKSZ(SDHCI_DEFAULT_BOUNDARY_ARG,
+                               data->blocksize),
+                               SDHCI_BLOCK_SIZE);
+               sdhci_writew(host, data->blocks, SDHCI_BLOCK_COUNT);
+               sdhci_writew(host, mode, SDHCI_TRANSFER_MODE);
+       }
+
+       sdhci_writel(host, cmd->cmdarg, SDHCI_ARGUMENT);
+#ifdef CONFIG_MMC_SDMA
+       flush_cache(0, ~0);
+#endif
+       sdhci_writew(host, SDHCI_MAKE_CMD(cmd->cmdidx, flags), SDHCI_COMMAND);
+       do {
+               stat = sdhci_readl(host, SDHCI_INT_STATUS);
+               if (stat & SDHCI_INT_ERROR)
+                       break;
+       } while ((stat & mask) != mask);
+
+       if ((stat & (SDHCI_INT_ERROR | mask)) == mask) {
+               sdhci_cmd_done(host, cmd);
+               sdhci_writel(host, mask, SDHCI_INT_STATUS);
+       } else
+               ret = -1;
+
+       if (!ret && data)
+               ret = sdhci_transfer_data(host, data, start_addr);
+
+       stat = sdhci_readl(host, SDHCI_INT_STATUS);
+       sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_STATUS);
+       if (!ret) {
+               if ((host->quirks & SDHCI_QUIRK_32BIT_DMA_ADDR) &&
+                               !is_aligned && (data->flags == MMC_DATA_READ))
+                       memcpy(data->dest, aligned_buffer, trans_bytes);
+               return 0;
+       }
+
+       sdhci_reset(host, SDHCI_RESET_CMD);
+       sdhci_reset(host, SDHCI_RESET_DATA);
+       if (stat & SDHCI_INT_TIMEOUT)
+               return TIMEOUT;
+       else
+               return COMM_ERR;
+}
+
+static int sdhci_set_clock(struct mmc *mmc, unsigned int clock)
+{
+       struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+       unsigned int div, clk, timeout;
+
+       sdhci_writew(host, 0, SDHCI_CLOCK_CONTROL);
+
+       if (clock == 0)
+               return 0;
+
+       if (host->version >= SDHCI_SPEC_300) {
+               /* Version 3.00 divisors must be a multiple of 2. */
+               if (mmc->f_max <= clock)
+                       div = 1;
+               else {
+                       for (div = 2; div < SDHCI_MAX_DIV_SPEC_300; div += 2) {
+                               if ((mmc->f_max / div) <= clock)
+                                       break;
+                       }
+               }
+       } else {
+               /* Version 2.00 divisors must be a power of 2. */
+               for (div = 1; div < SDHCI_MAX_DIV_SPEC_200; div *= 2) {
+                       if ((mmc->f_max / div) <= clock)
+                               break;
+               }
+       }
+       div >>= 1;
+
+       clk = (div & SDHCI_DIV_MASK) << SDHCI_DIVIDER_SHIFT;
+       clk |= ((div & SDHCI_DIV_HI_MASK) >> SDHCI_DIV_MASK_LEN)
+               << SDHCI_DIVIDER_HI_SHIFT;
+       clk |= SDHCI_CLOCK_INT_EN;
+       sdhci_writew(host, clk, SDHCI_CLOCK_CONTROL);
+
+       /* Wait max 20 ms */
+       timeout = 20;
+       while (!((clk = sdhci_readw(host, SDHCI_CLOCK_CONTROL))
+               & SDHCI_CLOCK_INT_STABLE)) {
+               if (timeout == 0) {
+                       printf("Internal clock never stabilised.\n");
+                       return -1;
+               }
+               timeout--;
+               udelay(1000);
+       }
+
+       clk |= SDHCI_CLOCK_CARD_EN;
+       sdhci_writew(host, clk, SDHCI_CLOCK_CONTROL);
+       return 0;
+}
+
+static void sdhci_set_power(struct sdhci_host *host, unsigned short power)
+{
+       u8 pwr = 0;
+
+       if (power != (unsigned short)-1) {
+               switch (1 << power) {
+               case MMC_VDD_165_195:
+                       pwr = SDHCI_POWER_180;
+                       break;
+               case MMC_VDD_29_30:
+               case MMC_VDD_30_31:
+                       pwr = SDHCI_POWER_300;
+                       break;
+               case MMC_VDD_32_33:
+               case MMC_VDD_33_34:
+                       pwr = SDHCI_POWER_330;
+                       break;
+               }
+       }
+
+       if (pwr == 0) {
+               sdhci_writeb(host, 0, SDHCI_POWER_CONTROL);
+               return;
+       }
+
+       pwr |= SDHCI_POWER_ON;
+
+       sdhci_writeb(host, pwr, SDHCI_POWER_CONTROL);
+}
+
+void sdhci_set_ios(struct mmc *mmc)
+{
+       u32 ctrl;
+       struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+
+       if (mmc->clock != host->clock)
+               sdhci_set_clock(mmc, mmc->clock);
+
+       /* Set bus width */
+       ctrl = sdhci_readb(host, SDHCI_HOST_CONTROL);
+       if (mmc->bus_width == 8) {
+               ctrl &= ~SDHCI_CTRL_4BITBUS;
+               if (host->version >= SDHCI_SPEC_300)
+                       ctrl |= SDHCI_CTRL_8BITBUS;
+       } else {
+               if (host->version >= SDHCI_SPEC_300)
+                       ctrl &= ~SDHCI_CTRL_8BITBUS;
+               if (mmc->bus_width == 4)
+                       ctrl |= SDHCI_CTRL_4BITBUS;
+               else
+                       ctrl &= ~SDHCI_CTRL_4BITBUS;
+       }
+
+       if (mmc->clock > 26000000)
+               ctrl |= SDHCI_CTRL_HISPD;
+       else
+               ctrl &= ~SDHCI_CTRL_HISPD;
+
+       sdhci_writeb(host, ctrl, SDHCI_HOST_CONTROL);
+}
+
+int sdhci_init(struct mmc *mmc)
+{
+       struct sdhci_host *host = (struct sdhci_host *)mmc->priv;
+
+       if ((host->quirks & SDHCI_QUIRK_32BIT_DMA_ADDR) && !aligned_buffer) {
+               aligned_buffer = memalign(8, 512*1024);
+               if (!aligned_buffer) {
+                       printf("Aligned buffer alloc failed!!!");
+                       return -1;
+               }
+       }
+
+       /* Eable all state */
+       sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_ENABLE);
+       sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_SIGNAL_ENABLE);
+
+       sdhci_set_power(host, fls(mmc->voltages) - 1);
+
+       return 0;
+}
+
+int add_sdhci(struct sdhci_host *host, u32 max_clk, u32 min_clk)
+{
+       struct mmc *mmc;
+       unsigned int caps;
+
+       mmc = malloc(sizeof(struct mmc));
+       if (!mmc) {
+               printf("mmc malloc fail!\n");
+               return -1;
+       }
+
+       mmc->priv = host;
+
+       sprintf(mmc->name, "%s", host->name);
+       mmc->send_cmd = sdhci_send_command;
+       mmc->set_ios = sdhci_set_ios;
+       mmc->init = sdhci_init;
+
+       caps = sdhci_readl(host, SDHCI_CAPABILITIES);
+#ifdef CONFIG_MMC_SDMA
+       if (!(caps & SDHCI_CAN_DO_SDMA)) {
+               printf("Your controller don't support sdma!!\n");
+               return -1;
+       }
+#endif
+
+       if (max_clk)
+               mmc->f_max = max_clk;
+       else {
+               if (host->version >= SDHCI_SPEC_300)
+                       mmc->f_max = (caps & SDHCI_CLOCK_V3_BASE_MASK)
+                               >> SDHCI_CLOCK_BASE_SHIFT;
+               else
+                       mmc->f_max = (caps & SDHCI_CLOCK_BASE_MASK)
+                               >> SDHCI_CLOCK_BASE_SHIFT;
+               mmc->f_max *= 1000000;
+       }
+       if (mmc->f_max == 0) {
+               printf("Hardware doesn't specify base clock frequency\n");
+               return -1;
+       }
+       if (min_clk)
+               mmc->f_min = min_clk;
+       else {
+               if (host->version >= SDHCI_SPEC_300)
+                       mmc->f_min = mmc->f_max / SDHCI_MAX_DIV_SPEC_300;
+               else
+                       mmc->f_min = mmc->f_max / SDHCI_MAX_DIV_SPEC_200;
+       }
+
+       mmc->voltages = 0;
+       if (caps & SDHCI_CAN_VDD_330)
+               mmc->voltages |= MMC_VDD_32_33 | MMC_VDD_33_34;
+       if (caps & SDHCI_CAN_VDD_300)
+               mmc->voltages |= MMC_VDD_29_30 | MMC_VDD_30_31;
+       if (caps & SDHCI_CAN_VDD_180)
+               mmc->voltages |= MMC_VDD_165_195;
+       mmc->host_caps = MMC_MODE_HS | MMC_MODE_HS_52MHz | MMC_MODE_4BIT;
+       if (caps & SDHCI_CAN_DO_8BIT)
+               mmc->host_caps |= MMC_MODE_8BIT;
+
+       sdhci_reset(host, SDHCI_RESET_ALL);
+       mmc_register(mmc);
+
+       return 0;
+}
include/sdhci.h new file +325 -1
new file mode 100644
index 0000000..6d52ce9
@@ -0,0 +1,325 @@
+/*
+ * Copyright 2011, Marvell Semiconductor Inc.
+ * Lei Wen <[email protected]>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ *
+ * Back ported to the 8xx platform (from the 8260 platform) by
+ * Murray.Jensen at cmst.csiro.au, 27-Jan-01.
+ */
+#ifndef __SDHCI_HW_H
+#define __SDHCI_HW_H
+
+#include <asm/io.h>
+/*
+ * Controller registers
+ */
+
+#define SDHCI_DMA_ADDRESS      0x00
+
+#define SDHCI_BLOCK_SIZE       0x04
+#define  SDHCI_MAKE_BLKSZ(dma, blksz) (((dma & 0x7) << 12) | (blksz & 0xFFF))
+
+#define SDHCI_BLOCK_COUNT      0x06
+
+#define SDHCI_ARGUMENT         0x08
+
+#define SDHCI_TRANSFER_MODE    0x0C
+#define  SDHCI_TRNS_DMA                0x01
+#define  SDHCI_TRNS_BLK_CNT_EN 0x02
+#define  SDHCI_TRNS_ACMD12     0x04
+#define  SDHCI_TRNS_READ       0x10
+#define  SDHCI_TRNS_MULTI      0x20
+
+#define SDHCI_COMMAND          0x0E
+#define  SDHCI_CMD_RESP_MASK   0x03
+#define  SDHCI_CMD_CRC         0x08
+#define  SDHCI_CMD_INDEX       0x10
+#define  SDHCI_CMD_DATA                0x20
+#define  SDHCI_CMD_ABORTCMD    0xC0
+
+#define  SDHCI_CMD_RESP_NONE   0x00
+#define  SDHCI_CMD_RESP_LONG   0x01
+#define  SDHCI_CMD_RESP_SHORT  0x02
+#define  SDHCI_CMD_RESP_SHORT_BUSY 0x03
+
+#define SDHCI_MAKE_CMD(c, f) (((c & 0xff) << 8) | (f & 0xff))
+#define SDHCI_GET_CMD(c) ((c>>8) & 0x3f)
+
+#define SDHCI_RESPONSE         0x10
+
+#define SDHCI_BUFFER           0x20
+
+#define SDHCI_PRESENT_STATE    0x24
+#define  SDHCI_CMD_INHIBIT     0x00000001
+#define  SDHCI_DATA_INHIBIT    0x00000002
+#define  SDHCI_DOING_WRITE     0x00000100
+#define  SDHCI_DOING_READ      0x00000200
+#define  SDHCI_SPACE_AVAILABLE 0x00000400
+#define  SDHCI_DATA_AVAILABLE  0x00000800
+#define  SDHCI_CARD_PRESENT    0x00010000
+#define  SDHCI_WRITE_PROTECT   0x00080000
+
+#define SDHCI_HOST_CONTROL     0x28
+#define  SDHCI_CTRL_LED                0x01
+#define  SDHCI_CTRL_4BITBUS    0x02
+#define  SDHCI_CTRL_HISPD      0x04
+#define  SDHCI_CTRL_DMA_MASK   0x18
+#define   SDHCI_CTRL_SDMA      0x00
+#define   SDHCI_CTRL_ADMA1     0x08
+#define   SDHCI_CTRL_ADMA32    0x10
+#define   SDHCI_CTRL_ADMA64    0x18
+#define   SDHCI_CTRL_8BITBUS   0x20
+
+#define SDHCI_POWER_CONTROL    0x29
+#define  SDHCI_POWER_ON                0x01
+#define  SDHCI_POWER_180       0x0A
+#define  SDHCI_POWER_300       0x0C
+#define  SDHCI_POWER_330       0x0E
+
+#define SDHCI_BLOCK_GAP_CONTROL        0x2A
+
+#define SDHCI_WAKE_UP_CONTROL  0x2B
+#define  SDHCI_WAKE_ON_INT     0x01
+#define  SDHCI_WAKE_ON_INSERT  0x02
+#define  SDHCI_WAKE_ON_REMOVE  0x04
+
+#define SDHCI_CLOCK_CONTROL    0x2C
+#define  SDHCI_DIVIDER_SHIFT   8
+#define  SDHCI_DIVIDER_HI_SHIFT        6
+#define  SDHCI_DIV_MASK        0xFF
+#define  SDHCI_DIV_MASK_LEN    8
+#define  SDHCI_DIV_HI_MASK     0x300
+#define  SDHCI_CLOCK_CARD_EN   0x0004
+#define  SDHCI_CLOCK_INT_STABLE        0x0002
+#define  SDHCI_CLOCK_INT_EN    0x0001
+
+#define SDHCI_TIMEOUT_CONTROL  0x2E
+
+#define SDHCI_SOFTWARE_RESET   0x2F
+#define  SDHCI_RESET_ALL       0x01
+#define  SDHCI_RESET_CMD       0x02
+#define  SDHCI_RESET_DATA      0x04
+
+#define SDHCI_INT_STATUS       0x30
+#define SDHCI_INT_ENABLE       0x34
+#define SDHCI_SIGNAL_ENABLE    0x38
+#define  SDHCI_INT_RESPONSE    0x00000001
+#define  SDHCI_INT_DATA_END    0x00000002
+#define  SDHCI_INT_DMA_END     0x00000008
+#define  SDHCI_INT_SPACE_AVAIL 0x00000010
+#define  SDHCI_INT_DATA_AVAIL  0x00000020
+#define  SDHCI_INT_CARD_INSERT 0x00000040
+#define  SDHCI_INT_CARD_REMOVE 0x00000080
+#define  SDHCI_INT_CARD_INT    0x00000100
+#define  SDHCI_INT_ERROR       0x00008000
+#define  SDHCI_INT_TIMEOUT     0x00010000
+#define  SDHCI_INT_CRC         0x00020000
+#define  SDHCI_INT_END_BIT     0x00040000
+#define  SDHCI_INT_INDEX       0x00080000
+#define  SDHCI_INT_DATA_TIMEOUT        0x00100000
+#define  SDHCI_INT_DATA_CRC    0x00200000
+#define  SDHCI_INT_DATA_END_BIT        0x00400000
+#define  SDHCI_INT_BUS_POWER   0x00800000
+#define  SDHCI_INT_ACMD12ERR   0x01000000
+#define  SDHCI_INT_ADMA_ERROR  0x02000000
+
+#define  SDHCI_INT_NORMAL_MASK 0x00007FFF
+#define  SDHCI_INT_ERROR_MASK  0xFFFF8000
+
+#define  SDHCI_INT_CMD_MASK    (SDHCI_INT_RESPONSE | SDHCI_INT_TIMEOUT | \
+               SDHCI_INT_CRC | SDHCI_INT_END_BIT | SDHCI_INT_INDEX)
+#define  SDHCI_INT_DATA_MASK   (SDHCI_INT_DATA_END | SDHCI_INT_DMA_END | \
+               SDHCI_INT_DATA_AVAIL | SDHCI_INT_SPACE_AVAIL | \
+               SDHCI_INT_DATA_TIMEOUT | SDHCI_INT_DATA_CRC | \
+               SDHCI_INT_DATA_END_BIT | SDHCI_INT_ADMA_ERROR)
+#define SDHCI_INT_ALL_MASK     ((unsigned int)-1)
+
+#define SDHCI_ACMD12_ERR       0x3C
+
+/* 3E-3F reserved */
+
+#define SDHCI_CAPABILITIES     0x40
+#define  SDHCI_TIMEOUT_CLK_MASK        0x0000003F
+#define  SDHCI_TIMEOUT_CLK_SHIFT 0
+#define  SDHCI_TIMEOUT_CLK_UNIT        0x00000080
+#define  SDHCI_CLOCK_BASE_MASK 0x00003F00
+#define  SDHCI_CLOCK_V3_BASE_MASK      0x0000FF00
+#define  SDHCI_CLOCK_BASE_SHIFT        8
+#define  SDHCI_MAX_BLOCK_MASK  0x00030000
+#define  SDHCI_MAX_BLOCK_SHIFT  16
+#define  SDHCI_CAN_DO_8BIT     0x00040000
+#define  SDHCI_CAN_DO_ADMA2    0x00080000
+#define  SDHCI_CAN_DO_ADMA1    0x00100000
+#define  SDHCI_CAN_DO_HISPD    0x00200000
+#define  SDHCI_CAN_DO_SDMA     0x00400000
+#define  SDHCI_CAN_VDD_330     0x01000000
+#define  SDHCI_CAN_VDD_300     0x02000000
+#define  SDHCI_CAN_VDD_180     0x04000000
+#define  SDHCI_CAN_64BIT       0x10000000
+
+#define SDHCI_CAPABILITIES_1   0x44
+
+#define SDHCI_MAX_CURRENT      0x48
+
+/* 4C-4F reserved for more max current */
+
+#define SDHCI_SET_ACMD12_ERROR 0x50
+#define SDHCI_SET_INT_ERROR    0x52
+
+#define SDHCI_ADMA_ERROR       0x54
+
+/* 55-57 reserved */
+
+#define SDHCI_ADMA_ADDRESS     0x58
+
+/* 60-FB reserved */
+
+#define SDHCI_SLOT_INT_STATUS  0xFC
+
+#define SDHCI_HOST_VERSION     0xFE
+#define  SDHCI_VENDOR_VER_MASK 0xFF00
+#define  SDHCI_VENDOR_VER_SHIFT        8
+#define  SDHCI_SPEC_VER_MASK   0x00FF
+#define  SDHCI_SPEC_VER_SHIFT  0
+#define   SDHCI_SPEC_100       0
+#define   SDHCI_SPEC_200       1
+#define   SDHCI_SPEC_300       2
+
+/*
+ * End of controller registers.
+ */
+
+#define SDHCI_MAX_DIV_SPEC_200 256
+#define SDHCI_MAX_DIV_SPEC_300 2046
+
+/*
+ * quirks
+ */
+#define SDHCI_QUIRK_32BIT_DMA_ADDR     (1 << 0)
+
+/*
+ * Host SDMA buffer boundary. Valid values from 4K to 512K in powers of 2.
+ */
+#define SDHCI_DEFAULT_BOUNDARY_SIZE    (512 * 1024)
+#define SDHCI_DEFAULT_BOUNDARY_ARG     (7)
+struct sdhci_ops {
+#ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS
+       u32             (*read_l)(struct sdhci_host *host, int reg);
+       u16             (*read_w)(struct sdhci_host *host, int reg);
+       u8              (*read_b)(struct sdhci_host *host, int reg);
+       void            (*write_l)(struct sdhci_host *host, u32 val, int reg);
+       void            (*write_w)(struct sdhci_host *host, u16 val, int reg);
+       void            (*write_b)(struct sdhci_host *host, u8 val, int reg);
+#endif
+};
+
+struct sdhci_host {
+       char *name;
+       void *ioaddr;
+       unsigned int quirks;
+       unsigned int version;
+       unsigned int clock;
+       const struct sdhci_ops *ops;
+};
+
+#ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS
+
+static inline void sdhci_writel(struct sdhci_host *host, u32 val, int reg)
+{
+       if (unlikely(host->ops->write_l))
+               host->ops->write_l(host, val, reg);
+       else
+               writel(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writew(struct sdhci_host *host, u16 val, int reg)
+{
+       if (unlikely(host->ops->write_w))
+               host->ops->write_w(host, val, reg);
+       else
+               writew(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writeb(struct sdhci_host *host, u8 val, int reg)
+{
+       if (unlikely(host->ops->write_b))
+               host->ops->write_b(host, val, reg);
+       else
+               writeb(val, host->ioaddr + reg);
+}
+
+static inline u32 sdhci_readl(struct sdhci_host *host, int reg)
+{
+       if (unlikely(host->ops->read_l))
+               return host->ops->read_l(host, reg);
+       else
+               return readl(host->ioaddr + reg);
+}
+
+static inline u16 sdhci_readw(struct sdhci_host *host, int reg)
+{
+       if (unlikely(host->ops->read_w))
+               return host->ops->read_w(host, reg);
+       else
+               return readw(host->ioaddr + reg);
+}
+
+static inline u8 sdhci_readb(struct sdhci_host *host, int reg)
+{
+       if (unlikely(host->ops->read_b))
+               return host->ops->read_b(host, reg);
+       else
+               return readb(host->ioaddr + reg);
+}
+
+#else
+
+static inline void sdhci_writel(struct sdhci_host *host, u32 val, int reg)
+{
+       writel(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writew(struct sdhci_host *host, u16 val, int reg)
+{
+       writew(val, host->ioaddr + reg);
+}
+
+static inline void sdhci_writeb(struct sdhci_host *host, u8 val, int reg)
+{
+       writeb(val, host->ioaddr + reg);
+}
+static inline u32 sdhci_readl(struct sdhci_host *host, int reg)
+{
+       return readl(host->ioaddr + reg);
+}
+
+static inline u16 sdhci_readw(struct sdhci_host *host, int reg)
+{
+       return readw(host->ioaddr + reg);
+}
+
+static inline u8 sdhci_readb(struct sdhci_host *host, int reg)
+{
+       return readb(host->ioaddr + reg);
+}
+#endif
+
+int add_sdhci(struct sdhci_host *host, u32 max_clk, u32 min_clk);
+#endif /* __SDHCI_HW_H */
--

1.7.0.4

Quoted 5 lines

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Signature · 4 lines
A+ - Amicalement - Gk2 [:-]
---
Un arc-en-ciel ?a n'existe pas ! Et pourtant tout le monde en a d?j?
vu... (Albert Jacquart)
#4 lore raw

PATCH v2 1/2 MMC: add sdhci generic framework

Hi G?rald,

Quoted 3 lines · G?rald Kerma

Hello,

Is this driver adaptable for sheevaplug ?

Sheevaplug use kirkwood series, which is different from the platform current develop
on. But since this is a generic framework, any controller follow sdhci host spec certainly
could be supported.

In the Linux kernel, the sdhci-dove.c does the similar way to support the kirkwood.

Best regards,
Lei

#5 lore raw

PATCH 1/2 MMC: add sdhci generic framework

Hello Lei,

Le 29/06/2011 07:58, Lei Wen a ?crit :

Quoted 9 lines

Hi G?rald,

Quoted 3 lines · G?rald Kerma
Quoted 3 lines

Hello,

Is this driver adaptable for sheevaplug ?

In the Linux kernel, the sdhci-dove.c does the similar way to support the kirkwood.

The linux kernel use mvsdio driver for kirkwood.

MMC_MVSDIO
Marvell MMC/SD/SDIO host driver
This selects the Marvell SDIO host driver.
SDIO may currently be found on the Kirkwood 88F6281 and 88F6192 SoC controllers.

MMC_SDHCI_DOVE
SDHCI support on Marvell's Dove SoC
This selects the Secure Digital Host Controller Interface in Marvell's Dove SoC.

a driver patch which use the old generic driver was adapted from Marvell UBoot (mvsdio).

It's not included in the Denx uboot mainline but is available in the Debian Uboot.

Signature · 4 lines
A+ - Amicalement - Gk2 [:-]
---
Un arc-en-ciel ?a n'existe pas ! Et pourtant tout le monde en a d?j?
vu... (Albert Jacquart)
#6 lore raw

PATCH v2 1/2 MMC: add sdhci generic framework

Hi G?rald,

Quoted 26 lines · G?rald Kerma

Hello Lei,

Le 29/06/2011 07:58, Lei Wen a ?crit :

Quoted 9 lines

Hi G?rald,

Quoted 3 lines · G?rald Kerma
Quoted 3 lines

Hello,

Is this driver adaptable for sheevaplug ?

In the Linux kernel, the sdhci-dove.c does the similar way to support the kirkwood.

The linux kernel use mvsdio driver for kirkwood.

MMC_MVSDIO
Marvell MMC/SD/SDIO host driver
? ?This selects the Marvell SDIO host driver.
? ?SDIO may currently be found on the Kirkwood 88F6281 and 88F6192 ? ?SoC controllers.

MMC_SDHCI_DOVE
SDHCI support on Marvell's Dove SoC
? ?This selects the Secure Digital Host Controller Interface in ? ?Marvell's Dove SoC.

Quoted 6 lines

a driver patch which use the old generic driver was adapted from Marvell UBoot (mvsdio).

It's not included in the Denx uboot mainline but is available in the Debian Uboot.

I just don't object kirkwood to resue the old way. But since sdhci could be a generic
framework, it is nice that denx mainline could also use sdhci for kirkwood which is
actively maintained like in the linux kernel.

Best regards,
Lei

#7 lore raw

PATCH 1/2 MMC: add sdhci generic framework

Quoted 49 lines

-----Original Message-----
From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Lei Wen
Sent: Sunday, July 17, 2011 12:54 PM
To: G?rald Kerma
Cc: Lei Wen; Yu Tang; Rob Herring; u-boot at lists.denx.de; Andy Fleming; Ashish Karkare; Prabhanjan Sarnaik
Subject: Re: [U-Boot] [PATCH 1/2] MMC: add sdhci generic framework

Hi G?rald,

Quoted 6 lines · G?rald Kerma

Hello Lei,

Le 29/06/2011 07:58, Lei Wen a ?crit :

Quoted 3 lines

Hi G?rald,

On Wed, Jun 29, 2011 at 1:53 PM, G?rald Kerma <[email protected]>

Quoted 15 lines · wrote:
Quoted 6 lines
Quoted 3 lines
Quoted 3 lines

Hello,

Is this driver adaptable for sheevaplug ?

In the Linux kernel, the sdhci-dove.c does the similar way to support the kirkwood.

The linux kernel use mvsdio driver for kirkwood.

MMC_MVSDIO
Marvell MMC/SD/SDIO host driver
? ?This selects the Marvell SDIO host driver.
? ?SDIO may currently be found on the Kirkwood 88F6281 and 88F6192 ? ?SoC controllers.

Quoted 6 lines

MMC_SDHCI_DOVE
SDHCI support on Marvell's Dove SoC
? ?This selects the Secure Digital Host Controller Interface in ? ?Marvell's Dove SoC.

a driver patch which use the old generic driver was adapted from

Marvell

Quoted 5 lines

UBoot (mvsdio).

It's not included in the Denx uboot mainline but is available in the Debian Uboot.

Quoted 5 lines

I just don't object kirkwood to resue the old way. But since sdhci could be a generic
framework, it is nice that denx mainline could also use sdhci for kirkwood which is
actively maintained like in the linux kernel.

I don't see an issue here, if sdhci is more generic, it will support few more SoCs then it should go for mainlining.

Regards..
Prafulla . . .

Quoted 7 lines

Best regards,
Lei
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

#8 lore raw

PATCH 1/2 MMC: add sdhci generic framework

In reply to Lei Wen
Quoted 10 lines

-----Original Message-----
From: Lei Wen [mailto:adrian.wenl at gmail.com]
Sent: Wednesday, June 29, 2011 9:17 AM
To: Lei Wen
Cc: Andy Fleming; Rob Herring; u-boot at lists.denx.de; Prafulla Wadaskar; Yu Tang; Prabhanjan Sarnaik; Ashish Karkare
Subject: Re: [U-Boot] [PATCH 1/2] MMC: add sdhci generic framework

Quoted 3 lines

Hi All,

Any comment to this patch set, could it be merged in this open window?

Hi Lei,
I ack for the driver.
Can you re-submit v2 patches as per review comments for v1?

Copying Andy Fleming on his alternate email id.

Regards..
Prafulla . .

#9 lore raw

PATCH v2 1/2 MMC: add sdhci generic framework

Hi Prafulla,

Quoted 10 lines · Prafulla Wadaskar
Quoted 10 lines

-----Original Message-----
From: Lei Wen [mailto:adrian.wenl at gmail.com]
Sent: Wednesday, June 29, 2011 9:17 AM
To: Lei Wen
Cc: Andy Fleming; Rob Herring; u-boot at lists.denx.de; Prafulla Wadaskar; Yu Tang; Prabhanjan Sarnaik; Ashish Karkare
Subject: Re: [U-Boot] [PATCH 1/2] MMC: add sdhci generic framework

Quoted 7 lines
Quoted 3 lines

Hi All,

Any comment to this patch set, could it be merged in this open window?

Hi Lei,
I ack for the driver.
Can you re-submit v2 patches as per review comments for v1?

Ok, I would push the v2 patch later.

Thanks,
Lei

author
#10 lore raw

v2 cover add sdhci generic framework

In reply to Lei Wen

V1:
add sdhci generic framework and with marvell sdhci implementation

V2:
rename the previous file name from sdhci-mv to mv_sdhci

Lei Wen (2):
MMC: add sdhci generic framework
MMC: add marvell sdhci driver

 drivers/mmc/Makefile   |    2 +
 drivers/mmc/mv_sdhci.c |   21 +++
 drivers/mmc/sdhci.c    |  433 ++++++++++++++++++++++++++++++++++++++++++++++++
 include/sdhci.h        |  325 ++++++++++++++++++++++++++++++++++++
 4 files changed, 781 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/mv_sdhci.c
 create mode 100644 drivers/mmc/sdhci.c
 create mode 100644 include/sdhci.h
#11 lore raw

v2 cover add sdhci generic framework

Hi Andy,

Could this version be accepted to be merged?

Thanks,
Lei

Quoted 24 lines · Lei Wen

V1:
add sdhci generic framework and with marvell sdhci implementation

V2:
rename the previous file name from sdhci-mv to mv_sdhci

Lei Wen (2):
?MMC: add sdhci generic framework
?MMC: add marvell sdhci driver

?drivers/mmc/Makefile ? | ? ?2 +
?drivers/mmc/mv_sdhci.c | ? 21 +++
?drivers/mmc/sdhci.c ? ?| ?433 ++++++++++++++++++++++++++++++++++++++++++++++++
?include/sdhci.h ? ? ? ?| ?325 ++++++++++++++++++++++++++++++++++++

?4 files changed, 781 insertions(+), 0 deletions(-) ?create mode 100644 drivers/mmc/mv_sdhci.c ?create mode 100644 drivers/mmc/sdhci.c ?create mode 100644 include/sdhci.h

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

#12 lore raw

v2 cover add sdhci generic framework

Quoted 3 lines · Lei Wen

Hi Andy,

Could this version be accepted to be merged?

I'm working on merging these (and other) patches this weekend.

Andy

#13 lore raw

v2 cover add sdhci generic framework

In reply to Lei Wen

Hi Wolfgang,

How about merge this for the next release?

Thanks,
Lei

Quoted 24 lines · Lei Wen

V1:
add sdhci generic framework and with marvell sdhci implementation

V2:
rename the previous file name from sdhci-mv to mv_sdhci

Lei Wen (2):
?MMC: add sdhci generic framework
?MMC: add marvell sdhci driver

?drivers/mmc/Makefile ? | ? ?2 +
?drivers/mmc/mv_sdhci.c | ? 21 +++
?drivers/mmc/sdhci.c ? ?| ?433 ++++++++++++++++++++++++++++++++++++++++++++++++
?include/sdhci.h ? ? ? ?| ?325 ++++++++++++++++++++++++++++++++++++

?4 files changed, 781 insertions(+), 0 deletions(-) ?create mode 100644 drivers/mmc/mv_sdhci.c ?create mode 100644 drivers/mmc/sdhci.c ?create mode 100644 include/sdhci.h

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

#14 lore raw

v2 cover add sdhci generic framework

In reply to Lei Wen

Hello Lei,

I am looking into adding eMMC support for some intel SDHCI controllers based on this common SDHCI driver submission.

One thing which is not quite clear is the quirks - can you (or somebody else on this list) please shed some light on what the quirks are. Is there some kind of formal description/definition of various defined quirks and their processing.

Thank you in advance,

--vb

Quoted 23 lines · Lei Wen

V1:
add sdhci generic framework and with marvell sdhci implementation

V2:
rename the previous file name from sdhci-mv to mv_sdhci

Lei Wen (2):
MMC: add sdhci generic framework
MMC: add marvell sdhci driver

 drivers/mmc/Makefile   |    2 +
 drivers/mmc/mv_sdhci.c |   21 +++
 drivers/mmc/sdhci.c    |  433 ++++++++++++++++++++++++++++++++++++++++++++++++
 include/sdhci.h        |  325 ++++++++++++++++++++++++++++++++++++
 4 files changed, 781 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/mv_sdhci.c
 create mode 100644 drivers/mmc/sdhci.c
 create mode 100644 include/sdhci.h

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

author
#15 lore raw

v2 cover add sdhci generic framework

Hi Vadim,

The quirk is standing for some workaround for those host that has some limitation that it cannot directly be applied with standard sdhci.c code, so that we add one quirk for it, which let it could use the sdhci.c.

Like the quirk as SDHCI_QUIRK_32BIT_DMA_ADDR, it indicate that some host has dma engine limitation that it must start from address with alignement. So if sdhci find caller set this quirk, it would follow corresponding workaround for it.

There is no formal description for it as I could see.

Thanks,
Lei

Quoted 49 lines

-----Original Message-----
From: vbendeb at gmail.com [mailto:vbendeb at gmail.com] On Behalf Of Vadim Bendebury
Sent: Tuesday, October 15, 2013 10:09 AM
To: Lei Wen
Cc: Prafulla Wadaskar; afleming at gmail.com; Rob Herring; u- boot at lists.denx.de; Yu Tang; Prabhanjan Sarnaik; Ashish Karkare; Andy Fleming
Subject: Re: [U-Boot] [PATCH V2 0/2] add sdhci generic framework

Hello Lei,

I am looking into adding eMMC support for some intel SDHCI controllers based on this common SDHCI driver submission.

One thing which is not quite clear is the quirks - can you (or somebody else on this list) please shed some light on what the quirks are. Is there some kind of formal description/definition of various defined quirks and their processing.

Thank you in advance,

--vb

Quoted 6 lines · Lei Wen

V1:
add sdhci generic framework and with marvell sdhci implementation

V2:
rename the previous file name from sdhci-mv to mv_sdhci

Quoted 7 lines

Lei Wen (2):
MMC: add sdhci generic framework
MMC: add marvell sdhci driver

 drivers/mmc/Makefile   |    2 +
 drivers/mmc/mv_sdhci.c |   21 +++
 drivers/mmc/sdhci.c    |  433

++++++++++++++++++++++++++++++++++++++++++++++++

Quoted 9 lines
 include/sdhci.h        |  325 ++++++++++++++++++++++++++++++++++++
 4 files changed, 781 insertions(+), 0 deletions(-)  create mode

100644 drivers/mmc/mv_sdhci.c create mode 100644 drivers/mmc/sdhci.c create mode 100644 include/sdhci.h

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

#16 lore raw

v2 cover add sdhci generic framework

Quoted 6 lines · Lei Wen

Hi Vadim,

The quirk is standing for some workaround for those host that has some limitation that it cannot directly be applied with standard sdhci.c code, so that we add one quirk for it, which let it could use the sdhci.c.

Quoted 7 lines

Like the quirk as SDHCI_QUIRK_32BIT_DMA_ADDR, it indicate that some host has dma engine limitation that it must start from address with alignement. So if sdhci find caller set this quirk, it would follow corresponding workaround for it.

There is no formal description for it as I could see.

All right, that's what I figured :). Thank you for the explanation,

--vb

Quoted 56 lines

Thanks,
Lei

Quoted 32 lines

-----Original Message-----
From: vbendeb at gmail.com [mailto:vbendeb at gmail.com] On Behalf Of Vadim Bendebury
Sent: Tuesday, October 15, 2013 10:09 AM
To: Lei Wen
Cc: Prafulla Wadaskar; afleming at gmail.com; Rob Herring; u- boot at lists.denx.de; Yu Tang; Prabhanjan Sarnaik; Ashish Karkare; Andy Fleming
Subject: Re: [U-Boot] [PATCH V2 0/2] add sdhci generic framework

Hello Lei,

I am looking into adding eMMC support for some intel SDHCI controllers based on this common SDHCI driver submission.

One thing which is not quite clear is the quirks - can you (or somebody else on this list) please shed some light on what the quirks are. Is there some kind of formal description/definition of various defined quirks and their processing.

Thank you in advance,

--vb

Quoted 6 lines · Lei Wen

V1:
add sdhci generic framework and with marvell sdhci implementation

V2:
rename the previous file name from sdhci-mv to mv_sdhci

Quoted 17 lines
Quoted 7 lines

Lei Wen (2):
MMC: add sdhci generic framework
MMC: add marvell sdhci driver

 drivers/mmc/Makefile   |    2 +
 drivers/mmc/mv_sdhci.c |   21 +++
 drivers/mmc/sdhci.c    |  433

++++++++++++++++++++++++++++++++++++++++++++++++

Quoted 9 lines
 include/sdhci.h        |  325 ++++++++++++++++++++++++++++++++++++
 4 files changed, 781 insertions(+), 0 deletions(-)  create mode

100644 drivers/mmc/mv_sdhci.c create mode 100644 drivers/mmc/sdhci.c create mode 100644 include/sdhci.h

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

author
#17 lore raw

PATCH v2 2/2 MMC: add marvell sdhci driver

In reply to Lei Wen

This could support both armada100 and pantheon serial in the mainline, while this driver also be tested to support upcoming mg, mmp2 and mmp3 hardware.

---
 drivers/mmc/Makefile   |    1 +
 drivers/mmc/sdhci-mv.c |   21 +++++++++++++++++++++
 2 files changed, 22 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/sdhci-mv.c
drivers/mmc/Makefile +1 -0
index 50b5117..fd84389 100644
@@ -39,6 +39,7 @@ COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
 COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
 COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
 COBJS-$(CONFIG_SDHCI) += sdhci.o
+COBJS-$(CONFIG_SDHCI_MV) += sdhci-mv.o
 
 COBJS	:= $(COBJS-y)
 SRCS	:= $(COBJS:.o=.c)
drivers/mmc/sdhci-mv.c new file +21 -0
new file mode 100644
index 0000000..97d79ee
@@ -0,0 +1,21 @@
+#include <common.h>
+#include <malloc.h>
+#include <sdhci.h>
+
+static char *MVSDH_NAME = "sdh-mv";
+int mv_sdh_init(u32 regbase, u32 max_clk, u32 min_clk, u32 quirks)
+{
+	struct sdhci_host *host = NULL;
+	host = (struct sdhci_host *)malloc(sizeof(struct sdhci_host));
+	if (!host) {
+		printf("sdh_host malloc fail!\n");
+		return 1;
+	}
+
+	host->name = MVSDH_NAME;
+	host->ioaddr = (void *)regbase;
+	host->quirks = quirks;
+	host->version = sdhci_readw(host, SDHCI_HOST_VERSION);
+	add_sdhci(host, max_clk, min_clk);
+	return 0;
+}
Signature · 1 line
1.7.0.4
#18 lore raw

PATCH 2/2 MMC: add marvell sdhci driver

Quoted 9 lines

-----Original Message-----
From: Lei Wen [mailto:leiwen at marvell.com]
Sent: Thursday, June 16, 2011 8:48 PM
To: Andy Fleming; Rob Herring; u-boot at lists.denx.de; Prafulla Wadaskar; Yu Tang; Prabhanjan Sarnaik; Ashish Karkare
Subject: [PATCH 2/2] MMC: add marvell sdhci driver

Quoted 8 lines

This could support both armada100 and pantheon serial in the mainline, while this driver also be tested to support upcoming mg, mmp2 and mmp3 hardware.

---
 drivers/mmc/Makefile   |    1 +
 drivers/mmc/sdhci-mv.c |   21 +++++++++++++++++++++

The file name should be mv_sdhci.c

drivers/mmc/Makefile COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o · line 42

Quoted 12 lines
 2 files changed, 22 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/sdhci-mv.c
drivers/mmc/Makefile +1 -0
index 50b5117..fd84389 100644
@@ -39,6 +39,7 @@ COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
 COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
 COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
 COBJS-$(CONFIG_SDHCI) += sdhci.o
+COBJS-$(CONFIG_SDHCI_MV) += sdhci-mv.o

Same here CONFIG_MV_SDHCI

Quoted 9 lines
drivers/mmc/Makefile +0 -0
 COBJS	:= $(COBJS-y)
 SRCS	:= $(COBJS:.o=.c)
drivers/mmc/sdhci-mv.c new file +0 -0
new file mode 100644
index 0000000..97d79ee

drivers/mmc/sdhci-mv.c line 5

Quoted 5 lines
drivers/mmc/sdhci-mv.c new file +5 -0
@@ -0,0 +1,21 @@
+#include <common.h>
+#include <malloc.h>
+#include <sdhci.h>
+
+static char *MVSDH_NAME = "sdh-mv";

"mv_sdhci" sounds better.

Regards..
Prafulla . .

Quoted 18 lines
drivers/mmc/sdhci-mv.c new file +16 -1
+int mv_sdh_init(u32 regbase, u32 max_clk, u32 min_clk, u32 quirks)
+{
+	struct sdhci_host *host = NULL;
+	host = (struct sdhci_host *)malloc(sizeof(struct sdhci_host));
+	if (!host) {
+		printf("sdh_host malloc fail!\n");
+		return 1;
+	}
+
+	host->name = MVSDH_NAME;
+	host->ioaddr = (void *)regbase;
+	host->quirks = quirks;
+	host->version = sdhci_readw(host, SDHCI_HOST_VERSION);
+	add_sdhci(host, max_clk, min_clk);
+	return 0;
+}
--

1.7.0.4

#19 lore raw

PATCH v2 2/2 MMC: add marvell sdhci driver

Hi Prafulla,

Quoted 36 lines · Prafulla Wadaskar
Quoted 9 lines

-----Original Message-----
From: Lei Wen [mailto:leiwen at marvell.com]
Sent: Thursday, June 16, 2011 8:48 PM
To: Andy Fleming; Rob Herring; u-boot at lists.denx.de; Prafulla Wadaskar; Yu Tang; Prabhanjan Sarnaik; Ashish Karkare
Subject: [PATCH 2/2] MMC: add marvell sdhci driver

Quoted 8 lines

This could support both armada100 and pantheon serial in the mainline, while this driver also be tested to support upcoming mg, mmp2 and mmp3 hardware.

---
?drivers/mmc/Makefile ? | ? ?1 + ?drivers/mmc/sdhci-mv.c | ? 21 +++++++++++++++++++++

The file name should be mv_sdhci.c

Quoted 8 lines

?2 files changed, 22 insertions(+), 0 deletions(-) ?create mode 100644 drivers/mmc/sdhci-mv.c

drivers/mmc/Makefile +0 -0
index 50b5117..fd84389 100644
@@ -39,6 +39,7 @@ COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
Quoted 4 lines

?COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
?COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
?COBJS-$(CONFIG_SDHCI) += sdhci.o
+COBJS-$(CONFIG_SDHCI_MV) += sdhci-mv.o

Same here CONFIG_MV_SDHCI

Quoted 1 line
Quoted 16 lines

drivers/mmc/sdhci-mv.c line 5

Quoted 13 lines

?COBJS ? ? ? ?:= $(COBJS-y)
?SRCS := $(COBJS:.o=.c)

drivers/mmc/sdhci-mv.c new file +5 -0
new file mode 100644
index 0000000..97d79ee
@@ -0,0 +1,21 @@
+#include <common.h>
+#include <malloc.h>
+#include <sdhci.h>
+
+static char *MVSDH_NAME = "sdh-mv";

"mv_sdhci" sounds better.

I have no concern with the name change, this would be included in next patch set.

Best regards,
Lei

author
#20 lore raw

PATCH v2 2/2 MMC: add marvell sdhci driver

In reply to Lei Wen

This could support both armada100 and pantheon serial in the mainline, while this driver also be tested to support upcoming mg, mmp2 and mmp3 hardware.

---
V1:
add marvell sdhci driver implementation

V2:
rename the previous file name from sdhci-mv to mv_sdhci

 drivers/mmc/Makefile   |    1 +
 drivers/mmc/mv_sdhci.c |   21 +++++++++++++++++++++
 2 files changed, 22 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mmc/mv_sdhci.c
drivers/mmc/Makefile +1 -0
index 50b5117..bf6534a 100644
@@ -39,6 +39,7 @@ COBJS-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
 COBJS-$(CONFIG_PXA_MMC) += pxa_mmc.o
 COBJS-$(CONFIG_S5P_MMC) += s5p_mmc.o
 COBJS-$(CONFIG_SDHCI) += sdhci.o
+COBJS-$(CONFIG_MV_SDHCI) += mv_sdhci.o
 
 COBJS	:= $(COBJS-y)
 SRCS	:= $(COBJS:.o=.c)
drivers/mmc/mv_sdhci.c new file +21 -0
new file mode 100644
index 0000000..af0d486
@@ -0,0 +1,21 @@
+#include <common.h>
+#include <malloc.h>
+#include <sdhci.h>
+
+static char *MVSDH_NAME = "mv_sdh";
+int mv_sdh_init(u32 regbase, u32 max_clk, u32 min_clk, u32 quirks)
+{
+	struct sdhci_host *host = NULL;
+	host = (struct sdhci_host *)malloc(sizeof(struct sdhci_host));
+	if (!host) {
+		printf("sdh_host malloc fail!\n");
+		return 1;
+	}
+
+	host->name = MVSDH_NAME;
+	host->ioaddr = (void *)regbase;
+	host->quirks = quirks;
+	host->version = sdhci_readw(host, SDHCI_HOST_VERSION);
+	add_sdhci(host, max_clk, min_clk);
+	return 0;
+}
Signature · 1 line
1.7.0.4