* [PATCH 2/9] regulator: as3722: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 1:59 ` Mark Brown
2014-02-17 9:03 ` [PATCH 3/9] regulator: da9063: " Sachin Kamat
` (7 subsequent siblings)
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/as3722-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/as3722-regulator.c b/drivers/regulator/as3722-regulator.c
index 8b17d786cb71..85585219ce82 100644
--- a/drivers/regulator/as3722-regulator.c
+++ b/drivers/regulator/as3722-regulator.c
@@ -719,6 +719,7 @@ static int as3722_get_regulator_dt_data(struct platform_device *pdev,
ret = of_regulator_match(&pdev->dev, np, as3722_regulator_matches,
ARRAY_SIZE(as3722_regulator_matches));
+ of_node_put(np);
if (ret < 0) {
dev_err(&pdev->dev, "Parsing of regulator node failed: %d\n",
ret);
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 3/9] regulator: da9063: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
2014-02-17 9:03 ` [PATCH 2/9] regulator: as3722: " Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 2:00 ` Mark Brown
2014-02-17 9:03 ` [PATCH 4/9] regulator: max77686: " Sachin Kamat
` (6 subsequent siblings)
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/da9063-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/da9063-regulator.c b/drivers/regulator/da9063-regulator.c
index bfd7f2f795a6..a61b5d997a08 100644
--- a/drivers/regulator/da9063-regulator.c
+++ b/drivers/regulator/da9063-regulator.c
@@ -674,6 +674,7 @@ static struct da9063_regulators_pdata *da9063_parse_regulators_dt(
num = of_regulator_match(&pdev->dev, node, da9063_matches,
ARRAY_SIZE(da9063_matches));
+ of_node_put(node);
if (num < 0) {
dev_err(&pdev->dev, "Failed to match regulators\n");
return ERR_PTR(-EINVAL);
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 4/9] regulator: max77686: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
2014-02-17 9:03 ` [PATCH 2/9] regulator: as3722: " Sachin Kamat
2014-02-17 9:03 ` [PATCH 3/9] regulator: da9063: " Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 2:01 ` Mark Brown
2014-02-17 9:03 ` [PATCH 5/9] regulator: max77693: " Sachin Kamat
` (5 subsequent siblings)
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/max77686.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/regulator/max77686.c b/drivers/regulator/max77686.c
index 0e725f6ed455..b411af00065d 100644
--- a/drivers/regulator/max77686.c
+++ b/drivers/regulator/max77686.c
@@ -412,6 +412,7 @@ static int max77686_pmic_dt_parse_pdata(struct platform_device *pdev,
if (!rdata) {
dev_err(&pdev->dev,
"could not allocate memory for regulator data\n");
+ of_node_put(regulators_np);
return -ENOMEM;
}
@@ -425,6 +426,7 @@ static int max77686_pmic_dt_parse_pdata(struct platform_device *pdev,
}
pdata->regulators = rdata;
+ of_node_put(regulators_np);
return 0;
}
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 5/9] regulator: max77693: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
` (2 preceding siblings ...)
2014-02-17 9:03 ` [PATCH 4/9] regulator: max77686: " Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 2:01 ` Mark Brown
2014-02-17 9:03 ` [PATCH 6/9] regulator: max8660: " Sachin Kamat
` (4 subsequent siblings)
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/max77693.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/regulator/max77693.c b/drivers/regulator/max77693.c
index 057d040e6feb..d6807fd7eadc 100644
--- a/drivers/regulator/max77693.c
+++ b/drivers/regulator/max77693.c
@@ -176,13 +176,16 @@ static int max77693_pmic_dt_parse_rdata(struct device *dev,
rmatch = devm_kzalloc(dev,
sizeof(*rmatch) * ARRAY_SIZE(regulators), GFP_KERNEL);
- if (!rmatch)
+ if (!rmatch) {
+ of_node_put(np);
return -ENOMEM;
+ }
for (i = 0; i < ARRAY_SIZE(regulators); i++)
rmatch[i].name = regulators[i].name;
matched = of_regulator_match(dev, np, rmatch, ARRAY_SIZE(regulators));
+ of_node_put(np);
if (matched <= 0)
return matched;
*rdata = devm_kzalloc(dev, sizeof(**rdata) * matched, GFP_KERNEL);
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 6/9] regulator: max8660: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
` (3 preceding siblings ...)
2014-02-17 9:03 ` [PATCH 5/9] regulator: max77693: " Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 2:02 ` Mark Brown
2014-02-17 9:03 ` [PATCH 7/9] regulator: max8998: " Sachin Kamat
` (3 subsequent siblings)
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/max8660.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/max8660.c b/drivers/regulator/max8660.c
index 70351abbd1e9..61ecd0892637 100644
--- a/drivers/regulator/max8660.c
+++ b/drivers/regulator/max8660.c
@@ -340,6 +340,7 @@ static int max8660_pdata_from_dt(struct device *dev,
rmatch[i].name = max8660_reg[i].name;
matched = of_regulator_match(dev, np, rmatch, ARRAY_SIZE(rmatch));
+ of_node_put(np);
if (matched <= 0)
return matched;
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 7/9] regulator: max8998: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
` (4 preceding siblings ...)
2014-02-17 9:03 ` [PATCH 6/9] regulator: max8660: " Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 2:02 ` Mark Brown
2014-02-17 9:03 ` [PATCH 8/9] regulator: tps6507x: " Sachin Kamat
` (2 subsequent siblings)
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/max8998.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/regulator/max8998.c b/drivers/regulator/max8998.c
index ae3f0656feb0..ce22d4f3f3eb 100644
--- a/drivers/regulator/max8998.c
+++ b/drivers/regulator/max8998.c
@@ -674,8 +674,10 @@ static int max8998_pmic_dt_parse_pdata(struct max8998_dev *iodev,
rdata = devm_kzalloc(iodev->dev, sizeof(*rdata) *
pdata->num_regulators, GFP_KERNEL);
- if (!rdata)
+ if (!rdata) {
+ of_node_put(regulators_np);
return -ENOMEM;
+ }
pdata->regulators = rdata;
for (i = 0; i < ARRAY_SIZE(regulators); ++i) {
@@ -692,6 +694,9 @@ static int max8998_pmic_dt_parse_pdata(struct max8998_dev *iodev,
}
pdata->num_regulators = rdata - pdata->regulators;
+ of_node_put(reg_np);
+ of_node_put(regulators_np);
+
ret = max8998_pmic_dt_parse_dvs_gpio(iodev, pdata, pmic_np);
if (ret)
return -EINVAL;
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 8/9] regulator: tps6507x: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
` (5 preceding siblings ...)
2014-02-17 9:03 ` [PATCH 7/9] regulator: max8998: " Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 2:04 ` Mark Brown
2014-02-17 9:03 ` [PATCH 9/9] regulator: tps65090: " Sachin Kamat
2014-02-18 1:57 ` [PATCH 1/9] regulator: act8865: " Mark Brown
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/tps6507x-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/tps6507x-regulator.c b/drivers/regulator/tps6507x-regulator.c
index 5a558dad27e3..e506e7f54253 100644
--- a/drivers/regulator/tps6507x-regulator.c
+++ b/drivers/regulator/tps6507x-regulator.c
@@ -395,6 +395,7 @@ static struct tps6507x_board *tps6507x_parse_dt_reg_data(
matches = tps6507x_matches;
ret = of_regulator_match(&pdev->dev, regulators, matches, count);
+ of_node_put(regulators);
if (ret < 0) {
dev_err(&pdev->dev, "Error parsing regulator init data: %d\n",
ret);
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 9/9] regulator: tps65090: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
` (6 preceding siblings ...)
2014-02-17 9:03 ` [PATCH 8/9] regulator: tps6507x: " Sachin Kamat
@ 2014-02-17 9:03 ` Sachin Kamat
2014-02-18 2:05 ` Mark Brown
2014-02-18 1:57 ` [PATCH 1/9] regulator: act8865: " Mark Brown
8 siblings, 1 reply; 18+ messages in thread
From: Sachin Kamat @ 2014-02-17 9:03 UTC (permalink / raw)
To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat
Add of_node_put to decrement the ref count.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
drivers/regulator/tps65090-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/tps65090-regulator.c b/drivers/regulator/tps65090-regulator.c
index 676f75548f00..5674f886c471 100644
--- a/drivers/regulator/tps65090-regulator.c
+++ b/drivers/regulator/tps65090-regulator.c
@@ -188,6 +188,7 @@ static struct tps65090_platform_data *tps65090_parse_dt_reg_data(
ret = of_regulator_match(&pdev->dev, regulators, tps65090_matches,
ARRAY_SIZE(tps65090_matches));
+ of_node_put(regulators);
if (ret < 0) {
dev_err(&pdev->dev,
"Error parsing regulator init data: %d\n", ret);
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/9] regulator: act8865: Add missing of_node_put
2014-02-17 9:03 [PATCH 1/9] regulator: act8865: Add missing of_node_put Sachin Kamat
` (7 preceding siblings ...)
2014-02-17 9:03 ` [PATCH 9/9] regulator: tps65090: " Sachin Kamat
@ 2014-02-18 1:57 ` Mark Brown
8 siblings, 0 replies; 18+ messages in thread
From: Mark Brown @ 2014-02-18 1:57 UTC (permalink / raw)
To: Sachin Kamat; +Cc: linux-kernel, lgirdwood
[-- Attachment #1: Type: text/plain, Size: 126 bytes --]
On Mon, Feb 17, 2014 at 02:33:29PM +0530, Sachin Kamat wrote:
> Add of_node_put to decrement the ref count.
Applied, thanks.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread