>From 72cc96ae11ec2623d22aef05b7a6880f7ee5847f Mon Sep 17 00:00:00 2001
From: Chuanxiao Dong <[email protected]>
Date: Tue, 9 Nov 2010 16:52:20 +0800
Subject: [PATCH 1/3] mmc: implemented eMMC4.4 hardware reset feature in mmc 
core layer

When core layer detect a data timeout error, core driver need to
first reset eMMC card and then do a reinit for card. After that,
posts the original timeout error to mmc block layer.

1. enable hardware reset feature: to enable hardware reset
feature, mmc driver need to set RST_n_function bit in ext_csd
register. But this bit is onetime programmable, not sure whether
it is suitable for driver to set this bit.

2. The reset part was resvered for host controller to implement.

3. The reinit card part was implemented.

Signed-off-by: Chuanxiao Dong <[email protected]>
---
 drivers/mmc/core/core.c  |   27 +++++++++++++++++++++++++++
 drivers/mmc/core/core.h  |    1 +
 drivers/mmc/core/mmc.c   |   46 ++++++++++++++++++++++++++++++++++++++++++++++
 include/linux/mmc/card.h |    1 +
 include/linux/mmc/host.h |    5 +++++
 include/linux/mmc/mmc.h  |    1 +
 6 files changed, 81 insertions(+), 0 deletions(-)

diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index d464252..fb589b0 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -209,6 +209,8 @@ static void mmc_wait_done(struct mmc_request *mrq)
  */
 void mmc_wait_for_req(struct mmc_host *host, struct mmc_request *mrq)
 {
+       struct mmc_card *card = host->card;
+
        DECLARE_COMPLETION_ONSTACK(complete);
 
        mrq->done_data = &complete;
@@ -221,6 +223,31 @@ void mmc_wait_for_req(struct mmc_host *host, struct 
mmc_request *mrq)
 
        wait_for_completion(&complete);
 
+       /* as MMC4.4 standard says, when some data timeout conditions
+        * occur, HC need to do a hardware reset for eMMC4.4 card.
+        * If the card is eMMC4.4 card && data error is timeout,
+        * do the following things:
+        * 1. let host controller do a specific hardware reset for eMMC
+        *    card (trigger RST_n signal).
+        * 2. after reset done, reinit eMMC4.4 card.
+        * */
+       if (mrq->data && card &&
+               mrq->data->error == -ETIMEDOUT &&
+               card->ext_csd.rst == 1) {
+               int err = 1;
+               if (host->ops->hardware_reset &&
+                               host->bus_ops->reinit) {
+                       err = host->ops->hardware_reset(host);
+                       if (err)
+                               pr_warn("MMC card reset failed, no reinit\n");
+                       else
+                               err = host->bus_ops->reinit(host);
+               }
+
+               if (err)
+                       pr_warn("cannot reset and reinit eMMC4.4 card\n");
+       }
+
        if (host->port_mutex)
                mutex_unlock(host->port_mutex);
 }
diff --git a/drivers/mmc/core/core.h b/drivers/mmc/core/core.h
index 77240cd..232a3d9 100644
--- a/drivers/mmc/core/core.h
+++ b/drivers/mmc/core/core.h
@@ -24,6 +24,7 @@ struct mmc_bus_ops {
        int (*resume)(struct mmc_host *);
        int (*power_save)(struct mmc_host *);
        int (*power_restore)(struct mmc_host *);
+       int (*reinit)(struct mmc_host *);
 };
 
 void mmc_attach_bus(struct mmc_host *host, const struct mmc_bus_ops *ops);
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
index 91c35fd..42c077c 100644
--- a/drivers/mmc/core/mmc.c
+++ b/drivers/mmc/core/mmc.c
@@ -237,6 +237,8 @@ static int mmc_read_ext_csd(struct mmc_card *card)
                }
        }
 
