[PATCH] fix hald-addon-dell-backlight AC detection
David Zeuthen
david at fubar.dk
Mon Mar 3 16:49:54 PST 2008
On Tue, 2008-03-04 at 01:47 +0100, Danny Kukawka wrote:
> Hi,
>
> this fixes fd.o bug #13669 and hald-addon-dell-backlight which uses a
> hardcoded udi to get AC adapter information. With the switch to sysfs
> power_supply devices for AC adapter the path is no longer valid. I changed
> the code to check all devices with ac_adapter capability and check then all
> AC adapter until a device has ac_adapter.present and then get the info about
> the adapter state from there.
>
> See attached patch (untested atm, test it tomorrow/later today).
Looks ok to me from a cursory look but please put the declarations at
the start of the function (there's a dbus_error_init() before new
variables). Thanks.
David
More information about the hal
mailing list