Revert "UPSTREAM: mmc: mmc: Use 500ms as the default generic CMD6 timeout"
This reverts commit 0b3bd19f180588e4a8338eb0669a1e50ca3b98dc.
Reason for revert:
It triggers a regression where Hynix eMMC needs to be power cycled.
BUG=chromium:997389,b:138033787
TEST=Compile ToT Bob without it, able to install R78.
Original change's description:
> UPSTREAM: mmc: mmc: Use 500ms as the default generic CMD6 timeout
>
> In the eMMC 4.51 version of the spec, an EXT_CSD field called
> GENERIC_CMD6_TIME[248] was added. This allows cards to specify the maximum
> time it may need to move out from its busy state, when a CMD6 command has
> been sent.
>
> In cases when the card is compliant to versions < 4.51 of the eMMC spec,
> obviously the core needs to use a fall-back value for this timeout, which
> currently is set to 10 minutes. This value is completely in the wrong range
> and importantly in some cases it causes a card initialization to take more
> than 10 minute to complete.
>
> Earlier this scenario was avoided as the mmc core used CMD13 to poll the
> card, to find out when it stopped signaling busy. Commit 08573eaf1a70
> ("mmc: mmc: do not use CMD13 to get status after speed mode switch")
> changed this behavior.
>
> Instead of reverting that commit, which would cause other issues, let's
> instead start by picking a simple solution for the problem, by using a
> 500ms default generic CMD6 timeout.
>
> The reason for using exactly 500ms, comes from observations that shows it's
> quite common for cards to specify 250ms. 500ms is two times that value so
> likely it should be enough for most cards.
>
> Cc: <stable@vger.kernel.org> # v4.8+
> Fixes: 08573eaf1a70 ("mmc: mmc: do not use CMD13 to get status after speed
> mode switch")
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> Tested-by: Stephen Boyd <sboyd@codeaurora.org>
> Tested-by: Linus Walleij <linus.walleij@linaro.org>
> (cherry picked from commit fe1b5700c70faac6e027982d59667bc6020de5a8)
>
> BUG=b:138033787
> TEST=compile
>
>
> Change-Id: I4078e8144d461cab5e3d04d3c7a22f0557772a79
> Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
> Reviewed-on: https://chromium-review.googlesource.com/1727139
> Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
> Reviewed-by: Douglas Anderson <dianders@chromium.org>
> Reviewed-by: Sean Paul <seanpaul@chromium.org>
> (cherry picked from commit 7afee62b49042be91e379c7dd7060a8a08020cf1)
>
> Signed-off-by: Tim Chen <tim-chen@quanta.corp-partner.google.com>
> Cq-Depend: chromium:1716943
> Change-Id: I4078e8144d461cab5e3d04d3c7a22f0557772a79
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1753486
> Reviewed-by: Sean Paul <seanpaul@chromium.org>
> Reviewed-by: Vincent Wang <vwang@chromium.org>
> Reviewed-by: Douglas Anderson <dianders@chromium.org>
Bug: b:138033787
Change-Id: I5768acc6ba5b9d1704f6714ddfdddcd996c0bf33
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1770857
Tested-by: Gwendal Grignou <gwendal@chromium.org>
Commit-Queue: Gwendal Grignou <gwendal@chromium.org>
Reviewed-by: Gwendal Grignou <gwendal@chromium.org>
1 file changed
