HAL battery backend, new thread!

Danny Kukawka danny.kukawka at web.de
Mon Aug 1 13:37:58 PDT 2005


On Monday 01 August 2005 19:20, David Zeuthen wrote:
> Looks good - I've committed this and the doc patch. Thanks a lot!

The patch is broken. HAL segfaults on a Laptop with 2 battery bays but only 
with one battery in function battery_refresh_poll() in line: 

	"if (strcmp (reporting_unit, "mWh") == 0) {"

The reason: battery_refresh_poll() polls the battery also if the battery bay 
is empty and no battery present. The attached patch should fix this.

I have currently no battery for bay BAT1, so I can't test what happens if you 
insert the battery while machine is running. But I test this tomorrow.

Cheers,

Danny

 acpi.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletion(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-fix-poll_empty_batteryBay.diff
Type: text/x-diff
Size: 687 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20050801/17cbadd8/hal-fix-poll_empty_batteryBay.bin
-------------- next part --------------
_______________________________________________
hal mailing list
hal at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/hal


More information about the Hal mailing list