On 06/09/2012 12:56 AM, philipspatc...@gmail.com wrote:
> From: Philip Rakity <prak...@marvell.com>
> 
> If we are using a regulator the SD Host Controller and the
> regulator should agree about the voltages supported.  Use
> the common subset that is supported.
> 
> Signed-off-by: Philip Rakity <prak...@marvell.com>
> ---
>  drivers/mmc/host/sdhci.c |   15 +++++++++++++++
>  1 files changed, 15 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
> index 248f68b..78be427 100644
> --- a/drivers/mmc/host/sdhci.c
> +++ b/drivers/mmc/host/sdhci.c
> @@ -2894,6 +2894,21 @@ int sdhci_add_host(struct sdhci_host *host)
>        * register is set. The actual current value is 4 times the register
>        * value.
>        */
> +     if (host->vmmc) {
> +             ret = regulator_is_supported_voltage(host->vmmc, 3300000,
> +                     3300000);
> +             if ((ret <= 0) || (!(caps[0] & SDHCI_CAN_VDD_330)))
> +                     caps[0] &= ~SDHCI_CAN_VDD_330;
> +             ret = regulator_is_supported_voltage(host->vmmc, 3000000,
> +                     3000000);
> +             if ((ret <= 0) || (!(caps[0] & SDHCI_CAN_VDD_300)))
> +                     caps[0] &= ~SDHCI_CAN_VDD_300;
> +             ret = regulator_is_supported_voltage(host->vmmc, 1800000,
> +                     1800000);
> +             if ((ret <= 0) || (!(caps[0] & SDHCI_CAN_VDD_180)))
> +                     caps[0] &= ~SDHCI_CAN_VDD_180;
> +     }
> +
>       max_current_caps = sdhci_readl(host, SDHCI_MAX_CURRENT);
>       if (!max_current_caps && host->vmmc) {
>               u32 curr = regulator_get_current_limit(host->vmmc);
> 

Does the above code mean that 2.8V for vmmc regulator is not a valid value?

Currently I am working on a board for which vmmc regulator voltage is
set as 2.8V and the SD/MMC card is working fine (by not defining vmmc in
platform code).

-- 
Tushar Behera
--
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