Hi, On Fri, Jun 29 2012, Aaron Lu wrote: > Some of the host settings are affected by different cards inserted, e.g. > when an UHS-I card is inserted, the SDHCI_NEEDS_RETUING flag might be > set when the tuning timer expired and host's max_blk_count will be > reduced to make sure the data transfer for a command does not exceed 4MiB > to meet the retuning mode 1's requirement. > > When the card is removed, we should restore the original setting of the > host since we can't be sure the next card being inserted will still be > an UHS-I card that needs tuning. The original setting include its > max_blk_count and no set of the flag of SDHCI_NEEDS_RETUNING. > > Signed-off-by: Aaron Lu <aaron.lu@xxxxxxx> > --- > drivers/mmc/host/sdhci.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c > index ff522ec..7e182ad 100644 > --- a/drivers/mmc/host/sdhci.c > +++ b/drivers/mmc/host/sdhci.c > @@ -245,6 +245,18 @@ static void sdhci_init(struct sdhci_host *host, int soft) > static void sdhci_reinit(struct sdhci_host *host) > { > sdhci_init(host, 0); > + /* > + * Retuning stuffs are affected by different cards inserted and only > + * applicable to UHS-I cards. So reset these fields to their initial > + * value when card is removed. > + */ > + if (host->version >= SDHCI_SPEC_300 && host->tuning_count && > + host->tuning_mode == SDHCI_TUNING_MODE_1) { > + del_timer_sync(&host->tuning_timer); > + host->flags &= ~SDHCI_NEEDS_RETUNING; > + host->mmc->max_blk_count = > + (host->quirks & SDHCI_QUIRK_NO_MULTIBLOCK) ? 1 : 65535; > + } > sdhci_enable_card_detection(host); > } Thanks, pushed to mmc-next for 3.6. - Chris. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html