Browse Source

regulator: 88pm8607: Use of_get_child_by_name

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>
Signed-off-by: Mark Brown <broonie@linaro.org>
Sachin Kamat 11 years ago
parent
commit
ca1e3f33d9
1 changed files with 1 additions and 1 deletions
  1. 1 1
      drivers/regulator/88pm8607.c

+ 1 - 1
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);
 	nproot = of_node_get(pdev->dev.parent->of_node);
 	if (!nproot)
 	if (!nproot)
 		return -ENODEV;
 		return -ENODEV;
-	nproot = of_find_node_by_name(nproot, "regulators");
+	nproot = of_get_child_by_name(nproot, "regulators");
 	if (!nproot) {
 	if (!nproot) {
 		dev_err(&pdev->dev, "failed to find regulators node\n");
 		dev_err(&pdev->dev, "failed to find regulators node\n");
 		return -ENODEV;
 		return -ENODEV;