Re: acpi battery rework patch

On Sun, 24 Jul 2005 21:58:18 +0200
Marcin Jessa <lists_at_yazzy.org> wrote:

> On Sun, 24 Jul 2005 12:47:13 +0900
> Eric Kjeldergaard <kjelderg_at_gmail.com> wrote:
> 
> > On 7/24/05, Nate Lawson <nate_at_root.org> wrote:
> > > Marcin Jessa wrote:
> > > > On Sat, 23 Jul 2005 19:00:01 +0900
> > > > Eric Kjeldergaard <kjelderg_at_gmail.com> wrote:
> > > >>>Please test to be sure your battery status works as usual, along with 
> > > >>>any apps.  Since most apps (xbatt, gnome, etc.) use the apm compat 
> > > >>>layer, they should work as before with no recompilation needed.
> > > > 
> > > > 
> > > >>-CURRENT as of 23/07/2005, does not compile.
> > > > 
> > > > 
> > > > Prolly because it's for 6.0, not 7.0
> > > 
> > > Sorry, the problem was I left a file out of the diff.  I just committed 
> > > the patch so please just cvsup and test from 7-CURRENT.
> > > 
> > > Thanks,
> > > -- 
> > > Nate
> > > 
> > Excellent.  compiled and appears working.  It does however have
> > different values than the old system for some of the sysctl variables.
> >  I get
> > 
> > hw.acpi.battery.life: 98
> > hw.acpi.battery.time: 0
> > hw.acpi.battery.state: 0
> > hw.acpi.battery.units: 1
> > hw.acpi.battery.info_expire: 5
> > 
> > when plugged in.  But used to get time as -1 when plugged in and I
> > think state may have been 1 when plugged in before (less sure about
> > that than the time).  Thanks for the enormous amount of work your
> > efforts are very much appreciated,
> > 
> > Eric
> 
> Ditto, thanks a lot. It works well here too. I can now actually correctly see whether the lapper is connected to power or runs on battery.
> Before the state change could not be detected.
> What lacks is remining time which AFAIR worked before, now showing : hw.acpi.battery.time: -1
> I don't miss it though since showing remining percentage is more than enough.
> Thanks again for great work!
 
It seems like I was wrong. The time values showed up after a short while of running on battery.