mmc: core: Support the optional init_card() callback for MMC and SD
authorDoug Anderson <dianders@chromium.org>
Tue, 2 Dec 2014 23:42:45 +0000 (15:42 -0800)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 19 Jan 2015 08:56:04 +0000 (09:56 +0100)
In (3fcb027 ARM: MXC: mxcmmc: work around a bug in the SDHC busy line
handling) the optional init_card() callback was added.  According to
the original change it was "for now only called from
mmc_sdio_init_card()".

This callback really ought to be called from the SD and MMC init
functions as well.  One current user of this callback
(mxcmci_init_card) will not work as expected if you insert an SDIO
card, then eject it and put a normal SD card in.  Specifically the
normal SD card will not get to run with 4-bit data.

I'd like to use the init_card() callback to handle a similar quirk on
dw_mmc when using SDIO Interrupts (the "low power" feature of the card
needs to be disabled), so that will add a second user of the function.

Signed-off-by: Doug Anderson <dianders@chromium.org>
Reviewed-by: Grant Grundler <grundler@chromium.org>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/core/mmc.c
drivers/mmc/core/sd.c

index 7466ce098e60a086e4f60c5111c19c1b85cd6d60..d854bff037a248e2c26e1f5f0d1993e4ac11e767 100644 (file)
@@ -1296,6 +1296,12 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
                memcpy(card->raw_cid, cid, sizeof(card->raw_cid));
        }
 
+       /*
+        * Call the optional HC's init_card function to handle quirks.
+        */
+       if (host->ops->init_card)
+               host->ops->init_card(host, card);
+
        /*
         * For native busses:  set card RCA and quit open drain mode.
         */
index d90a6de7901d7514550b3c9ed64d383e528244ad..29fccdcacd6878fce71fad022da78d3013a5686e 100644 (file)
@@ -932,6 +932,12 @@ static int mmc_sd_init_card(struct mmc_host *host, u32 ocr,
                memcpy(card->raw_cid, cid, sizeof(card->raw_cid));
        }
 
+       /*
+        * Call the optional HC's init_card function to handle quirks.
+        */
+       if (host->ops->init_card)
+               host->ops->init_card(host, card);
+
        /*
         * For native busses:  get card RCA and quit open drain mode.
         */
@@ -1271,4 +1277,3 @@ err:
 
        return err;
 }
-