[PATCH] fix hald-addon-dell-backlight AC detection

Martin Pitt martin at piware.de
Tue Mar 4 01:03:57 PST 2008


Hi Danny,

Danny Kukawka [2008-03-04  1:47 +0100]:
> 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).

Great, thank you! I verified it on my Latitude D430, works perfectly.

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)

My 5 today: #193521 (jockey), #151025 (gnome-mount), #195548 (jockey),
#194963 (jockey), #194052 (gnome-power-manager, hal)
Do 5 a day - every day! https://wiki.ubuntu.com/5-A-Day


More information about the hal mailing list