mmc: sdhci: Add quirk to indicate MMC_RSP_136 has CRC
authorKishon Vijay Abraham I <kishon@ti.com>
Mon, 21 Aug 2017 07:41:29 +0000 (13:11 +0530)
committerUlf Hansson <ulf.hansson@linaro.org>
Wed, 30 Aug 2017 13:03:43 +0000 (15:03 +0200)
TI's implementation of sdhci controller used in DRA7 SoC's has
CRC in responses with length 136 bits. Add quirk to indicate
the controller has CRC in MMC_RSP_136. If this quirk is
set sdhci library shouldn't shift the response present in
SDHCI_RESPONSE register.

Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
Acked-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 1cea35d8bc79eeaf3c1fa00bce134047db216424..0d5fcca18c9ec14344d63444a53a39e6f53dde03 100644 (file)
@@ -1182,6 +1182,9 @@ static void sdhci_read_rsp_136(struct sdhci_host *host, struct mmc_command *cmd)
                cmd->resp[i] = sdhci_readl(host, reg);
        }
 
+       if (host->quirks2 & SDHCI_QUIRK2_RSP_136_HAS_CRC)
+               return;
+
        /* CRC is stripped so we need to do some shifting */
        for (i = 0; i < 4; i++) {
                cmd->resp[i] <<= 8;
index 399edc681623186ff8ec8b3c32127dc366456e45..54bc444c317f18406488df67bc89d6db4c3a2b66 100644 (file)
@@ -435,6 +435,8 @@ struct sdhci_host {
 #define SDHCI_QUIRK2_ACMD23_BROKEN                     (1<<14)
 /* Broken Clock divider zero in controller */
 #define SDHCI_QUIRK2_CLOCK_DIV_ZERO_BROKEN             (1<<15)
+/* Controller has CRC in 136 bit Command Response */
+#define SDHCI_QUIRK2_RSP_136_HAS_CRC                   (1<<16)
 
        int irq;                /* Device IRQ */
        void __iomem *ioaddr;   /* Mapped address */