mmc: sdhci: Balance vmmc regulator_enable(), and always enable vqmmc
authorKevin Liu <kliu5@marvell.com>
Sat, 17 Nov 2012 22:55:51 +0000 (17:55 -0500)
committerChris Ball <cjb@laptop.org>
Thu, 6 Dec 2012 18:54:35 +0000 (13:54 -0500)
The vmmc regulator enable in sdhci_add_host is NOT necessary since
it can be enabled during mmc_power_up by function mmc_regulator_set_ocr.
And this extra enable will make regulator_enable/regulator_disable
unbalanced. Consequently, vmmc can't be disabled during mmc_power_off.

Also, if the vqmmc regulator exists, it should be enabled regardless it
support 1.8v or not.

Signed-off-by: Kevin Liu <kliu5@marvell.com>
Signed-off-by: Philip Rakity <prakity@marvell.com>
Signed-off-by: Chris Ball <cjb@laptop.org>
drivers/mmc/host/sdhci.c

index c7851c0aabce52226d118d50bbe95fe61b4094c2..2f62fe4309f2fdccf6838d5760ec88f72b32125d 100644 (file)
@@ -2858,12 +2858,14 @@ int sdhci_add_host(struct sdhci_host *host)
                                mmc_hostname(mmc));
                        host->vqmmc = NULL;
                }
-       }
-       else if (regulator_is_supported_voltage(host->vqmmc, 1800000, 1800000))
+       } else {
                regulator_enable(host->vqmmc);
-       else
-               caps[1] &= ~(SDHCI_SUPPORT_SDR104 | SDHCI_SUPPORT_SDR50 |
-                      SDHCI_SUPPORT_DDR50);
+               if (!regulator_is_supported_voltage(host->vqmmc, 1800000,
+                       1800000))
+                       caps[1] &= ~(SDHCI_SUPPORT_SDR104 |
+                                       SDHCI_SUPPORT_SDR50 |
+                                       SDHCI_SUPPORT_DDR50);
+       }
 
        /* Any UHS-I mode in caps implies SDR12 and SDR25 support. */
        if (caps[1] & (SDHCI_SUPPORT_SDR104 | SDHCI_SUPPORT_SDR50 |
@@ -2919,8 +2921,7 @@ int sdhci_add_host(struct sdhci_host *host)
                                mmc_hostname(mmc));
                        host->vmmc = NULL;
                }
-       } else
-               regulator_enable(host->vmmc);
+       }
 
 #ifdef CONFIG_REGULATOR
        if (host->vmmc) {