mmc: renesas_sdhi: drop local flag for tuning
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Tue, 22 Sep 2020 17:22:53 +0000 (19:22 +0200)
committerUlf Hansson <ulf.hansson@linaro.org>
Fri, 25 Sep 2020 11:24:02 +0000 (13:24 +0200)
The MMC core has now a generic check if some tuning is in progress. Its
protected area is a bit larger than the custom one in this driver but we
concluded that this works equally well for the intended case. So, drop
the local flag and switch to the generic one.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Tested-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Link: https://lore.kernel.org/r/20200922172253.4458-1-wsa@kernel.org
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/renesas_sdhi.h
drivers/mmc/host/renesas_sdhi_core.c

index 24958de274c174af42f196e282d38c7f31bd1686..cb962c7883dcd05a537b129aabca21aa2330c985 100644 (file)
@@ -63,7 +63,6 @@ struct renesas_sdhi {
        u32 scc_tappos_hs400;
        const u8 *adjust_hs400_calib_table;
        bool needs_adjust_hs400;
-       bool doing_tune;
 
        /* Tuning values: 1 for success, 0 for failure */
        DECLARE_BITMAP(taps, BITS_PER_LONG);
index d621a4af8e87edbb713712c2413245f665f2e98c..20e5eb63caf8b7f829a2c197a4bf984d379c2b66 100644 (file)
@@ -581,7 +581,6 @@ static int renesas_sdhi_select_tuning(struct tmio_mmc_host *host)
        unsigned int taps_size = priv->tap_num * 2, min_tap_row;
        unsigned long *bitmap;
 
-       priv->doing_tune = false;
        sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_RVSREQ, 0);
 
        /*
@@ -656,7 +655,6 @@ static int renesas_sdhi_execute_tuning(struct mmc_host *mmc, u32 opcode)
                return -EINVAL;
        }
 
-       priv->doing_tune = true;
        bitmap_zero(priv->taps, priv->tap_num * 2);
        bitmap_zero(priv->smpcmp, priv->tap_num * 2);
 
@@ -765,7 +763,7 @@ static bool renesas_sdhi_check_scc_error(struct tmio_mmc_host *host)
            !(host->mmc->ios.timing == MMC_TIMING_MMC_HS400 && !use_4tap))
                return false;
 
-       if (mmc_doing_retune(host->mmc) || priv->doing_tune)
+       if (mmc_doing_tune(host->mmc))
                return false;
 
        if (sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL) &