linux-mmc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sowjanya Komatineni <skomatineni@nvidia.com>
To: adrian.hunter@intel.com, ulf.hansson@linaro.org,
	baolin.wang@linaro.org, kstewart@linuxfoundation.org,
	tglx@linutronix.de, bradleybolen@gmail.com,
	gregkh@linuxfoundation.org, thierry.reding@gmail.com,
	jonathanh@nvidia.com
Cc: skomatineni@nvidia.com, anrao@nvidia.com,
	linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-mmc@vger.kernel.org
Subject: [PATCH v1 1/3] mmc: core: Add capability for hosts supporting long HW busy wait
Date: Mon,  9 Mar 2020 17:13:23 -0700	[thread overview]
Message-ID: <1583799205-8442-1-git-send-email-skomatineni@nvidia.com> (raw)

Some mmc operation like SLEEP_AWAKE and cache flush through SWITCH
command takes longer time and for host controllers supporting HW
busy detection, core driver converts R1B type to R1 type response
for these commands if operation time is more than HW busy timeout
and uses SW poll for busy state.

This patch adds a host capability MMC_CAP2_LONG_WAIT_HW_BUSY for
hosts supporting long or infinite busy wait of HW busy detection
for these long mmc operations and prevents core driver from
converting R1B to R1 response type.

Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com>
---
 drivers/mmc/core/mmc.c     | 3 ++-
 drivers/mmc/core/mmc_ops.c | 3 ++-
 include/linux/mmc/host.h   | 1 +
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
index c2abd41..5aea3a6 100644
--- a/drivers/mmc/core/mmc.c
+++ b/drivers/mmc/core/mmc.c
@@ -1916,7 +1916,8 @@ static int mmc_sleep(struct mmc_host *host)
 	 * from doing hw busy detection, which is done by converting to a R1
 	 * response instead of a R1B.
 	 */
-	if (host->max_busy_timeout && (timeout_ms > host->max_busy_timeout)) {
+	if (host->max_busy_timeout && (timeout_ms > host->max_busy_timeout) &&
+	    !(host->caps2 & MMC_CAP2_LONG_WAIT_HW_BUSY)) {
 		cmd.flags = MMC_RSP_R1 | MMC_CMD_AC;
 	} else {
 		cmd.flags = MMC_RSP_R1B | MMC_CMD_AC;
diff --git a/drivers/mmc/core/mmc_ops.c b/drivers/mmc/core/mmc_ops.c
index aa0cab1..251b4c7 100644
--- a/drivers/mmc/core/mmc_ops.c
+++ b/drivers/mmc/core/mmc_ops.c
@@ -579,7 +579,8 @@ int __mmc_switch(struct mmc_card *card, u8 set, u8 index, u8 value,
 	 * the host to avoid HW busy detection, by converting to a R1 response
 	 * instead of a R1B.
 	 */
-	if (host->max_busy_timeout && (timeout_ms > host->max_busy_timeout))
+	if (host->max_busy_timeout && (timeout_ms > host->max_busy_timeout) &&
+	    !(host->caps2 & MMC_CAP2_LONG_WAIT_HW_BUSY))
 		use_r1b_resp = false;
 
 	cmd.opcode = MMC_SWITCH;
diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
index 6aaa15e..1cda4dd 100644
--- a/include/linux/mmc/host.h
+++ b/include/linux/mmc/host.h
@@ -371,6 +371,7 @@ struct mmc_host {
 #define MMC_CAP2_CQE_DCMD	(1 << 24)	/* CQE can issue a direct command */
 #define MMC_CAP2_AVOID_3_3V	(1 << 25)	/* Host must negotiate down from 3.3V */
 #define MMC_CAP2_MERGE_CAPABLE	(1 << 26)	/* Host can merge a segment over the segment size */
+#define MMC_CAP2_LONG_WAIT_HW_BUSY (1 << 27)	/* Host can support long busy timeout, do not convert R1B to R1 response */
 
 	int			fixed_drv_type;	/* fixed driver type for non-removable media */
 
-- 
2.7.4


             reply	other threads:[~2020-03-13  1:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-10  0:13 Sowjanya Komatineni [this message]
2020-03-10  0:13 ` [PATCH v1 2/3] mmc: tegra: Implement HW busy wait timeout based on command busy time Sowjanya Komatineni
2020-03-17  9:51   ` Adrian Hunter
2020-03-10  0:13 ` [PATCH v1 3/3] mmc: tegra: Enable host capability MMC_CAP2_LONG_WAIT_HW_BUSY Sowjanya Komatineni
2020-03-17  9:51   ` 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=1583799205-8442-1-git-send-email-skomatineni@nvidia.com \
    --to=skomatineni@nvidia.com \
    --cc=adrian.hunter@intel.com \
    --cc=anrao@nvidia.com \
    --cc=baolin.wang@linaro.org \
    --cc=bradleybolen@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=jonathanh@nvidia.com \
    --cc=kstewart@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=tglx@linutronix.de \
    --cc=thierry.reding@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).