mmc: sdhci: Clear SDHCI_HS400_TUNING flag after platform_execute_tuning
authorRitesh Harjani <riteshh@codeaurora.org>
Tue, 10 Jan 2017 07:00:50 +0000 (12:30 +0530)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 13 Feb 2017 12:20:30 +0000 (13:20 +0100)
Clear SDHCI_HS400_TUNING flag after platform_execute_tuning
so that platform_execute_tuning may use it if needed.

Signed-off-by: Ritesh Harjani <riteshh@codeaurora.org>
Tested-by: Jeremy McNicoll <jeremymc@redhat.com>
Acked-by: Adrian Hunter <adrian.hunter@intel.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/sdhci.c

index a2efa25c7f3baad85020a3d3aa2a1fa165fcb243..6fdd7a70f229b8bfd08f6d8b9df509dd0fec2bbd 100644 (file)
@@ -2114,7 +2114,6 @@ int sdhci_execute_tuning(struct mmc_host *mmc, u32 opcode)
        spin_lock_irqsave(&host->lock, flags);
 
        hs400_tuning = host->flags & SDHCI_HS400_TUNING;
-       host->flags &= ~SDHCI_HS400_TUNING;
 
        if (host->tuning_mode == SDHCI_TUNING_MODE_1)
                tuning_count = host->tuning_count;
@@ -2156,7 +2155,9 @@ int sdhci_execute_tuning(struct mmc_host *mmc, u32 opcode)
 
        if (host->ops->platform_execute_tuning) {
                spin_unlock_irqrestore(&host->lock, flags);
-               return host->ops->platform_execute_tuning(host, opcode);
+               err = host->ops->platform_execute_tuning(host, opcode);
+               spin_lock_irqsave(&host->lock, flags);
+               goto out_unlock;
        }
 
        host->mmc->retune_period = tuning_count;
@@ -2167,6 +2168,7 @@ int sdhci_execute_tuning(struct mmc_host *mmc, u32 opcode)
 
        sdhci_end_tuning(host);
 out_unlock:
+       host->flags &= ~SDHCI_HS400_TUNING;
        spin_unlock_irqrestore(&host->lock, flags);
 
        return err;