]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "mmc: meson-gx: include tx phase in the tuning process"
authorJerome Brunet <jbrunet@baylibre.com>
Mon, 12 Feb 2018 13:13:59 +0000 (14:13 +0100)
committerUlf Hansson <ulf.hansson@linaro.org>
Wed, 14 Feb 2018 10:30:03 +0000 (11:30 +0100)
commitc761e867a4c53d8d09cda0304acf97477922b9ef
tree12fd6c6512ae1e6778f7614136ce5b6da9eb25a7
parentbcac507057f6668c45308343ba41f4c91a086136
Revert "mmc: meson-gx: include tx phase in the tuning process"

This reverts commit 130c19ef8c3f48d87cc1790e3225076bf5e2f44d.

This commit was initially intended to fix problems with hs200 and hs400
on some boards, mainly the odroid-c2. The OC2 (Rev 0.2) I have performs
well in this modes, so I could not confirm these issues.

We've had several reports about the issues being still present on (some)
OC2, so apparently, this change does not do what it was supposed to do.
Maybe the eMMC signal quality is on the edge on the board. This may
explain the variability we see in term of stability, but this is just a
guess. Lowering the max_frequency to 100Mhz seems to do trick for those
affected by the issue

Worse, the commit created new issues (CRC errors and hangs) on other
boards, such as the kvim 1 and 2, the p200 or the libretech-cc.

According to amlogic, the Tx phase should not be tuned and left in its
default configuration, so it is best to just revert the commit.

Fixes: 130c19ef8c3f ("mmc: meson-gx: include tx phase in the tuning process")
Cc: <stable@vger.kernel.org> # 4.14+
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/meson-gx-mmc.c