[PATCH v6 15/23] regulator: max77686: Setup DVS-related GPIOs on probe

On Fri, Jul 11, 2014 at 7:33 AM, Javier Martinez Canillas
[off-list ref] wrote:

Hello Amit,

On 07/10/2014 12:08 PM, amit daniel kachhap wrote:

quoted

On Fri, Jul 4, 2014 at 3:25 PM, Javier Martinez Canillas
[off-list ref] wrote:

quoted

MAX77686 PMIC support Dyamic Voltage Scaling (DVS) on a set
of Buck regulators. A number of GPIO are connected to these
lines and are requested by the mfd driver. Setup the GPIO
pins from the regulator driver.

If possible merge this patch with patch 8. Both are adding DVS
support. Put regmap_copy dependency patch in very beginning.

As Lee already said, I split the changes to minimize the cross-subsystem churn.

Yes agreed. This comment doesn't hold.

quoted

quoted

Signed-off-by: Javier Martinez Canillas <redacted>
---
 drivers/regulator/max77686.c | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/drivers/regulator/max77686.c b/drivers/regulator/max77686.c
index ef1af2d..ecce77a 100644
--- a/drivers/regulator/max77686.c
+++ b/drivers/regulator/max77686.c

@@ -435,6 +435,12 @@ static int max77686_pmic_dt_parse_pdata(struct platform_device *pdev,
 }
 #endif /* CONFIG_OF */

+static inline bool max77686_is_dvs_buck(int id)
+{
+       /* BUCK 2,3 and 4 support DVS */
+       return (id >= MAX77686_BUCK2 && id <= MAX77686_BUCK4);

I am just wondering if along with above check, SELB gpios (if present)
can be used to confirm if BUCK's are DVS based or not.

I don't know if SELB gpios being present or not should be used to determine
whether a BUCK includes the DVS feature. AFAIK boards could have some of these
lines hardwired and pulled high or low instead of using a GPIO.

As per the max77686 data sheet, selb2.3.4 uses logic high for no DVS
and logic low for DVS enabled.
So may be if DT is supplying selb gpios then the above checks can be
put otherwise not required.
Anyway from your other comments, since this patch series is not
handling complete DVS scenario.
So putting this check is not useful in this stage.

quoted

quoted

+}
+
 static int max77686_pmic_probe(struct platform_device *pdev)
 {
        struct max77686_dev *iodev = dev_get_drvdata(pdev->dev.parent);

@@ -442,6 +448,9 @@ static int max77686_pmic_probe(struct platform_device *pdev)
        struct max77686_data *max77686;
        int i, ret = 0;
        struct regulator_config config = { };
+       unsigned int reg;
+       int buck_default_idx;
+       int buck_old_idx;

        dev_dbg(&pdev->dev, "%s\n", __func__);

@@ -472,13 +481,34 @@ static int max77686_pmic_probe(struct platform_device *pdev)
        config.driver_data = max77686;
        platform_set_drvdata(pdev, max77686);

+       buck_default_idx = pdata->buck_default_idx;
+       buck_old_idx = max77686_read_gpios(pdata);
+
        for (i = 0; i < MAX77686_REGULATORS; i++) {
                struct regulator_dev *rdev;
+               int id = pdata->regulators[i].id;

                config.init_data = pdata->regulators[i].initdata;
                config.of_node = pdata->regulators[i].of_node;

                max77686->opmode[i] = regulators[i].enable_mask;
+
+               if (max77686_is_dvs_buck(id)) {
+                       /* Try to copy over data so we keep firmware settings */
+                       reg = regulators[i].vsel_reg;
+
+                       ret = regmap_reg_copy(iodev->regmap,
+                                             reg + buck_default_idx,
+                                             reg + buck_old_idx);
+
+                       if (ret)
+                               dev_warn(&pdev->dev, "Copy err %d => %d (%d)\n",
+                                        reg + buck_old_idx,
+                                        reg + buck_default_idx, ret);
+
+                       regulators[i].vsel_reg += buck_default_idx;
+               }
+
                rdev = devm_regulator_register(&pdev->dev,
                                                &regulators[i], &config);
                if (IS_ERR(rdev)) {

@@ -488,6 +518,10 @@ static int max77686_pmic_probe(struct platform_device *pdev)
                }
        }

+       ret = max77686_setup_gpios(iodev->dev);
+       if (ret)
+               return ret;
+
        return 0;
 }

--

2.0.0.rc2

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html