From: Adrian Hunter <adrian.hunter@intel.com>
To: Ulf Hansson <ulf.hansson@linaro.org>, Chris Ball <chris@printf.net>
Cc: linux-mmc <linux-mmc@vger.kernel.org>,
Aaron Lu <aaron.lu@intel.com>, Philip Rakity <prakity@nvidia.com>,
Girish K S <girish.shivananjappa@linaro.org>,
Al Cooper <alcooperx@gmail.com>,
Arend van Spriel <arend@broadcom.com>
Subject: [PATCH V3 04/15] mmc: core: Check re-tuning before retrying
Date: Tue, 10 Mar 2015 14:42:59 +0200 [thread overview]
Message-ID: <1425991390-4983-5-git-send-email-adrian.hunter@intel.com> (raw)
In-Reply-To: <1425991390-4983-1-git-send-email-adrian.hunter@intel.com>
Possibly a command is failing because re-tuning is needed.
Use mmc_retune_recheck() to check re-tuning. At that point
re-tuning is held, at least by the request, so
mmc_retune_recheck() flags host->retune_now if the hold
count is 1.
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
---
drivers/mmc/core/core.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index 12bddc9..01b9edf 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -419,6 +419,7 @@ static int mmc_wait_for_data_req_done(struct mmc_host *host,
host->areq);
break; /* return err */
} else {
+ mmc_retune_recheck(host);
pr_info("%s: req failed (CMD%u): %d, retrying...\n",
mmc_hostname(host),
cmd->opcode, cmd->error);
@@ -468,6 +469,8 @@ static void mmc_wait_for_req_done(struct mmc_host *host,
mmc_card_removed(host->card))
break;
+ mmc_retune_recheck(host);
+
pr_debug("%s: req failed (CMD%u): %d, retrying...\n",
mmc_hostname(host), cmd->opcode, cmd->error);
cmd->retries--;
--
1.9.1
next prev parent reply other threads:[~2015-03-10 12:45 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-10 12:42 [PATCH V3 00/15] mmc: host: Add facility to support re-tuning Adrian Hunter
2015-03-10 12:42 ` [PATCH V3 01/15] " Adrian Hunter
2015-03-10 12:42 ` [PATCH V3 02/15] mmc: core: Disable re-tuning when card is no longer initialized Adrian Hunter
2015-03-10 12:42 ` [PATCH V3 03/15] mmc: core: Add support for re-tuning before each request Adrian Hunter
2015-03-10 12:42 ` Adrian Hunter [this message]
2015-03-10 12:43 ` [PATCH V3 05/15] mmc: core: Hold re-tuning during switch commands Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 06/15] mmc: core: Hold re-tuning during erase commands Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 07/15] mmc: core: Hold re-tuning while bkops ongoing Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 08/15] mmc: mmc: Comment that callers need to hold re-tuning if the card is put to sleep Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 09/15] mmc: core: Separate out the mmc_switch status check so it can be re-used Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 10/15] mmc: core: Add support for HS400 re-tuning Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 11/15] mmc: sdhci: Change to new way of doing re-tuning Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 12/15] mmc: sdhci: Flag re-tuning is needed on CRC or End-Bit errors Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 13/15] mmc: block: Check re-tuning in the recovery path Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 14/15] mmc: block: Retry errored data requests when re-tuning is needed Adrian Hunter
2015-03-10 12:43 ` [PATCH V3 15/15] mmc: core: Don't print reset warning if reset is not supported Adrian Hunter
2015-03-16 14:00 ` [PATCH V3 00/15] mmc: host: Add facility to support re-tuning Adrian Hunter
2015-03-23 8:06 ` Adrian Hunter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1425991390-4983-5-git-send-email-adrian.hunter@intel.com \
--to=adrian.hunter@intel.com \
--cc=aaron.lu@intel.com \
--cc=alcooperx@gmail.com \
--cc=arend@broadcom.com \
--cc=chris@printf.net \
--cc=girish.shivananjappa@linaro.org \
--cc=linux-mmc@vger.kernel.org \
--cc=prakity@nvidia.com \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.