battery.technology, take 2

David Zeuthen david at fubar.dk
Thu Aug 3 07:55:59 PDT 2006


Hi,

On Sun, 2006-07-30 at 12:26 +0100, Richard Hughes wrote:
> Okay, attached is an updated patch that modifies battery.technology to
> one of 3 present values "lithium-ion", "lead-acid" or "unknown" -- but
> it will be trivial to add more types as they are discovered.

I like this.

> The patch adds two more files, util-extern.c and util-extern.h -- these
> are used so that we don't have to drag in all the deps for util.c in the
> probers and addons that only use libhal, and not the hal_* functions.

I don't like this. I think Danny had a patch for cleaning up a lot of
this mess so addons, probers etc. can use hald/util.[ch] but it's not in
yet. Danny, any idea when this is going to land? I think I already
ACK'ed that patch...

> API is compatible, as the key battery.technology is now the "sanitised"
> key, and battery.reporting.technology is the key that is the "reported"
> key for the hardware, similar to what we do with the charge level.

Sounds good.

    David



More information about the hal mailing list