+       card->ext_csd.rst = ext_csd[EXT_CSD_RST];
+
        card->ext_csd.rev = ext_csd[EXT_CSD_REV];
        if (card->ext_csd.rev > 5) {
                printk(KERN_ERR "%s: unrecognised EXT_CSD revision %d\n",
@@ -512,6 +514,31 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
        }
 
        /*
+        * eMMC4.4 version card has HW reset capbility.
+        * Enable this feature here:
+        * RST_N_FUNCTION register is W/R, one time programmable
+        * or readable.
+        * So need to enable this register only once after power on
+        * */
+       if (card->csd.mmca_vsn >= CSD_SPEC_VER_4 &&
+                       card->ext_csd.rev >= 4 &&
+                       card->ext_csd.rst != 1) {
+               err = mmc_switch(card, EXT_CSD_CMD_SET_NORMAL,
+                               EXT_CSD_RST, 1);
+
+               if (err && err != -EBADMSG)
+                       goto free_card;
+
+               if (err) {
+                       printk(KERN_WARNING "%s: switch to rst enable "
+                                  "failed %d\n",
+                                  mmc_hostname(card->host), err);
+                       err = 0;
+               } else
+                       card->ext_csd.rst = 1;
+       }
+
+       /*
         * Activate high speed (if supported)
         */
        if ((card->ext_csd.hs_max_dtr != 0) &&
@@ -727,6 +754,23 @@ static int mmc_awake(struct mmc_host *host)
        return err;
 }
 
+static int mmc_reinit_card(struct mmc_host *host)
+{
+       int err;
+
+       BUG_ON(!host);
+       BUG_ON(!host->card);
+
+       host->ios.clock = host->f_init;
+       mmc_set_clock(host, host->ios.clock);
+
+       err = mmc_init_card(host, host->ocr, host->card);
+       if (err)
+               printk(KERN_ERR "%s: err %d\n", __func__, err);
+
+       return err;
+}
+
 static const struct mmc_bus_ops mmc_ops = {
        .awake = mmc_awake,
        .sleep = mmc_sleep,
@@ -735,6 +779,7 @@ static const struct mmc_bus_ops mmc_ops = {
        .suspend = NULL,
        .resume = NULL,
        .power_restore = mmc_power_restore,
+       .reinit = mmc_reinit_card,
 };
 
 static const struct mmc_bus_ops mmc_ops_unsafe = {
@@ -745,6 +790,7 @@ static const struct mmc_bus_ops mmc_ops_unsafe = {
        .suspend = mmc_suspend,
        .resume = mmc_resume,
        .power_restore = mmc_power_restore,
+       .reinit = mmc_reinit_card,
 };
 
 static void mmc_attach_bus_ops(struct mmc_host *host)
diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
index 8ce0827..4549db4 100644
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -54,6 +54,7 @@ struct mmc_ext_csd {
        unsigned int            sec_trim_mult;  /* Secure trim multiplier  */
        unsigned int            sec_erase_mult; /* Secure erase multiplier */
        unsigned int            trim_timeout;           /* In milliseconds */
+       unsigned char           rst; /* hardware enable bit */
 };
 
 struct sd_scr {
diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
index 01e4886..17a3919 100644
--- a/include/linux/mmc/host.h
+++ b/include/linux/mmc/host.h
@@ -117,6 +117,11 @@ struct mmc_host_ops {
 
        /* optional callback for HC quirks */
        void    (*init_card)(struct mmc_host *host, struct mmc_card *card);
+
+       /* HW reset callback, used for eMMC 4.4 new feature.
+        * when occurs data timeout, HC will need to reset eMMC4.4 card
+        * */
+       int (*hardware_reset)(struct mmc_host *host);
 };
 
 struct mmc_card;
diff --git a/include/linux/mmc/mmc.h b/include/linux/mmc/mmc.h
index 956fbd8..06f6b81 100644
--- a/include/linux/mmc/mmc.h
+++ b/include/linux/mmc/mmc.h
@@ -251,6 +251,7 @@ struct _mmc_csd {
  * EXT_CSD fields
  */
 
+#define EXT_CSD_RST            162             /* R/W */
 #define EXT_CSD_ERASE_GROUP_DEF                175     /* R/W */
 #define EXT_CSD_ERASED_MEM_CONT                181     /* RO */
 #define EXT_CSD_BUS_WIDTH              183     /* R/W */
-- 
1.6.6.1

_______________________________________________
MeeGo-kernel mailing list
[email protected]
http://lists.meego.com/listinfo/meego-kernel

Reply via email to