All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 01/16] regulator: max77686: Use of_get_child_by_name
@ 2014-02-14 11:49 Sachin Kamat
  2014-02-14 11:49 ` [PATCH 02/16] regulator: max77693: " Sachin Kamat
                   ` (15 more replies)
  0 siblings, 16 replies; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
Series compile tested. Some max* regulators tested on h/w.
---
 drivers/regulator/max77686.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/max77686.c b/drivers/regulator/max77686.c
index ae001ccf26f4..0e725f6ed455 100644
--- a/drivers/regulator/max77686.c
+++ b/drivers/regulator/max77686.c
@@ -400,7 +400,7 @@ static int max77686_pmic_dt_parse_pdata(struct platform_device *pdev,
 	unsigned int i;
 
 	pmic_np = iodev->dev->of_node;
-	regulators_np = of_find_node_by_name(pmic_np, "voltage-regulators");
+	regulators_np = of_get_child_by_name(pmic_np, "voltage-regulators");
 	if (!regulators_np) {
 		dev_err(&pdev->dev, "could not find regulators sub-node\n");
 		return -EINVAL;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 02/16] regulator: max77693: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:08   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 03/16] regulator: max8660: " Sachin Kamat
                   ` (14 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/max77693.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/max77693.c b/drivers/regulator/max77693.c
index 5fb899f461d0..057d040e6feb 100644
--- a/drivers/regulator/max77693.c
+++ b/drivers/regulator/max77693.c
@@ -170,7 +170,7 @@ static int max77693_pmic_dt_parse_rdata(struct device *dev,
 	struct max77693_regulator_data *tmp;
 	int i, matched = 0;
 
-	np = of_find_node_by_name(dev->parent->of_node, "regulators");
+	np = of_get_child_by_name(dev->parent->of_node, "regulators");
 	if (!np)
 		return -EINVAL;
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 03/16] regulator: max8660: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
  2014-02-14 11:49 ` [PATCH 02/16] regulator: max77693: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:09   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 04/16] regulator: max8907: " Sachin Kamat
                   ` (13 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/max8660.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/max8660.c b/drivers/regulator/max8660.c
index 8d94d3d7f97f..70351abbd1e9 100644
--- a/drivers/regulator/max8660.c
+++ b/drivers/regulator/max8660.c
@@ -330,7 +330,7 @@ static int max8660_pdata_from_dt(struct device *dev,
 	struct max8660_subdev_data *sub;
 	struct of_regulator_match rmatch[ARRAY_SIZE(max8660_reg)];
 
-	np = of_find_node_by_name(dev->of_node, "regulators");
+	np = of_get_child_by_name(dev->of_node, "regulators");
 	if (!np) {
 		dev_err(dev, "missing 'regulators' subnode in DT\n");
 		return -EINVAL;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 04/16] regulator: max8907: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
  2014-02-14 11:49 ` [PATCH 02/16] regulator: max77693: " Sachin Kamat
  2014-02-14 11:49 ` [PATCH 03/16] regulator: max8660: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:10   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 05/16] regulator: max8925: " Sachin Kamat
                   ` (12 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/max8907-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/max8907-regulator.c b/drivers/regulator/max8907-regulator.c
index 0c5fe6c6ac26..afda8c6af721 100644
--- a/drivers/regulator/max8907-regulator.c
+++ b/drivers/regulator/max8907-regulator.c
@@ -231,7 +231,7 @@ static int max8907_regulator_parse_dt(struct platform_device *pdev)
 	if (!np)
 		return 0;
 
-	regulators = of_find_node_by_name(np, "regulators");
+	regulators = of_get_child_by_name(np, "regulators");
 	if (!regulators) {
 		dev_err(&pdev->dev, "regulators node not found\n");
 		return -EINVAL;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 05/16] regulator: max8925: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (2 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 04/16] regulator: max8907: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:11   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 06/16] regulator: max8997: " Sachin Kamat
                   ` (11 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/max8925-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/max8925-regulator.c b/drivers/regulator/max8925-regulator.c
index 759510789e71..5b939894bfc4 100644
--- a/drivers/regulator/max8925-regulator.c
+++ b/drivers/regulator/max8925-regulator.c
@@ -254,7 +254,7 @@ static int max8925_regulator_dt_init(struct platform_device *pdev,
 	nproot = of_node_get(pdev->dev.parent->of_node);
 	if (!nproot)
 		return -ENODEV;
-	np = of_find_node_by_name(nproot, "regulators");
+	np = of_get_child_by_name(nproot, "regulators");
 	if (!np) {
 		dev_err(&pdev->dev, "failed to find regulators node\n");
 		return -ENODEV;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 06/16] regulator: max8997: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (3 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 05/16] regulator: max8925: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:11   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 07/16] regulator: 88pm8607: " Sachin Kamat
                   ` (10 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/max8997.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/max8997.c b/drivers/regulator/max8997.c
index 2d618fc9c1af..10108f2941e3 100644
--- a/drivers/regulator/max8997.c
+++ b/drivers/regulator/max8997.c
@@ -924,7 +924,7 @@ static int max8997_pmic_dt_parse_pdata(struct platform_device *pdev,
 		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) {
 		dev_err(&pdev->dev, "could not find regulators sub-node\n");
 		return -EINVAL;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 07/16] regulator: 88pm8607: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (4 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 06/16] regulator: max8997: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:14   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 08/16] regulator: act8865: " Sachin Kamat
                   ` (9 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/88pm8607.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/88pm8607.c b/drivers/regulator/88pm8607.c
index f704d83c93c4..fa99bfccaa6b 100644
--- a/drivers/regulator/88pm8607.c
+++ b/drivers/regulator/88pm8607.c
@@ -322,7 +322,7 @@ static int pm8607_regulator_dt_init(struct platform_device *pdev,
 	nproot = of_node_get(pdev->dev.parent->of_node);
 	if (!nproot)
 		return -ENODEV;
-	nproot = of_find_node_by_name(nproot, "regulators");
+	nproot = of_get_child_by_name(nproot, "regulators");
 	if (!nproot) {
 		dev_err(&pdev->dev, "failed to find regulators node\n");
 		return -ENODEV;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 08/16] regulator: act8865: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (5 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 07/16] regulator: 88pm8607: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:15   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 09/16] regulator: as3711: " Sachin Kamat
                   ` (8 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/act8865-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/act8865-regulator.c b/drivers/regulator/act8865-regulator.c
index 084cc0819a52..c9f98b061307 100644
--- a/drivers/regulator/act8865-regulator.c
+++ b/drivers/regulator/act8865-regulator.c
@@ -213,7 +213,7 @@ static int act8865_pdata_from_dt(struct device *dev,
 	struct device_node *np;
 	struct act8865_regulator_data *regulator;
 
-	np = of_find_node_by_name(dev->of_node, "regulators");
+	np = of_get_child_by_name(dev->of_node, "regulators");
 	if (!np) {
 		dev_err(dev, "missing 'regulators' subnode in DT\n");
 		return -EINVAL;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 09/16] regulator: as3711: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (6 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 08/16] regulator: act8865: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:15   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 10/16] regulator: da9052: " Sachin Kamat
                   ` (7 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/as3711-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/as3711-regulator.c b/drivers/regulator/as3711-regulator.c
index 67fd548dcdba..856c55f3a832 100644
--- a/drivers/regulator/as3711-regulator.c
+++ b/drivers/regulator/as3711-regulator.c
@@ -191,7 +191,7 @@ static int as3711_regulator_parse_dt(struct device *dev,
 {
 	struct as3711_regulator_pdata *pdata = dev_get_platdata(dev);
 	struct device_node *regulators =
-		of_find_node_by_name(dev->parent->of_node, "regulators");
+		of_get_child_by_name(dev->parent->of_node, "regulators");
 	struct of_regulator_match *match;
 	int ret, i;
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 10/16] regulator: da9052: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (7 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 09/16] regulator: as3711: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:16   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 11/16] regulator: da9055: " Sachin Kamat
                   ` (6 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/da9052-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/da9052-regulator.c b/drivers/regulator/da9052-regulator.c
index 3adeaeffc485..889c7c9b9f15 100644
--- a/drivers/regulator/da9052-regulator.c
+++ b/drivers/regulator/da9052-regulator.c
@@ -401,7 +401,7 @@ static int da9052_regulator_probe(struct platform_device *pdev)
 		if (!nproot)
 			return -ENODEV;
 
-		nproot = of_find_node_by_name(nproot, "regulators");
+		nproot = of_get_child_by_name(nproot, "regulators");
 		if (!nproot)
 			return -ENODEV;
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 11/16] regulator: da9055: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (8 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 10/16] regulator: da9052: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:18   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 12/16] regulator: da9063: " Sachin Kamat
                   ` (5 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/da9055-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/da9055-regulator.c b/drivers/regulator/da9055-regulator.c
index cf7404026588..126e7c6d2c12 100644
--- a/drivers/regulator/da9055-regulator.c
+++ b/drivers/regulator/da9055-regulator.c
@@ -559,7 +559,7 @@ static int da9055_regulator_dt_init(struct platform_device *pdev,
 	if (!nproot)
 		return -ENODEV;
 
-	np = of_find_node_by_name(nproot, "regulators");
+	np = of_get_child_by_name(nproot, "regulators");
 	if (!np)
 		return -ENODEV;
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 12/16] regulator: da9063: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (9 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 11/16] regulator: da9055: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:18   ` Mark Brown
  2014-02-14 11:49 ` [PATCH 13/16] regulator: mc13xxx: " Sachin Kamat
                   ` (4 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/da9063-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/da9063-regulator.c b/drivers/regulator/da9063-regulator.c
index 91e99a2c8dc1..bfd7f2f795a6 100644
--- a/drivers/regulator/da9063-regulator.c
+++ b/drivers/regulator/da9063-regulator.c
@@ -666,7 +666,7 @@ static struct da9063_regulators_pdata *da9063_parse_regulators_dt(
 	struct device_node *node;
 	int i, n, num;
 
-	node = of_find_node_by_name(pdev->dev.parent->of_node, "regulators");
+	node = of_get_child_by_name(pdev->dev.parent->of_node, "regulators");
 	if (!node) {
 		dev_err(&pdev->dev, "Regulators device node not found\n");
 		return ERR_PTR(-ENODEV);
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 13/16] regulator: mc13xxx: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (10 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 12/16] regulator: da9063: " Sachin Kamat
@ 2014-02-14 11:49 ` Sachin Kamat
  2014-02-14 21:19   ` Mark Brown
  2014-02-14 11:50 ` [PATCH 14/16] regulator: pfuze100: " Sachin Kamat
                   ` (3 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:49 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/mc13xxx-regulator-core.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/regulator/mc13xxx-regulator-core.c b/drivers/regulator/mc13xxx-regulator-core.c
index da4859282302..4498a3f0733d 100644
--- a/drivers/regulator/mc13xxx-regulator-core.c
+++ b/drivers/regulator/mc13xxx-regulator-core.c
@@ -168,7 +168,7 @@ int mc13xxx_get_num_regulators_dt(struct platform_device *pdev)
 	int num;
 
 	of_node_get(pdev->dev.parent->of_node);
-	parent = of_find_node_by_name(pdev->dev.parent->of_node, "regulators");
+	parent = of_get_child_by_name(pdev->dev.parent->of_node, "regulators");
 	if (!parent)
 		return -ENODEV;
 
@@ -188,7 +188,7 @@ struct mc13xxx_regulator_init_data *mc13xxx_parse_regulators_dt(
 	int i, parsed = 0;
 
 	of_node_get(pdev->dev.parent->of_node);
-	parent = of_find_node_by_name(pdev->dev.parent->of_node, "regulators");
+	parent = of_get_child_by_name(pdev->dev.parent->of_node, "regulators");
 	if (!parent)
 		return NULL;
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 14/16] regulator: pfuze100: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (11 preceding siblings ...)
  2014-02-14 11:49 ` [PATCH 13/16] regulator: mc13xxx: " Sachin Kamat
@ 2014-02-14 11:50 ` Sachin Kamat
  2014-02-14 21:20   ` Mark Brown
  2014-02-14 11:50 ` [PATCH 15/16] regulator: tps6507x: " Sachin Kamat
                   ` (2 subsequent siblings)
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:50 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/pfuze100-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/pfuze100-regulator.c b/drivers/regulator/pfuze100-regulator.c
index b699d4e7acc5..c1cb20204a94 100644
--- a/drivers/regulator/pfuze100-regulator.c
+++ b/drivers/regulator/pfuze100-regulator.c
@@ -254,7 +254,7 @@ static int pfuze_parse_regulators_dt(struct pfuze_chip *chip)
 	if (!np)
 		return 0;
 
-	parent = of_find_node_by_name(np, "regulators");
+	parent = of_get_child_by_name(np, "regulators");
 	if (!parent) {
 		dev_err(dev, "regulators node not found\n");
 		return -EINVAL;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 15/16] regulator: tps6507x: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (12 preceding siblings ...)
  2014-02-14 11:50 ` [PATCH 14/16] regulator: pfuze100: " Sachin Kamat
@ 2014-02-14 11:50 ` Sachin Kamat
  2014-02-14 21:21   ` Mark Brown
  2014-02-14 11:50 ` [PATCH 16/16] regulator: tps65217: " Sachin Kamat
  2014-02-14 21:08 ` [PATCH 01/16] regulator: max77686: " Mark Brown
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:50 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/tps6507x-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/tps6507x-regulator.c b/drivers/regulator/tps6507x-regulator.c
index 862cc81f822f..5a558dad27e3 100644
--- a/drivers/regulator/tps6507x-regulator.c
+++ b/drivers/regulator/tps6507x-regulator.c
@@ -385,7 +385,7 @@ static struct tps6507x_board *tps6507x_parse_dt_reg_data(
 		return NULL;
 	}
 
-	regulators = of_find_node_by_name(np, "regulators");
+	regulators = of_get_child_by_name(np, "regulators");
 	if (!regulators) {
 		dev_err(&pdev->dev, "regulator node not found\n");
 		return NULL;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 16/16] regulator: tps65217: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (13 preceding siblings ...)
  2014-02-14 11:50 ` [PATCH 15/16] regulator: tps6507x: " Sachin Kamat
@ 2014-02-14 11:50 ` Sachin Kamat
  2014-02-14 21:22   ` Mark Brown
  2014-02-14 21:08 ` [PATCH 01/16] regulator: max77686: " Mark Brown
  15 siblings, 1 reply; 32+ messages in thread
From: Sachin Kamat @ 2014-02-14 11:50 UTC (permalink / raw)
  To: linux-kernel; +Cc: broonie, lgirdwood, sachin.kamat

of_find_node_by_name walks the allnodes list, and can thus walk
outside of the parent node. Use of_get_child_by_name instead.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/regulator/tps65217-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/tps65217-regulator.c b/drivers/regulator/tps65217-regulator.c
index 9ea1bf26bd13..fd441f2738f5 100644
--- a/drivers/regulator/tps65217-regulator.c
+++ b/drivers/regulator/tps65217-regulator.c
@@ -187,7 +187,7 @@ static struct tps65217_board *tps65217_parse_dt(struct platform_device *pdev)
 	struct device_node *regs;
 	int i, count;
 
-	regs = of_find_node_by_name(node, "regulators");
+	regs = of_get_child_by_name(node, "regulators");
 	if (!regs)
 		return NULL;
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 32+ messages in thread

* Re: [PATCH 01/16] regulator: max77686: Use of_get_child_by_name
  2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
                   ` (14 preceding siblings ...)
  2014-02-14 11:50 ` [PATCH 16/16] regulator: tps65217: " Sachin Kamat
@ 2014-02-14 21:08 ` Mark Brown
  15 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:08 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 253 bytes --]

On Fri, Feb 14, 2014 at 05:19:47PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks - and thanks for doing this cleanup series.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 02/16] regulator: max77693: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 02/16] regulator: max77693: " Sachin Kamat
@ 2014-02-14 21:08   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:08 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:48PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 03/16] regulator: max8660: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 03/16] regulator: max8660: " Sachin Kamat
@ 2014-02-14 21:09   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:09 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:49PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 04/16] regulator: max8907: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 04/16] regulator: max8907: " Sachin Kamat
@ 2014-02-14 21:10   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:10 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:50PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 05/16] regulator: max8925: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 05/16] regulator: max8925: " Sachin Kamat
@ 2014-02-14 21:11   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:11 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:51PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 06/16] regulator: max8997: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 06/16] regulator: max8997: " Sachin Kamat
@ 2014-02-14 21:11   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:11 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:52PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 07/16] regulator: 88pm8607: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 07/16] regulator: 88pm8607: " Sachin Kamat
@ 2014-02-14 21:14   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:14 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:53PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 08/16] regulator: act8865: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 08/16] regulator: act8865: " Sachin Kamat
@ 2014-02-14 21:15   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:15 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:54PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 09/16] regulator: as3711: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 09/16] regulator: as3711: " Sachin Kamat
@ 2014-02-14 21:15   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:15 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:55PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 10/16] regulator: da9052: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 10/16] regulator: da9052: " Sachin Kamat
@ 2014-02-14 21:16   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:16 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:56PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 11/16] regulator: da9055: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 11/16] regulator: da9055: " Sachin Kamat
@ 2014-02-14 21:18   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:18 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:57PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 12/16] regulator: da9063: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 12/16] regulator: da9063: " Sachin Kamat
@ 2014-02-14 21:18   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:18 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:58PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 13/16] regulator: mc13xxx: Use of_get_child_by_name
  2014-02-14 11:49 ` [PATCH 13/16] regulator: mc13xxx: " Sachin Kamat
@ 2014-02-14 21:19   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:19 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:19:59PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 14/16] regulator: pfuze100: Use of_get_child_by_name
  2014-02-14 11:50 ` [PATCH 14/16] regulator: pfuze100: " Sachin Kamat
@ 2014-02-14 21:20   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:20 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:20:00PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 15/16] regulator: tps6507x: Use of_get_child_by_name
  2014-02-14 11:50 ` [PATCH 15/16] regulator: tps6507x: " Sachin Kamat
@ 2014-02-14 21:21   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:21 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:20:01PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 16/16] regulator: tps65217: Use of_get_child_by_name
  2014-02-14 11:50 ` [PATCH 16/16] regulator: tps65217: " Sachin Kamat
@ 2014-02-14 21:22   ` Mark Brown
  0 siblings, 0 replies; 32+ messages in thread
From: Mark Brown @ 2014-02-14 21:22 UTC (permalink / raw)
  To: Sachin Kamat; +Cc: linux-kernel, lgirdwood

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

On Fri, Feb 14, 2014 at 05:20:02PM +0530, Sachin Kamat wrote:
> of_find_node_by_name walks the allnodes list, and can thus walk
> outside of the parent node. Use of_get_child_by_name instead.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

end of thread, other threads:[~2014-02-15 10:13 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-14 11:49 [PATCH 01/16] regulator: max77686: Use of_get_child_by_name Sachin Kamat
2014-02-14 11:49 ` [PATCH 02/16] regulator: max77693: " Sachin Kamat
2014-02-14 21:08   ` Mark Brown
2014-02-14 11:49 ` [PATCH 03/16] regulator: max8660: " Sachin Kamat
2014-02-14 21:09   ` Mark Brown
2014-02-14 11:49 ` [PATCH 04/16] regulator: max8907: " Sachin Kamat
2014-02-14 21:10   ` Mark Brown
2014-02-14 11:49 ` [PATCH 05/16] regulator: max8925: " Sachin Kamat
2014-02-14 21:11   ` Mark Brown
2014-02-14 11:49 ` [PATCH 06/16] regulator: max8997: " Sachin Kamat
2014-02-14 21:11   ` Mark Brown
2014-02-14 11:49 ` [PATCH 07/16] regulator: 88pm8607: " Sachin Kamat
2014-02-14 21:14   ` Mark Brown
2014-02-14 11:49 ` [PATCH 08/16] regulator: act8865: " Sachin Kamat
2014-02-14 21:15   ` Mark Brown
2014-02-14 11:49 ` [PATCH 09/16] regulator: as3711: " Sachin Kamat
2014-02-14 21:15   ` Mark Brown
2014-02-14 11:49 ` [PATCH 10/16] regulator: da9052: " Sachin Kamat
2014-02-14 21:16   ` Mark Brown
2014-02-14 11:49 ` [PATCH 11/16] regulator: da9055: " Sachin Kamat
2014-02-14 21:18   ` Mark Brown
2014-02-14 11:49 ` [PATCH 12/16] regulator: da9063: " Sachin Kamat
2014-02-14 21:18   ` Mark Brown
2014-02-14 11:49 ` [PATCH 13/16] regulator: mc13xxx: " Sachin Kamat
2014-02-14 21:19   ` Mark Brown
2014-02-14 11:50 ` [PATCH 14/16] regulator: pfuze100: " Sachin Kamat
2014-02-14 21:20   ` Mark Brown
2014-02-14 11:50 ` [PATCH 15/16] regulator: tps6507x: " Sachin Kamat
2014-02-14 21:21   ` Mark Brown
2014-02-14 11:50 ` [PATCH 16/16] regulator: tps65217: " Sachin Kamat
2014-02-14 21:22   ` Mark Brown
2014-02-14 21:08 ` [PATCH 01/16] regulator: max77686: " Mark Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.