I did not test the patch but want to make you aware of the comment in dw_mmc:
/*
* During UHS tuning sequence, sending the stop
* command after the response CRC error would
* throw the system into a confused state
* causing all future tuning phases to report
* failure.
*
* In such case controller will move into a data
* transfer state after a response error or
* response CRC error. Let's let that finish
* before trying to send a stop, so we'll go to
* STATE_SENDING_DATA.
*
* Although letting the data transfer take place
* will waste a bit of time (we already know
* the command was bad), it can't cause any
* errors since it's possible it would have
* taken place anyway if this tasklet got
* delayed. Allowing the transfer to take place
* avoids races and keeps things simple.
*/
The message in 46d179525a1f6d16957dcb4624517bc04142b3e7
does not mention which card was causing problem, unfortunately.
From: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>
Sent: Tuesday, September 14, 2021 8:20 PM
To: linux-mmc@xxxxxxxxxxxxxxx
Cc: linux-renesas-soc@xxxxxxxxxxxxxxx; Yoshihiro Shimoda; Wolfram Sang
Subject: [PATCH 0/3] mmc: also abort tuning with CMD12 for SD
After my hackish RFC patch, here is a small series implementing
(hopefully) the solution we discussed. It will make
mmc_send_abort_tuning() also send CMD12 for SD cards which makes more SD
cards work for us. Details are in the patch descriptions.
Please let me know what you think.
Thanks, and happy hacking!
Wolfram Sang (3):
mmc: core: add helper to send STOP
mmc: core: also abort tuning with CMD12 for SD
mmc: core: remove obsolete parameter from mmc_send_abort_tuning
drivers/mmc/core/block.c | 14 +-------------
drivers/mmc/core/core.h | 1 +
drivers/mmc/core/mmc.c | 6 ++++++
drivers/mmc/core/mmc_ops.c | 23 ++++-------------------
drivers/mmc/core/mmc_ops.h | 14 ++++++++++++++
drivers/mmc/core/sd.c | 6 ++++++
drivers/mmc/host/renesas_sdhi_core.c | 2 +-
drivers/mmc/host/sdhci.c | 2 +-
include/linux/mmc/host.h | 2 +-
9 files changed, 35 insertions(+), 35 deletions(-)
--
2.30.2
=
Hyperstone GmbH | Line-Eid-Strasse 3 | 78467 Konstanz
Managing Directors: Dr. Jan Peter Berns.
Commercial register of local courts: Freiburg HRB381782
