Hi Shimoda-san,
> # In other words, if a controller doesn't have scc, the previous code cannot issue
> # CMD13 in such a case, I guess.
Makes sense.
> tmio_mmc_reset(host);
> + host->set_clock(host, host->clk_cache);
What about putting it into the reset function itself, so it will be
always enabled (like for the scc_ctl case)?
diff --git a/drivers/mmc/host/renesas_sdhi_core.c b/drivers/mmc/host/renesas_sdhi_core.c
index 473f155f6d3d..672953e3362d 100644
--- a/drivers/mmc/host/renesas_sdhi_core.c
+++ b/drivers/mmc/host/renesas_sdhi_core.c
@@ -572,6 +572,7 @@ static void renesas_sdhi_reset(struct tmio_mmc_host *host)
read_poll_timeout(reset_control_status, ret, ret == 0, 1, 100,
false, priv->rstc);
priv->needs_adjust_hs400 = false;
+ renesas_sdhi_set_clock(host, host->clk_cache);
} else if (priv->scc_ctl) {
renesas_sdhi_disable_scc(host->mmc);
renesas_sdhi_reset_hs400_mode(host, priv);
If you agree, I will fold this into v2 of this series.
Attachment:
signature.asc
Description: PGP signature
