Commit 667123f6 authored by Benoît Thébaudeau's avatar Benoît Thébaudeau Committed by Ulf Hansson

mmc: sdhci-esdhc-imx: Remove the ENGcm07207 workaround

The SDHCI_QUIRK_NO_MULTIBLOCK quirk was used as a workaround for the
ENGcm07207 erratum. However, it caused excruciatingly slow SD transfers
(300 kB/s on average), and this erratum actually does not imply that
multiple-block transfers are not supported, so this was overkill.

The suggested workaround for this erratum is to set SYSCTL.RSTA, but the
simple DAT line software reset (which resets the DMA circuit among
others) triggered by sdhci_finish_data() in case of errors seems to be
sufficient. Indeed, generating errors in a controlled manner on i.MX25
using the FEVT register right in the middle of read data transfers
without this quirk shows that nothing is written to the buffer by the
eSDHC past CMD12, and no extra Auto CMD12 is sent with AC12EN set, so
the data transfers on AHB are properly aborted. For write data
transfers, neither extra data nor extra Auto CMD12 is sent, as expected.
Moreover, after intensive stress tests on i.MX25, removing
SDHCI_QUIRK_NO_MULTIBLOCK seems to be safe.

SDHCI_QUIRK_BROKEN_ADMA has nothing to do with ENGcm07207, so set
ESDHC_FLAG_ERR004536 for the devices that had ESDHC_FLAG_ENGCM07207 set
in order to continue getting SDHCI_QUIRK_BROKEN_ADMA.
Signed-off-by: default avatarBenoît Thébaudeau <benoit@wsystem.com>
Acked-by: default avatarAdrian Hunter <adrian.hunter@intel.com>
Reviewed-by: default avatarFabio Estevam <fabio.estevam@nxp.com>
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
parent 5143c953
...@@ -114,11 +114,6 @@ ...@@ -114,11 +114,6 @@
* exception. Bit1 of Vendor Spec register is used to fix it. * exception. Bit1 of Vendor Spec register is used to fix it.
*/ */
#define ESDHC_FLAG_MULTIBLK_NO_INT BIT(1) #define ESDHC_FLAG_MULTIBLK_NO_INT BIT(1)
/*
* The flag enables the workaround for ESDHC erratum ENGcm07207 which
* affects i.MX25 and i.MX35.
*/
#define ESDHC_FLAG_ENGCM07207 BIT(2)
/* /*
* The flag tells that the ESDHC controller is an USDHC block that is * The flag tells that the ESDHC controller is an USDHC block that is
* integrated on the i.MX6 series. * integrated on the i.MX6 series.
...@@ -134,6 +129,8 @@ ...@@ -134,6 +129,8 @@
* The IP has erratum ERR004536 * The IP has erratum ERR004536
* uSDHC: ADMA Length Mismatch Error occurs if the AHB read access is slow, * uSDHC: ADMA Length Mismatch Error occurs if the AHB read access is slow,
* when reading data from the card * when reading data from the card
* This flag is also set for i.MX25 and i.MX35 in order to get
* SDHCI_QUIRK_BROKEN_ADMA, but for different reasons (ADMA capability bits).
*/ */
#define ESDHC_FLAG_ERR004536 BIT(7) #define ESDHC_FLAG_ERR004536 BIT(7)
/* The IP supports HS200 mode */ /* The IP supports HS200 mode */
...@@ -149,11 +146,11 @@ struct esdhc_soc_data { ...@@ -149,11 +146,11 @@ struct esdhc_soc_data {
}; };
static struct esdhc_soc_data esdhc_imx25_data = { static struct esdhc_soc_data esdhc_imx25_data = {
.flags = ESDHC_FLAG_ENGCM07207, .flags = ESDHC_FLAG_ERR004536,
}; };
static struct esdhc_soc_data esdhc_imx35_data = { static struct esdhc_soc_data esdhc_imx35_data = {
.flags = ESDHC_FLAG_ENGCM07207, .flags = ESDHC_FLAG_ERR004536,
}; };
static struct esdhc_soc_data esdhc_imx51_data = { static struct esdhc_soc_data esdhc_imx51_data = {
...@@ -1285,11 +1282,6 @@ static int sdhci_esdhc_imx_probe(struct platform_device *pdev) ...@@ -1285,11 +1282,6 @@ static int sdhci_esdhc_imx_probe(struct platform_device *pdev)
if (IS_ERR(imx_data->pins_default)) if (IS_ERR(imx_data->pins_default))
dev_warn(mmc_dev(host->mmc), "could not get default state\n"); dev_warn(mmc_dev(host->mmc), "could not get default state\n");
if (imx_data->socdata->flags & ESDHC_FLAG_ENGCM07207)
/* Fix erratum ENGcm07207 present on i.MX25 and i.MX35 */
host->quirks |= SDHCI_QUIRK_NO_MULTIBLOCK
| SDHCI_QUIRK_BROKEN_ADMA;
if (esdhc_is_usdhc(imx_data)) { if (esdhc_is_usdhc(imx_data)) {
host->quirks2 |= SDHCI_QUIRK2_PRESET_VALUE_BROKEN; host->quirks2 |= SDHCI_QUIRK2_PRESET_VALUE_BROKEN;
host->mmc->caps |= MMC_CAP_1_8V_DDR; host->mmc->caps |= MMC_CAP_1_8V_DDR;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment