From: Hai Pham <hai.pham...@renesas.com> When tuning command (CMD21) fails with command error, call mmc_send_stop_transmission() to send stop command (CMD12).
Reviewed-by: Takeshi Kihara <takeshi.kihara...@renesas.com> Reviewed-by: Marek Vasut <marek.vasut+rene...@mailbox.org> Signed-off-by: Hai Pham <hai.pham...@renesas.com> Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org> [Marek: Add dev_dbg() message in case tuning abort fails Move tuning opcode check from mmc_abort_tuning()] --- Cc: "Ying-Chun Liu (PaulLiu)" <paul....@linaro.org> Cc: Hai Pham <hai.pham...@renesas.com> Cc: Jaehoon Chung <jh80.ch...@samsung.com> Cc: Loic Poulain <loic.poul...@linaro.org> Cc: Peng Fan <peng....@nxp.com> Cc: Simon Glass <s...@chromium.org> Cc: Takeshi Kihara <takeshi.kihara...@renesas.com> --- drivers/mmc/renesas-sdhi.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/drivers/mmc/renesas-sdhi.c b/drivers/mmc/renesas-sdhi.c index 280d96dbc2d..74d98795838 100644 --- a/drivers/mmc/renesas-sdhi.c +++ b/drivers/mmc/renesas-sdhi.c @@ -611,6 +611,17 @@ int renesas_sdhi_execute_tuning(struct udevice *dev, uint opcode) priv->smpcmp |= BIT(i); mdelay(1); + + /* + * eMMC specification specifies that CMD12 can be used to stop a tuning + * command, but SD specification does not, so do nothing unless it is + * eMMC. + */ + if (ret && (opcode == MMC_CMD_SEND_TUNING_BLOCK_HS200)) { + ret = mmc_send_stop_transmission(mmc); + if (ret < 0) + dev_dbg(dev, "Tuning abort fail (%d)\n", ret); + } } ret = renesas_sdhi_select_tuning(priv, taps); -- 2.39.2