Instead of aborting the probe in case a slot initialization fails, allow
initialization of as many slots as possible. If there are atleast one
instance of slot that is successfully initialized, allow the driver probe
to succeed.

Signed-off-by: Thomas Abraham <thomas.abra...@linaro.org>
Acked-by: Will Newton <will.new...@imgtec.com>
---
 drivers/mmc/host/dw_mmc.c |   23 +++++++++++++----------
 1 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
index a18e73c..227c42e 100644
--- a/drivers/mmc/host/dw_mmc.c
+++ b/drivers/mmc/host/dw_mmc.c
@@ -1946,6 +1946,7 @@ int dw_mci_probe(struct dw_mci *host)
 {
        int width, i, ret = 0;
        u32 fifo_size;
+       int init_slots = 0;
 
        if (!host->pdata || !host->pdata->init) {
                dev_err(host->dev,
@@ -2054,10 +2055,18 @@ int dw_mci_probe(struct dw_mci *host)
        /* We need at least one slot to succeed */
        for (i = 0; i < host->num_slots; i++) {
                ret = dw_mci_init_slot(host, i);
-               if (ret) {
-                       ret = -ENODEV;
-                       goto err_init_slot;
-               }
+               if (ret)
+                       dev_dbg(host->dev, "slot %d init failed\n", i);
+               else
+                       init_slots++;
+       }
+
+       if (init_slots) {
+               dev_info(host->dev, "%d slots initialized\n", init_slots);
+       } else {
+               dev_dbg(host->dev, "attempted to initialize %d slots, "
+                                       "but failed on all\n", host->num_slots);
+               goto err_init_slot;
        }
 
        /*
@@ -2092,12 +2101,6 @@ int dw_mci_probe(struct dw_mci *host)
        return 0;
 
 err_init_slot:
-       /* De-init any initialized slots */
-       while (i > 0) {
-               if (host->slot[i])
-                       dw_mci_cleanup_slot(host->slot[i], i);
-               i--;
-       }
        free_irq(host->irq, host);
 
 err_workqueue:
-- 
1.6.6.rc2

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to