Commit a66664cc authored by Marcin Wojtas's avatar Marcin Wojtas Committed by Luis Henriques

mmc: sdhci-pxav3: Fix Armada 38x controller's caps according to erratum ERR-7878951

commit a39128bc upstream.

According to erratum 'ERR-7878951' Armada 38x SDHCI controller has
different capabilities than the ones shown in its registers:

- it doesn't support the voltage switching: it can work either with
  3.3V or 1.8V supply
- it doesn't support the SDR104 mode
- SDR50 mode doesn't need tuning

The SDHCI_QUIRK_MISSING_CAPS quirk is used for updating the
capabilities accordingly.

[gregory.clement@free-electrons.com: port from 3.10]

Fixes: 5491ce3f ("mmc: sdhci-pxav3: add support for the Armada 38x SDHCI controller")
Signed-off-by: default avatarGregory CLEMENT <gregory.clement@free-electrons.com>
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
parent c930c075
...@@ -112,8 +112,11 @@ static int mv_conf_mbus_windows(struct platform_device *pdev, ...@@ -112,8 +112,11 @@ static int mv_conf_mbus_windows(struct platform_device *pdev,
return 0; return 0;
} }
static int armada_38x_quirks(struct sdhci_host *host) static int armada_38x_quirks(struct platform_device *pdev,
struct sdhci_host *host)
{ {
struct device_node *np = pdev->dev.of_node;
host->quirks |= SDHCI_QUIRK_MISSING_CAPS; host->quirks |= SDHCI_QUIRK_MISSING_CAPS;
/* /*
* According to erratum 'FE-2946959' both SDR50 and DDR50 * According to erratum 'FE-2946959' both SDR50 and DDR50
...@@ -123,6 +126,21 @@ static int armada_38x_quirks(struct sdhci_host *host) ...@@ -123,6 +126,21 @@ static int armada_38x_quirks(struct sdhci_host *host)
*/ */
host->caps1 = sdhci_readl(host, SDHCI_CAPABILITIES_1); host->caps1 = sdhci_readl(host, SDHCI_CAPABILITIES_1);
host->caps1 &= ~(SDHCI_SUPPORT_SDR50 | SDHCI_SUPPORT_DDR50); host->caps1 &= ~(SDHCI_SUPPORT_SDR50 | SDHCI_SUPPORT_DDR50);
/*
* According to erratum 'ERR-7878951' Armada 38x SDHCI
* controller has different capabilities than the ones shown
* in its registers
*/
host->caps = sdhci_readl(host, SDHCI_CAPABILITIES);
if (of_property_read_bool(np, "no-1-8-v")) {
host->caps &= ~SDHCI_CAN_VDD_180;
host->mmc->caps &= ~MMC_CAP_1_8V_DDR;
} else {
host->caps &= ~SDHCI_CAN_VDD_330;
}
host->caps1 &= ~(SDHCI_SUPPORT_SDR104 | SDHCI_USE_SDR50_TUNING);
return 0; return 0;
} }
...@@ -323,8 +341,11 @@ static int sdhci_pxav3_probe(struct platform_device *pdev) ...@@ -323,8 +341,11 @@ static int sdhci_pxav3_probe(struct platform_device *pdev)
pltfm_host->clk = clk; pltfm_host->clk = clk;
clk_prepare_enable(clk); clk_prepare_enable(clk);
/* enable 1/8V DDR capable */
host->mmc->caps |= MMC_CAP_1_8V_DDR;
if (of_device_is_compatible(np, "marvell,armada-380-sdhci")) { if (of_device_is_compatible(np, "marvell,armada-380-sdhci")) {
ret = armada_38x_quirks(host); ret = armada_38x_quirks(pdev, host);
if (ret < 0) if (ret < 0)
goto err_clk_get; goto err_clk_get;
ret = mv_conf_mbus_windows(pdev, mv_mbus_dram_info()); ret = mv_conf_mbus_windows(pdev, mv_mbus_dram_info());
...@@ -332,9 +353,6 @@ static int sdhci_pxav3_probe(struct platform_device *pdev) ...@@ -332,9 +353,6 @@ static int sdhci_pxav3_probe(struct platform_device *pdev)
goto err_mbus_win; goto err_mbus_win;
} }
/* enable 1/8V DDR capable */
host->mmc->caps |= MMC_CAP_1_8V_DDR;
match = of_match_device(of_match_ptr(sdhci_pxav3_of_match), &pdev->dev); match = of_match_device(of_match_ptr(sdhci_pxav3_of_match), &pdev->dev);
if (match) { if (match) {
ret = mmc_of_parse(host->mmc); ret = mmc_of_parse(host->mmc);
......
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