All of lore.kernel.org
 help / color / mirror / Atom feed
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 03/15] mmc: core: Add support for re-tuning before each request
Date: Tue, 10 Mar 2015 14:42:58 +0200	[thread overview]
Message-ID: <1425991390-4983-4-git-send-email-adrian.hunter@intel.com> (raw)
In-Reply-To: <1425991390-4983-1-git-send-email-adrian.hunter@intel.com>

At the start of each request, re-tune if needed and
then hold off re-tuning again until the request is done.

Note that though there is one function that starts
requests (mmc_start_request) there are two that wait for
the request to be done (mmc_wait_for_req_done and
mmc_wait_for_data_req_done).  Also note that
mmc_wait_for_data_req_done can return even when the
request is not done (which allows the block driver
to prepare a newly arrived request while still
waiting for the previous request).

This patch ensures re-tuning is held for the duration
of a request.  Subsequent patches will also hold
re-tuning at other times when it might cause a
conflict.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
---
 drivers/mmc/core/core.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index aaca0e1..12bddc9 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -192,6 +192,8 @@ static int mmc_start_request(struct mmc_host *host, struct mmc_request *mrq)
 	unsigned int i, sz;
 	struct scatterlist *sg;
 #endif
+	mmc_retune_hold(host);
+
 	if (mmc_card_removed(host->card))
 		return -ENOMEDIUM;
 
@@ -427,12 +429,11 @@ static int mmc_wait_for_data_req_done(struct mmc_host *host,
 			}
 		} else if (context_info->is_new_req) {
 			context_info->is_new_req = false;
-			if (!next_req) {
-				err = MMC_BLK_NEW_REQUEST;
-				break; /* return err */
-			}
+			if (!next_req)
+				return MMC_BLK_NEW_REQUEST;
 		}
 	}
+	mmc_retune_release(host);
 	return err;
 }
 
@@ -473,6 +474,8 @@ static void mmc_wait_for_req_done(struct mmc_host *host,
 		cmd->error = 0;
 		host->ops->request(host, mrq);
 	}
+
+	mmc_retune_release(host);
 }
 
 /**
-- 
1.9.1


  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 ` Adrian Hunter [this message]
2015-03-10 12:42 ` [PATCH V3 04/15] mmc: core: Check re-tuning before retrying Adrian Hunter
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-4-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.