diff options
| author | Alan Jenkins <alan-jenkins@tuffmail.co.uk> | 2009-06-30 14:36:16 +0000 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2010-04-04 19:53:23 -0400 |
| commit | f79e1cec8c8aa64895fd7b595dc7b48157df0754 (patch) | |
| tree | fe54a474018cd1561dee79049a4fa22d2a62a9c4 /drivers/acpi | |
| parent | e363a755e8033e18f733fc0d1687039df8efade0 (diff) | |
| download | op-kernel-dev-f79e1cec8c8aa64895fd7b595dc7b48157df0754.zip op-kernel-dev-f79e1cec8c8aa64895fd7b595dc7b48157df0754.tar.gz | |
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>
Diffstat (limited to 'drivers/acpi')
| -rw-r--r-- | 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 } |
