mmc: sdhci: Add sdhci_read_caps()
authorAdrian Hunter <adrian.hunter@intel.com>
Wed, 29 Jun 2016 13:24:18 +0000 (16:24 +0300)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 25 Jul 2016 08:34:35 +0000 (10:34 +0200)
Add sdhci_read_caps() and __sdhci_read_caps() to make it easier for drivers
to fix the version and capabilities registers.

Pedantically, the SDHCI specification states that the capabilities
registers are valid when the host controller resets the Software Reset For
All bit. That requirement has always been satisfied by performing a reset
at the start of initialization, and consequently that is now part of the
new functions.

Although the SDHCI_QUIRK_MISSING_CAPS quirk has not yet been removed,
drivers that want to provide their own caps can now use these functions
instead of that quirk.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/sdhci.c
drivers/mmc/host/sdhci.h

index 185dbf27689ed866aa84e4d49b04a303e93cb35d..44cdb6d035f215d9ebafe49eaaa5c115c2ea5318 100644 (file)
@@ -2841,6 +2841,38 @@ static int sdhci_set_dma_mask(struct sdhci_host *host)
        return ret;
 }
 
+void __sdhci_read_caps(struct sdhci_host *host, u16 *ver, u32 *caps, u32 *caps1)
+{
+       u16 v;
+
+       if (host->read_caps)
+               return;
+
+       host->read_caps = true;
+
+       if (debug_quirks)
+               host->quirks = debug_quirks;
+
+       if (debug_quirks2)
+               host->quirks2 = debug_quirks2;
+
+       sdhci_do_reset(host, SDHCI_RESET_ALL);
+
+       v = ver ? *ver : sdhci_readw(host, SDHCI_HOST_VERSION);
+       host->version = (v & SDHCI_SPEC_VER_MASK) >> SDHCI_SPEC_VER_SHIFT;
+
+       if (host->quirks & SDHCI_QUIRK_MISSING_CAPS)
+               return;
+
+       host->caps = caps ? *caps : sdhci_readl(host, SDHCI_CAPABILITIES);
+
+       if (host->version < SDHCI_SPEC_300)
+               return;
+
+       host->caps1 = caps1 ? *caps1 : sdhci_readl(host, SDHCI_CAPABILITIES_1);
+}
+EXPORT_SYMBOL_GPL(__sdhci_read_caps);
+
 int sdhci_setup_host(struct sdhci_host *host)
 {
        struct mmc_host *mmc;
@@ -2856,29 +2888,15 @@ int sdhci_setup_host(struct sdhci_host *host)
 
        mmc = host->mmc;
 
-       if (debug_quirks)
-               host->quirks = debug_quirks;
-       if (debug_quirks2)
-               host->quirks2 = debug_quirks2;
+       sdhci_read_caps(host);
 
        override_timeout_clk = host->timeout_clk;
 
-       sdhci_do_reset(host, SDHCI_RESET_ALL);
-
-       host->version = sdhci_readw(host, SDHCI_HOST_VERSION);
-       host->version = (host->version & SDHCI_SPEC_VER_MASK)
-                               >> SDHCI_SPEC_VER_SHIFT;
        if (host->version > SDHCI_SPEC_300) {
                pr_err("%s: Unknown controller version (%d). You may experience problems.\n",
                       mmc_hostname(mmc), host->version);
        }
 
-       if (!(host->quirks & SDHCI_QUIRK_MISSING_CAPS)) {
-               host->caps = sdhci_readl(host, SDHCI_CAPABILITIES);
-               if (host->version >= SDHCI_SPEC_300)
-                       host->caps1 = sdhci_readl(host, SDHCI_CAPABILITIES_1);
-       }
-
        if (host->quirks & SDHCI_QUIRK_FORCE_DMA)
                host->flags |= SDHCI_USE_SDMA;
        else if (!(host->caps & SDHCI_CAN_DO_SDMA))
index 8696c9365ef2702a118fa57fcdf42e9188e269d2..e332e4a4082353d850b7fc0c789b78783a0b3bab 100644 (file)
@@ -492,6 +492,7 @@ struct sdhci_host {
 
        u32 caps;               /* CAPABILITY_0 */
        u32 caps1;              /* CAPABILITY_1 */
+       bool read_caps;         /* Capability flags have been read */
 
        unsigned int            ocr_avail_sdio; /* OCR bit masks */
        unsigned int            ocr_avail_sd;
@@ -648,6 +649,8 @@ static inline void *sdhci_priv(struct sdhci_host *host)
 }
 
 extern void sdhci_card_detect(struct sdhci_host *host);
+extern void __sdhci_read_caps(struct sdhci_host *host, u16 *ver, u32 *caps,
+                             u32 *caps1);
 extern int sdhci_setup_host(struct sdhci_host *host);
 extern int __sdhci_add_host(struct sdhci_host *host);
 extern int sdhci_add_host(struct sdhci_host *host);
@@ -655,6 +658,11 @@ extern void sdhci_remove_host(struct sdhci_host *host, int dead);
 extern void sdhci_send_command(struct sdhci_host *host,
                                struct mmc_command *cmd);
 
+static inline void sdhci_read_caps(struct sdhci_host *host)
+{
+       __sdhci_read_caps(host, NULL, NULL, NULL);
+}
+
 static inline bool sdhci_sdio_irq_enabled(struct sdhci_host *host)
 {
        return !!(host->flags & SDHCI_SDIO_IRQ_ENABLED);