Commit 24c85258 authored by Linus Torvalds's avatar Linus Torvalds

Merge tag 'regulator-v3.14-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator

Pull regulator fixes from Mark Brown:
 "Mostly unexciting driver fixes, plus one fix to lower the severity of
  the log message when we don't use an optional regulator - the fixes
  for ACPI system made this come up more often and it was correctly
  observed that it was causing undue concern for users"

* tag 'regulator-v3.14-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator:
  regulator: max14577: Fix invalid return value on DT parse success
  regulator: core: Change dummy supplies error message to a warning
  regulator: s5m8767: Add missing of_node_put
  regulator: s5m8767: Use of_get_child_by_name
  regulator: da9063: Bug fix when setting max voltage on LDOs 5-11
parents b2880eb8 bcad2ca3
...@@ -1359,7 +1359,7 @@ static struct regulator *_regulator_get(struct device *dev, const char *id, ...@@ -1359,7 +1359,7 @@ static struct regulator *_regulator_get(struct device *dev, const char *id,
goto found; goto found;
/* Don't log an error when called from regulator_get_optional() */ /* Don't log an error when called from regulator_get_optional() */
} else if (!have_full_constraints() || exclusive) { } else if (!have_full_constraints() || exclusive) {
dev_err(dev, "dummy supplies not allowed\n"); dev_warn(dev, "dummy supplies not allowed\n");
} }
mutex_unlock(&regulator_list_mutex); mutex_unlock(&regulator_list_mutex);
......
/* /*
* Regulator driver for DA9063 PMIC series * Regulator driver for DA9063 PMIC series
* *
...@@ -60,7 +61,8 @@ struct da9063_regulator_info { ...@@ -60,7 +61,8 @@ struct da9063_regulator_info {
.desc.ops = &da9063_ldo_ops, \ .desc.ops = &da9063_ldo_ops, \
.desc.min_uV = (min_mV) * 1000, \ .desc.min_uV = (min_mV) * 1000, \
.desc.uV_step = (step_mV) * 1000, \ .desc.uV_step = (step_mV) * 1000, \
.desc.n_voltages = (((max_mV) - (min_mV))/(step_mV) + 1), \ .desc.n_voltages = (((max_mV) - (min_mV))/(step_mV) + 1 \
+ (DA9063_V##regl_name##_BIAS)), \
.desc.enable_reg = DA9063_REG_##regl_name##_CONT, \ .desc.enable_reg = DA9063_REG_##regl_name##_CONT, \
.desc.enable_mask = DA9063_LDO_EN, \ .desc.enable_mask = DA9063_LDO_EN, \
.desc.vsel_reg = DA9063_REG_V##regl_name##_A, \ .desc.vsel_reg = DA9063_REG_V##regl_name##_A, \
......
...@@ -166,9 +166,10 @@ static int max14577_regulator_dt_parse_pdata(struct platform_device *pdev) ...@@ -166,9 +166,10 @@ static int max14577_regulator_dt_parse_pdata(struct platform_device *pdev)
ret = of_regulator_match(&pdev->dev, np, max14577_regulator_matches, ret = of_regulator_match(&pdev->dev, np, max14577_regulator_matches,
MAX14577_REG_MAX); MAX14577_REG_MAX);
if (ret < 0) { if (ret < 0)
dev_err(&pdev->dev, "Error parsing regulator init data: %d\n", ret); dev_err(&pdev->dev, "Error parsing regulator init data: %d\n", ret);
} else
ret = 0;
of_node_put(np); of_node_put(np);
......
...@@ -535,7 +535,7 @@ static int s5m8767_pmic_dt_parse_pdata(struct platform_device *pdev, ...@@ -535,7 +535,7 @@ static int s5m8767_pmic_dt_parse_pdata(struct platform_device *pdev,
return -ENODEV; return -ENODEV;
} }
regulators_np = of_find_node_by_name(pmic_np, "regulators"); regulators_np = of_get_child_by_name(pmic_np, "regulators");
if (!regulators_np) { if (!regulators_np) {
dev_err(iodev->dev, "could not find regulators sub-node\n"); dev_err(iodev->dev, "could not find regulators sub-node\n");
return -EINVAL; return -EINVAL;
...@@ -591,6 +591,8 @@ static int s5m8767_pmic_dt_parse_pdata(struct platform_device *pdev, ...@@ -591,6 +591,8 @@ static int s5m8767_pmic_dt_parse_pdata(struct platform_device *pdev,
rmode++; rmode++;
} }
of_node_put(regulators_np);
if (of_get_property(pmic_np, "s5m8767,pmic-buck2-uses-gpio-dvs", NULL)) { if (of_get_property(pmic_np, "s5m8767,pmic-buck2-uses-gpio-dvs", NULL)) {
pdata->buck2_gpiodvs = true; pdata->buck2_gpiodvs = true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment