mmc: host: omap_hsmmc: add separate functions for enable/disable supply
authorKishon Vijay Abraham I <kishon@ti.com>
Thu, 27 Aug 2015 09:14:00 +0000 (14:44 +0530)
committerUlf Hansson <ulf.hansson@linaro.org>
Thu, 27 Aug 2015 12:54:06 +0000 (14:54 +0200)
No functional change. Cleanup omap_hsmmc_set_power by adding separate
functions for enable/disable supply and invoke it from
omap_hsmmc_set_power.

Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
Tested-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/omap_hsmmc.c

index 284ab0063156e9693425987dc03ecdf1e7835197..3fa78d477a415ee236485876129281138346147e 100644 (file)
@@ -246,6 +246,65 @@ static int omap_hsmmc_get_cover_state(struct device *dev)
 
 #ifdef CONFIG_REGULATOR
 
+static int omap_hsmmc_enable_supply(struct mmc_host *mmc, int vdd)
+{
+       int ret;
+
+       if (mmc->supply.vmmc) {
+               ret = mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, vdd);
+               if (ret)
+                       return ret;
+       }
+
+       /* Enable interface voltage rail, if needed */
+       if (mmc->supply.vqmmc) {
+               ret = regulator_enable(mmc->supply.vqmmc);
+               if (ret) {
+                       dev_err(mmc_dev(mmc), "vmmc_aux reg enable failed\n");
+                       goto err_vqmmc;
+               }
+       }
+
+       return 0;
+
+err_vqmmc:
+       if (mmc->supply.vmmc)
+               mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, 0);
+
+       return ret;
+}
+
+static int omap_hsmmc_disable_supply(struct mmc_host *mmc)
+{
+       int ret;
+       int status;
+
+       if (mmc->supply.vqmmc) {
+               ret = regulator_disable(mmc->supply.vqmmc);
+               if (ret) {
+                       dev_err(mmc_dev(mmc), "vmmc_aux reg disable failed\n");
+                       return ret;
+               }
+       }
+
+       if (mmc->supply.vmmc) {
+               ret = mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, 0);
+               if (ret)
+                       goto err_set_ocr;
+       }
+
+       return 0;
+
+err_set_ocr:
+       if (mmc->supply.vqmmc) {
+               status = regulator_enable(mmc->supply.vqmmc);
+               if (status)
+                       dev_err(mmc_dev(mmc), "vmmc_aux re-enable failed\n");
+       }
+
+       return ret;
+}
+
 static int omap_hsmmc_set_power(struct device *dev, int power_on, int vdd)
 {
        struct omap_hsmmc_host *host =
@@ -291,36 +350,13 @@ static int omap_hsmmc_set_power(struct device *dev, int power_on, int vdd)
         * chips/cards need an interface voltage rail too.
         */
        if (power_on) {
-               if (mmc->supply.vmmc) {
-                       ret = mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, vdd);
-                       if (ret)
-                               return ret;
-               }
-
-               /* Enable interface voltage rail, if needed */
-               if (mmc->supply.vqmmc) {
-                       ret = regulator_enable(mmc->supply.vqmmc);
-                       if (ret) {
-                               dev_err(dev, "vmmc_aux reg enable failed\n");
-                               goto err_set_vqmmc;
-                       }
-               }
+               ret = omap_hsmmc_enable_supply(mmc, vdd);
+               if (ret)
+                       return ret;
        } else {
-               /* Shut down the rail */
-               if (mmc->supply.vqmmc) {
-                       ret = regulator_disable(mmc->supply.vqmmc);
-                       if (ret) {
-                               dev_err(dev, "vmmc_aux reg disable failed\n");
-                               return ret;
-                       }
-               }
-
-               if (mmc->supply.vmmc) {
-                       /* Then proceed to shut down the local regulator */
-                       ret = mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, 0);
-                       if (ret)
-                               return ret;
-               }
+               ret = omap_hsmmc_disable_supply(mmc);
+               if (ret)
+                       return ret;
        }
 
        if (host->pbias) {
@@ -350,12 +386,7 @@ static int omap_hsmmc_set_power(struct device *dev, int power_on, int vdd)
        return 0;
 
 err_set_voltage:
-       if (mmc->supply.vqmmc)
-               regulator_disable(mmc->supply.vqmmc);
-
-err_set_vqmmc:
-       if (mmc->supply.vmmc)
-               mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, 0);
+       omap_hsmmc_disable_supply(mmc);
 
        return ret;
 }