mmc: meson-gx: align default phase on soc vendor tree
authorJerome Brunet <jbrunet@baylibre.com>
Thu, 6 Dec 2018 15:18:27 +0000 (16:18 +0100)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 17 Dec 2018 07:26:24 +0000 (08:26 +0100)
Align the default Core and Tx phase with the SoC vendor tree.
Even if the Tx phase is different from what the documentation
recommends, it seems to provide better results.

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/meson-gx-mmc.c

index 5cc31e4..837bed0 100644 (file)
@@ -634,14 +634,8 @@ static int meson_mmc_clk_init(struct meson_host *host)
        if (ret)
                return ret;
 
-       /*
-        * Set phases : These values are mostly the datasheet recommended ones
-        * except for the Tx phase. Datasheet recommends 180 but some cards
-        * fail at initialisation with it. 270 works just fine, it fixes these
-        * initialisation issues and enable eMMC DDR52 mode.
-        */
        clk_set_phase(host->mmc_clk, 180);
-       clk_set_phase(host->tx_clk, 270);
+       clk_set_phase(host->tx_clk, 0);
        clk_set_phase(host->rx_clk, 0);
 
        return clk_prepare_enable(host->mmc_clk);