Message ID | 20220926192022.85660-3-adrian.hunter@intel.com |
---|---|
State | New |
Headers | show |
Series | mmc: sdhci: Tidy reset handling | expand |
diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 34b351363f41..100c7d6be385 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -3059,10 +3059,6 @@ static bool sdhci_request_done(struct sdhci_host *host) /* This is to force an update */ host->ops->set_clock(host, host->clock); - /* - * Spec says we should do both at the same time, but Ricoh - * controllers do not like that. - */ sdhci_do_reset(host, SDHCI_RESET_CMD); sdhci_do_reset(host, SDHCI_RESET_DATA);
In SDHCI specification section 3.10.1 Error Interrupt Recovery, the flow chart shows Software Reset for CMD separately and before Software Reset for DAT, so the comment "Spec says we should do both at the same time" is not correct. Remove it. Signed-off-by: Adrian Hunter <adrian.hunter@intel.com> --- drivers/mmc/host/sdhci.c | 4 ---- 1 file changed, 4 deletions(-)