Hi!
On 29/10/15 16:53, Jaehoon Chung wrote:
There is no place where host->clock is assgined to any value. If real clock is changed to mmc->clock, host->clock can be assigned to mmc->clock. If card request to same clock value, it doesn't need to call sdhci_set_clock() function.
Signed-off-by: Jaehoon Chung jh80.chung@samsung.com
drivers/mmc/sdhci.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c index d89e302..3f87a23 100644 --- a/drivers/mmc/sdhci.c +++ b/drivers/mmc/sdhci.c @@ -380,8 +380,10 @@ static void sdhci_set_ios(struct mmc *mmc) if (host->set_control_reg) host->set_control_reg(host);
- if (mmc->clock != host->clock)
sdhci_set_clock(mmc, mmc->clock);
- if (mmc->clock != host->clock) {
if (!sdhci_set_clock(mmc, mmc->clock))host->clock = mmc->clock;
I think it's better to move into sdhci_set_clock function.
}
/* Set bus width */ ctrl = sdhci_readb(host, SDHCI_HOST_CONTROL);
Thanks, Minkyu Kang.