lkml.org 
[lkml]   [2008]   [May]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] power_supply: support CHARGE_NOW in OLPC battery
Anton Vorontsov wrote:
> On Thu, May 08, 2008 at 01:01:02PM -0400, Andres Salomon wrote:
>> Ok, CC'ing Richard as I should've done in the first place...
> [...]
>>>> + case POWER_SUPPLY_PROP_CHARGE_NOW:
>>>> + ret = olpc_ec_cmd(EC_BAT_ACR, NULL, 0, (void
>>>> *)&ec_word, 2);
>>>> + if (ret)
>>>> + return ret;
>>>> +
>>>> + val->intval = be16_to_cpu(ec_word);
>>> But you didn't convert it to the uAh, I think you should.
>>
>> We have logic in userspace that converts ACR to a useful number; for
>> our purposes, we'd just like to read the raw ACR values.
>
> No. The purpose of power supply class is to report values in common
> units. Not some random value that userspace then should guess how to
> compute.

I'm fine with that. I'll have to make my scripts do some detection on
what kernel is running and convert or not convert but thats not too hard.

I remember Dave asking me if I wanted it converted and I told him not to
bother and just to export the raw value. I would deal with the units.
At the time I was just trying to get some simple debugging info. Turns
out that ACR is really useful for debugging battery problems and it stuck.

A single ACR reading itself does not tell you anything usefull. Its a
signed 16 bit free running coulomb counter that counts up and down based
on how much juice goes into or out of the battery. So to extract any
meaningful info you need 2 readings separated by some instance in time.
The difference between the 2 tells you the relative energy you have
consumed or added.

The units for the ACR register in the DS2765 are:

6.25 uVh / Rsense
OLPC Rsense = .015 Ohms

The integer math conversion for ACR to mAh that I use in my scripts is:

(ACR2 - ACR1) * 625 / 1500

>> Perhaps CHARGE_NOW is the wrong thing to be using, if we can't get
>> uAh and CHARGE_* requires that unit.
>
> I don't think that converting X * uAh or uAh / Y to uAh is impossible.
> I'd suggest you to convert the ACR value to uAh using fractions, see
> ds2760_battery.c. Even if the final value losing precision a bit, it is
> still useful to do CHARGE_NOW.

Is CHARGE_NOW supposed to be an absolute indication of the level of
charge in the battery? If so then thats not ACR. That would be SOC% *
battery capacity. The EC uses the relative motion of the ACR to update
the SOC%.

--
Richard Smith <richard@laptop.org>
One Laptop Per Child


\
 
 \ /
  Last update: 2008-05-08 20:57    [W:0.086 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site