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...@amd.com>
> ---
>  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   <c...@laptop.org>   <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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to