usb: phy: msm: Move global regulators variables to driver state
authorIvan T. Ivanov <iivanov@mm-sol.com>
Mon, 28 Apr 2014 13:34:06 +0000 (16:34 +0300)
committerFelipe Balbi <balbi@ti.com>
Wed, 30 Apr 2014 16:28:04 +0000 (11:28 -0500)
Eliminating global variables allows driver to handle multiple
device instances.

Signed-off-by: Ivan T. Ivanov <iivanov@mm-sol.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/phy/phy-msm-usb.c
include/linux/usb/msm_hsusb.h

index 5b37b81f2ef6f30827eb87c8ee80a6af11bcd8f0..878f67d29ed54215db952590d1f8d725fa41b939 100644 (file)
 #define USB_PHY_VDD_DIG_VOL_MIN        1000000 /* uV */
 #define USB_PHY_VDD_DIG_VOL_MAX        1320000 /* uV */
 
-static struct regulator *hsusb_3p3;
-static struct regulator *hsusb_1p8;
-static struct regulator *hsusb_vddcx;
-
 static int msm_hsusb_init_vddcx(struct msm_otg *motg, int init)
 {
        int ret = 0;
 
        if (init) {
-               hsusb_vddcx = regulator_get(motg->phy.dev, "HSUSB_VDDCX");
-               if (IS_ERR(hsusb_vddcx)) {
+               motg->vddcx = regulator_get(motg->phy.dev, "HSUSB_VDDCX");
+               if (IS_ERR(motg->vddcx)) {
                        dev_err(motg->phy.dev, "unable to get hsusb vddcx\n");
-                       return PTR_ERR(hsusb_vddcx);
+                       return PTR_ERR(motg->vddcx);
                }
 
-               ret = regulator_set_voltage(hsusb_vddcx,
+               ret = regulator_set_voltage(motg->vddcx,
                                USB_PHY_VDD_DIG_VOL_MIN,
                                USB_PHY_VDD_DIG_VOL_MAX);
                if (ret) {
                        dev_err(motg->phy.dev, "unable to set the voltage "
                                        "for hsusb vddcx\n");
-                       regulator_put(hsusb_vddcx);
+                       regulator_put(motg->vddcx);
                        return ret;
                }
 
-               ret = regulator_enable(hsusb_vddcx);
+               ret = regulator_enable(motg->vddcx);
                if (ret) {
                        dev_err(motg->phy.dev, "unable to enable hsusb vddcx\n");
-                       regulator_put(hsusb_vddcx);
+                       regulator_put(motg->vddcx);
                }
        } else {
-               ret = regulator_set_voltage(hsusb_vddcx, 0,
+               ret = regulator_set_voltage(motg->vddcx, 0,
                        USB_PHY_VDD_DIG_VOL_MAX);
                if (ret)
                        dev_err(motg->phy.dev, "unable to set the voltage "
                                        "for hsusb vddcx\n");
-               ret = regulator_disable(hsusb_vddcx);
+               ret = regulator_disable(motg->vddcx);
                if (ret)
                        dev_err(motg->phy.dev, "unable to disable hsusb vddcx\n");
 
-               regulator_put(hsusb_vddcx);
+               regulator_put(motg->vddcx);
        }
 
        return ret;
@@ -109,38 +105,38 @@ static int msm_hsusb_ldo_init(struct msm_otg *motg, int init)
        int rc = 0;
 
        if (init) {
-               hsusb_3p3 = regulator_get(motg->phy.dev, "HSUSB_3p3");
-               if (IS_ERR(hsusb_3p3)) {
+               motg->v3p3 = regulator_get(motg->phy.dev, "HSUSB_3p3");
+               if (IS_ERR(motg->v3p3)) {
                        dev_err(motg->phy.dev, "unable to get hsusb 3p3\n");
-                       return PTR_ERR(hsusb_3p3);
+                       return PTR_ERR(motg->v3p3);
                }
 
-               rc = regulator_set_voltage(hsusb_3p3, USB_PHY_3P3_VOL_MIN,
+               rc = regulator_set_voltage(motg->v3p3, USB_PHY_3P3_VOL_MIN,
                                USB_PHY_3P3_VOL_MAX);
                if (rc) {
                        dev_err(motg->phy.dev, "unable to set voltage level "
                                        "for hsusb 3p3\n");
                        goto put_3p3;
                }
-               rc = regulator_enable(hsusb_3p3);
+               rc = regulator_enable(motg->v3p3);
                if (rc) {
                        dev_err(motg->phy.dev, "unable to enable the hsusb 3p3\n");
                        goto put_3p3;
                }
-               hsusb_1p8 = regulator_get(motg->phy.dev, "HSUSB_1p8");
-               if (IS_ERR(hsusb_1p8)) {
+               motg->v1p8 = regulator_get(motg->phy.dev, "HSUSB_1p8");
+               if (IS_ERR(motg->v1p8)) {
                        dev_err(motg->phy.dev, "unable to get hsusb 1p8\n");
-                       rc = PTR_ERR(hsusb_1p8);
+                       rc = PTR_ERR(motg->v1p8);
                        goto disable_3p3;
                }
-               rc = regulator_set_voltage(hsusb_1p8, USB_PHY_1P8_VOL_MIN,
+               rc = regulator_set_voltage(motg->v1p8, USB_PHY_1P8_VOL_MIN,
                                USB_PHY_1P8_VOL_MAX);
                if (rc) {
                        dev_err(motg->phy.dev, "unable to set voltage level "
                                        "for hsusb 1p8\n");
                        goto put_1p8;
                }
-               rc = regulator_enable(hsusb_1p8);
+               rc = regulator_enable(motg->v1p8);
                if (rc) {
                        dev_err(motg->phy.dev, "unable to enable the hsusb 1p8\n");
                        goto put_1p8;
@@ -149,54 +145,54 @@ static int msm_hsusb_ldo_init(struct msm_otg *motg, int init)
                return 0;
        }
 
-       regulator_disable(hsusb_1p8);
+       regulator_disable(motg->v1p8);
 put_1p8:
-       regulator_put(hsusb_1p8);
+       regulator_put(motg->v1p8);
 disable_3p3:
-       regulator_disable(hsusb_3p3);
+       regulator_disable(motg->v3p3);
 put_3p3:
-       regulator_put(hsusb_3p3);
+       regulator_put(motg->v3p3);
        return rc;
 }
 
-static int msm_hsusb_ldo_set_mode(int on)
+static int msm_hsusb_ldo_set_mode(struct msm_otg *motg, int on)
 {
        int ret = 0;
 
-       if (!hsusb_1p8 || IS_ERR(hsusb_1p8)) {
+       if (!motg->v1p8 || IS_ERR(motg->v1p8)) {
                pr_err("%s: HSUSB_1p8 is not initialized\n", __func__);
                return -ENODEV;
        }
 
-       if (!hsusb_3p3 || IS_ERR(hsusb_3p3)) {
+       if (!motg->v3p3 || IS_ERR(motg->v3p3)) {
                pr_err("%s: HSUSB_3p3 is not initialized\n", __func__);
                return -ENODEV;
        }
 
        if (on) {
-               ret = regulator_set_optimum_mode(hsusb_1p8,
+               ret = regulator_set_optimum_mode(motg->v1p8,
                                USB_PHY_1P8_HPM_LOAD);
                if (ret < 0) {
                        pr_err("%s: Unable to set HPM of the regulator "
                                "HSUSB_1p8\n", __func__);
                        return ret;
                }
-               ret = regulator_set_optimum_mode(hsusb_3p3,
+               ret = regulator_set_optimum_mode(motg->v3p3,
                                USB_PHY_3P3_HPM_LOAD);
                if (ret < 0) {
                        pr_err("%s: Unable to set HPM of the regulator "
                                "HSUSB_3p3\n", __func__);
-                       regulator_set_optimum_mode(hsusb_1p8,
+                       regulator_set_optimum_mode(motg->v1p8,
                                USB_PHY_1P8_LPM_LOAD);
                        return ret;
                }
        } else {
-               ret = regulator_set_optimum_mode(hsusb_1p8,
+               ret = regulator_set_optimum_mode(motg->v1p8,
                                USB_PHY_1P8_LPM_LOAD);
                if (ret < 0)
                        pr_err("%s: Unable to set LPM of the regulator "
                                "HSUSB_1p8\n", __func__);
-               ret = regulator_set_optimum_mode(hsusb_3p3,
+               ret = regulator_set_optimum_mode(motg->v3p3,
                                USB_PHY_3P3_LPM_LOAD);
                if (ret < 0)
                        pr_err("%s: Unable to set LPM of the regulator "
@@ -417,7 +413,7 @@ static int msm_otg_reset(struct usb_phy *phy)
 #ifdef CONFIG_PM
 
 #define USB_PHY_SUSP_DIG_VOL  500000
-static int msm_hsusb_config_vddcx(int high)
+static int msm_hsusb_config_vddcx(struct msm_otg *motg, int high)
 {
        int max_vol = USB_PHY_VDD_DIG_VOL_MAX;
        int min_vol;
@@ -428,7 +424,7 @@ static int msm_hsusb_config_vddcx(int high)
        else
                min_vol = USB_PHY_SUSP_DIG_VOL;
 
-       ret = regulator_set_voltage(hsusb_vddcx, min_vol, max_vol);
+       ret = regulator_set_voltage(motg->vddcx, min_vol, max_vol);
        if (ret) {
                pr_err("%s: unable to set the voltage for regulator "
                        "HSUSB_VDDCX\n", __func__);
@@ -518,8 +514,8 @@ static int msm_otg_suspend(struct msm_otg *motg)
 
        if (motg->pdata->phy_type == SNPS_28NM_INTEGRATED_PHY &&
                        motg->pdata->otg_control == OTG_PMIC_CONTROL) {
-               msm_hsusb_ldo_set_mode(0);
-               msm_hsusb_config_vddcx(0);
+               msm_hsusb_ldo_set_mode(motg, 0);
+               msm_hsusb_config_vddcx(motg, 0);
        }
 
        if (device_may_wakeup(phy->dev))
@@ -555,8 +551,8 @@ static int msm_otg_resume(struct msm_otg *motg)
 
        if (motg->pdata->phy_type == SNPS_28NM_INTEGRATED_PHY &&
                        motg->pdata->otg_control == OTG_PMIC_CONTROL) {
-               msm_hsusb_ldo_set_mode(1);
-               msm_hsusb_config_vddcx(1);
+               msm_hsusb_ldo_set_mode(motg, 1);
+               msm_hsusb_config_vddcx(motg, 1);
                writel(readl(USB_PHY_CTRL) & ~PHY_RETEN, USB_PHY_CTRL);
        }
 
@@ -1521,7 +1517,7 @@ static int __init msm_otg_probe(struct platform_device *pdev)
                dev_err(&pdev->dev, "hsusb vreg configuration failed\n");
                goto vddcx_exit;
        }
-       ret = msm_hsusb_ldo_set_mode(1);
+       ret = msm_hsusb_ldo_set_mode(motg, 1);
        if (ret) {
                dev_err(&pdev->dev, "hsusb vreg enable failed\n");
                goto ldo_exit;
index 32754835a39ba87743ea1c81419a110d82c1ebf7..8705b01646849c18b8fd4ae8fba5b11cccb00bdd 100644 (file)
@@ -183,6 +183,9 @@ struct msm_otg {
        enum usb_chg_state chg_state;
        enum usb_chg_type chg_type;
        u8 dcd_retries;
+       struct regulator *v3p3;
+       struct regulator *v1p8;
+       struct regulator *vddcx;
 };
 
 #endif