diff mbox

[15/15] power_supply: Don't iterate over devices to return -EPROBE_DEFER

Message ID e3ff2a09590e8a0e2d249d11ce3badc8ea3976e4.1409831636.git.viresh.kumar@linaro.org
State Accepted
Commit 7195c505c2e8e2dd372f7b1737f2a788e74889fb
Headers show

Commit Message

Viresh Kumar Sept. 4, 2014, 12:01 p.m. UTC
This piece of code was added so that we return -EPROBE_DEFER when no devices are
registered. But even if class_for_each_device() returns 0, we are going to
return -EPROBE_DEFER only.

And so this code isn't required at all. Remove it.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
 drivers/power/power_supply_core.c | 13 -------------
 1 file changed, 13 deletions(-)
diff mbox

Patch

diff --git a/drivers/power/power_supply_core.c b/drivers/power/power_supply_core.c
index 81177e2..6cb7fe5 100644
--- a/drivers/power/power_supply_core.c
+++ b/drivers/power/power_supply_core.c
@@ -179,19 +179,6 @@  static int  __power_supply_find_supply_from_node(struct device *dev,
 static int power_supply_find_supply_from_node(struct device_node *supply_node)
 {
 	int error;
-	struct device *dev;
-	struct class_dev_iter iter;
-
-	/*
-	 * Use iterator to see if any other device is registered.
-	 * This is required since class_for_each_device returns 0
-	 * if there are no devices registered.
-	 */
-	class_dev_iter_init(&iter, power_supply_class, NULL, NULL);
-	dev = class_dev_iter_next(&iter);
-
-	if (!dev)
-		return -EPROBE_DEFER;
 
 	/*
 	 * class_for_each_device() either returns its own errors or values