Commit 437590a1 authored by Ulf Hansson's avatar Ulf Hansson

mmc: core: Retry instead of ignore at CRC errors when polling for busy

After a CMD6 command has been sent, the __mmc_switch() function might be
advised to poll the card for busy by using CMD13 and also by ignoring CRC
errors.

In the case of ignoring CRC errors, the mmc core tells the mmc host to also
ignore these errors via masking the MMC_RSP_CRC response flag. This seems
wrong, as it leads to that the mmc host could propagate an unreliable
response, instead of a proper error code.

What we really want, is not to ignore CRC errors but instead retry the
polling attempt. So, let's change this by treating a CRC error as the card
is still being busy and thus continue to run the polling loop.
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Tested-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Acked-by: default avatarAdrian Hunter <adrian.hunter@intel.com>
parent 8f743d03
...@@ -503,10 +503,13 @@ static int mmc_poll_for_busy(struct mmc_card *card, unsigned int timeout_ms, ...@@ -503,10 +503,13 @@ static int mmc_poll_for_busy(struct mmc_card *card, unsigned int timeout_ms,
if (host->ops->card_busy) { if (host->ops->card_busy) {
busy = host->ops->card_busy(host); busy = host->ops->card_busy(host);
} else { } else {
err = __mmc_send_status(card, &status, ignore_crc); err = mmc_send_status(card, &status);
if (err) if (ignore_crc && err == -EILSEQ)
busy = true;
else if (err)
return err; return err;
busy = R1_CURRENT_STATE(status) == R1_STATE_PRG; else
busy = R1_CURRENT_STATE(status) == R1_STATE_PRG;
} }
/* Timeout if the device still remains busy. */ /* Timeout if the device still remains busy. */
......
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