From ec60ea5cd6c9f4b249de1580b8ebfa430055008c Mon Sep 17 00:00:00 2001 From: Paul Parsons Date: Thu, 20 Sep 2012 14:26:05 -0700 Subject: pda_power: Fix ac_draw usage before it being set When I reboot my iPAQ hx4700 in its cradle, the battery will not begin to charge even though the AC supply is connected. Charging will start only after the PDA power driver is tickled by some other power event, such as reseating the iPAQ in its cradle or connecting the USB cable. The problem lies in pda_power_probe(), where ac_draw is used by the call to update_charger() before being set by the call to regulator_get(). Moving the regulator_get() call to before the update_charger() call fixes the problem. Signed-off-by: Paul Parsons Cc: Philipp Zabel Signed-off-by: Anton Vorontsov --- drivers/power/pda_power.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'drivers/power/pda_power.c') diff --git a/drivers/power/pda_power.c b/drivers/power/pda_power.c index 7312f2651647..75536ebdb132 100644 --- a/drivers/power/pda_power.c +++ b/drivers/power/pda_power.c @@ -281,6 +281,14 @@ static int pda_power_probe(struct platform_device *pdev) goto init_failed; } + ac_draw = regulator_get(dev, "ac_draw"); + if (IS_ERR(ac_draw)) { + dev_dbg(dev, "couldn't get ac_draw regulator\n"); + ac_draw = NULL; + ret = PTR_ERR(ac_draw); + goto ac_draw_failed; + } + update_status(); update_charger(); @@ -309,13 +317,6 @@ static int pda_power_probe(struct platform_device *pdev) pda_psy_usb.num_supplicants = pdata->num_supplicants; } - ac_draw = regulator_get(dev, "ac_draw"); - if (IS_ERR(ac_draw)) { - dev_dbg(dev, "couldn't get ac_draw regulator\n"); - ac_draw = NULL; - ret = PTR_ERR(ac_draw); - } - #ifdef CONFIG_USB_OTG_UTILS transceiver = usb_get_phy(USB_PHY_TYPE_USB2); if (!IS_ERR_OR_NULL(transceiver)) { @@ -415,6 +416,7 @@ ac_supply_failed: regulator_put(ac_draw); ac_draw = NULL; } +ac_draw_failed: if (pdata->exit) pdata->exit(dev); init_failed: -- cgit v1.2.3 From c10a002a2b4af557730e71591c529fbdf355433f Mon Sep 17 00:00:00 2001 From: Paul Parsons Date: Fri, 21 Sep 2012 16:49:45 -0700 Subject: pda_power: Remove ac_draw_failed goto and label A previous patch added the ac_draw_failed goto and label to pda_power_probe(). The goto would be invoked after a failed call to regulator_get(). However the way ac_draw is used - always after a check for NULL - suggests that a failed call to regulator_get() was not fatal. This patch removes the ac_draw_failed goto and label, partly reverting the previous patch. This patch also removes the assignment of an error code to ret after a failed call to regulator_get(), since the error code is now never used. Signed-off-by: Paul Parsons Cc: Philipp Zabel Signed-off-by: Anton Vorontsov --- drivers/power/pda_power.c | 3 --- 1 file changed, 3 deletions(-) (limited to 'drivers/power/pda_power.c') diff --git a/drivers/power/pda_power.c b/drivers/power/pda_power.c index 75536ebdb132..7df7c5facc10 100644 --- a/drivers/power/pda_power.c +++ b/drivers/power/pda_power.c @@ -285,8 +285,6 @@ static int pda_power_probe(struct platform_device *pdev) if (IS_ERR(ac_draw)) { dev_dbg(dev, "couldn't get ac_draw regulator\n"); ac_draw = NULL; - ret = PTR_ERR(ac_draw); - goto ac_draw_failed; } update_status(); @@ -416,7 +414,6 @@ ac_supply_failed: regulator_put(ac_draw); ac_draw = NULL; } -ac_draw_failed: if (pdata->exit) pdata->exit(dev); init_failed: -- cgit v1.2.3