[patch] Reworking HAL's battery handling, was:Trivial error message fix

[Richard Hughes]

On Mon, 2006-11-27 at 12:55 -0500, David Zeuthen wrote:
> On Mon, 2006-11-27 at 16:38 +0000, Richard Hughes wrote:
> > On Mon, 2006-11-06 at 18:18 -0500, Stu Hood wrote:
> > > Quick nitpicks:
> > >       * @330 - Slight indentation problem
> > >       * @364 - TRUE in boolean statement
> > >       * @400 - Check against some ratio of bat->last_full instead: For
> > >         instance, (rate > 0.25 * last_full)? (huge/server batteries
> > >         could have rate >50000)
> > >       * @923 - Broken comment
> > > Otherwise, brilliant!
> > 
> > David, what do you think? I've fixed these little nits, and really want
> > to get this in well before the next HAL release.
> 
> Sure, I thought we were waiting for Danny to OK this? You and Danny are
> probably better at reviewing this than me... 

I think Danny is somewhat busy with suse stuff.

> Also, looking at the patch again.. since the new code can remove
> properties.. is this documented in the spec? Do we do this already? Does
> g-p-m and kpowersave cope with that? 

It's what we've done for ages with acpi.c - when the battery is removed
then the properties are removed along with it - or do I misunderstand
the question?

Thanks.

Richard.