From: Adrian Hunter Date: Mon, 26 Sep 2022 19:20:20 +0000 (+0300) Subject: mmc: sdhci: Remove misleading comment about resets X-Git-Tag: v6.1-rc5~312^2~3 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bd980df9c4d59ef416d8350f529f7611b06e355e;p=platform%2Fkernel%2Flinux-starfive.git mmc: sdhci: Remove misleading comment about resets 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 Reviewed-by: Thierry Reding Link: https://lore.kernel.org/r/20220926192022.85660-3-adrian.hunter@intel.com Signed-off-by: Ulf Hansson --- diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index a20123df2a6e..4d672e74468f 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -3063,10 +3063,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);