ACPI: battery drivers should call power_supply_changed()

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linux Kernel Mailing List
Date: Thursday, April 8, 2010 - 8:59 am

Gitweb:     http://git.kernel.org/linus/f79e1cec8c8aa64895fd7b595dc7b48157df0754
Commit:     f79e1cec8c8aa64895fd7b595dc7b48157df0754
Parent:     e363a755e8033e18f733fc0d1687039df8efade0
Author:     Alan Jenkins <alan-jenkins@tuffmail.co.uk>
AuthorDate: Tue Jun 30 14:36:16 2009 +0000
Committer:  Len Brown <len.brown@intel.com>
CommitDate: Sun Apr 4 19:53:23 2010 -0400

    ACPI: battery drivers should call power_supply_changed()
    
    Calling kobject_uevent() directly is a layering violation.  In
    particular, it means we'll miss updating the generic LED trigger.
    
    Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk>
    Acked-by: Alexey Starikovskiy <astarikovskiy@suse.de>
    Signed-off-by: Len Brown <len.brown@intel.com>
---
 drivers/acpi/battery.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/acpi/battery.c b/drivers/acpi/battery.c
index 52df994..db78b6e 100644
--- a/drivers/acpi/battery.c
+++ b/drivers/acpi/battery.c
@@ -879,7 +879,7 @@ static void acpi_battery_notify(struct acpi_device *device, u32 event)
 #ifdef CONFIG_ACPI_SYSFS_POWER
 	/* acpi_battery_update could remove power_supply object */
 	if (battery->bat.dev)
-		kobject_uevent(&battery->bat.dev->kobj, KOBJ_CHANGE);
+		power_supply_changed(&battery->bat);
 #endif
 }
 
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
ACPI: battery drivers should call power_supply_changed(), Linux Kernel Mailing ..., (Thu Apr 8, 8:59 am)