Re: HAL fix for multiple battaries

2008-02-21 Thread Matthew Garrett
On Wed, Feb 20, 2008 at 09:19:04AM -0800, Ted Gould wrote:

 Yes, but that HAL patch ignores all battery entries in /proc if it finds
 one in /sys.  Which seems rather risky to me... but I was curious if
 that seems logical to those who know the kernel better.

Yes, the only way a battery can end up in /proc/acpi is if it's 
supported by the acpi battery driver, which has now been ported to 
sysfs.

-- 
Matthew Garrett | [EMAIL PROTECTED]

-- 
Ubuntu-devel-discuss mailing list
Ubuntu-devel-discuss@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel-discuss


HAL fix for multiple battaries

2008-02-19 Thread Ted Gould
So, here is the bug:

https://bugs.launchpad.net/ubuntu/+source/gnome-power-manager/+bug/177570

Basically with the .24 kernel there are sysfs and proc interfaces for
many batteries making it so that HAL reports two, and thus GPM reports
two.  There is a link to a patch in upstream HAL that fixes the multiple
listing problem.  But, what it does is not list ANY proc batteries if a
sysfs one is found.  Is that correct?  Are all battery interfaces
migrated to sysfs in .24?

TIA,
Ted



-- 
Ubuntu-devel-discuss mailing list
Ubuntu-devel-discuss@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel-discuss


Re: HAL fix for multiple battaries

2008-02-19 Thread Martin Pitt
Hi Ted,

Ted Gould [2008-02-19 14:31 -0800]:
 So, here is the bug:
 
 https://bugs.launchpad.net/ubuntu/+source/gnome-power-manager/+bug/177570
 
 Basically with the .24 kernel there are sysfs and proc interfaces for
 many batteries making it so that HAL reports two, and thus GPM reports
 two.  There is a link to a patch in upstream HAL that fixes the multiple
 listing problem.  But, what it does is not list ANY proc batteries if a
 sysfs one is found.  Is that correct?  Are all battery interfaces
 migrated to sysfs in .24?

Fixing this is on my TODO list, there is an existing hal patch for
this.

Martin
-- 
Martin Pitthttp://www.piware.de
Ubuntu Developer  http://www.ubuntu.com
Debian Developer  http://www.debian.org

-- 
Ubuntu-devel-discuss mailing list
Ubuntu-devel-discuss@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel-discuss