Battery status needs to read /sys instead of /proc

Brought to you by: captnmark

#807 Battery status needs to read /sys instead of /proc

Milestone: taskbar

Status: closed-fixed

Owner: Marko Macek

Labels: icewm-1.2 (683)

Priority: 7

Updated: 2008-08-04

Created: 2008-07-15

Creator: Anonymous

Private: No

ACPI_PROCFS_POWER is depreciated. On Debian Lenny the kernel 2.6.25 already don't provide /proc/acpi/battery

It seems this option varnishes on kernel 2.6.26.

I think the source of the battery status should be changed to /sys/class/power_supply

Discussion

Santiago Garcia Mantinan - 2008-07-29

Logged In: YES
user_id=656905
Originator: NO

I have created a patch that implements this, I have posted it on the bug I had opened on Debian which is available at:

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=492403

The patch is available at http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=10;filename=sysfs_battery.dpatch;att=1;bug=492403 as a dpatch, you can use it as a normal diff patch if you remove the dpatch header.

This is against the latest stable version of icewm.

I have tested it in different laptops and seems to work ok on 2.6.25.

Regards...

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Marko Macek - 2008-08-01

labels: --> icewm-1.2

milestone: --> taskbar

priority: 5 --> 7

assigned_to: nobody --> captnmark
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Marko Macek - 2008-08-04

status: open --> open-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Marko Macek - 2008-08-04

Logged In: YES
user_id=1814
Originator: NO

Applied to 1.2 branch, with changes:
- check for sysfs first, and use it
- fixed handling of AC adapter
- fixed display of status window

Please test.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Marko Macek - 2008-08-04

status: open-fixed --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.