On Sat, May 23, 2009 at 5:54 PM, Michael Trimarchi
<[email protected]> wrote:
> Hi
>
> This is an update of the battery patch.
>
> Can you add the signed-off-by of the previus two people.

Michael, the previous patch was already applied. I sent the attached patch.
From: Michael Trimarchi <[email protected]>
Date: Sat, 23 May 2009 17:54:00 -0300
Subject: [PATCH] Improve battery status notification

This patch adds a call to cancel_delayed_work before a call
to schedule_delayed_work.

Signed-off-by: Michael Trimarchi <[email protected]>
Signed-off-by: Daniel Willmann <[email protected]>

diff --git a/drivers/power/bq27000_battery.c b/drivers/power/bq27000_battery.c
index 9f0f1a9..7e8b3f8 100644
--- a/drivers/power/bq27000_battery.c
+++ b/drivers/power/bq27000_battery.c
@@ -170,9 +170,11 @@ static int hdq_read16(struct bq27000_device_info *di, int address)
 
 static void bq27000_battery_external_power_changed(struct power_supply *psy)
 {
-	struct bq27000_device_info *di = container_of(psy, struct bq27000_device_info, bat);
+	struct bq27000_device_info *di =
+		container_of(psy, struct bq27000_device_info, bat);
 
 	dev_dbg(di->dev, "%s\n", __FUNCTION__);
+	cancel_delayed_work(&di->work);
 	schedule_delayed_work(&di->work, 0);
 }
 

Reply via email to