Re: [PATCH] power_supply: support CHARGE_NOW in OLPC battery

["Richard A. Smith" <richard@laptop.org>]

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