[PATCH] mmc: sdhci: Start status timeout after command issue

[PATCH] mmc: sdhci: Start status timeout after command issue

[Tanmay Kathpalia]

The status polling timeout in sdhci_send_command() should measure the
time spent waiting for the command interrupt after the command has been
issued.

Do not initialize the timer at function entry, since the command inhibit
wait and setup path can consume time before SDHCI_COMMAND is written.
Start the timer immediately after issuing the command instead.

Signed-off-by: Tanmay Kathpalia <tanmay.kathpalia@gmail.com>
---
 drivers/mmc/sdhci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
index 648dfa4b5ef..08594e10266 100644
--- a/drivers/mmc/sdhci.c
+++ b/drivers/mmc/sdhci.c
@@ -215,7 +215,7 @@ static int sdhci_send_command(struct mmc *mmc, struct mmc_cmd *cmd,
 	u32 mask, flags, mode = 0;
 	unsigned int time = 0;
 	int mmc_dev = mmc_get_blk_desc(mmc)->devnum;
-	ulong start = get_timer(0);
+	ulong start;
 
 	host->start_addr = 0;
 	/* Timeout unit - ms */
-- 
2.43.0

Re: [PATCH] mmc: sdhci: Start status timeout after command issue

[Peng Fan]

On Sat, May 09, 2026 at 11:29:32PM +0530, Tanmay Kathpalia wrote:
>The status polling timeout in sdhci_send_command() should measure the
>time spent waiting for the command interrupt after the command has been
>issued.
>
>Do not initialize the timer at function entry, since the command inhibit
>wait and setup path can consume time before SDHCI_COMMAND is written.
>Start the timer immediately after issuing the command instead.
>
>Signed-off-by: Tanmay Kathpalia <tanmay.kathpalia@gmail.com>

Reviewed-by: Peng Fan <peng.fan@nxp.com>