Ticket #255 (closed defect: fixed)

Opened 6 years ago

Last modified 5 years ago

battery voltage scale is not correct

Reported by: laforge@… Owned by: michael@…
Priority: high Milestone:
Component: kernel Version: current svn head
Severity: normal Keywords:
Cc: sean_chiang@…, buglog@…, speedevil@… Blocked By:
Blocking: Estimated Completion (week):
HasPatchForReview: PatchReviewResult:
Reproducible:

Description

Since we don't have a coulomb-counting battery gas gauge, we cannot give proper
battery metering.

Our current battery percentage reading is based on battery voltage, and uses a
linear scale between 2.8 and 4.2V (see drivers/i2c/pcf50606.c:battvolt_scale()).

This scaling is obviously wrong and just a 'proof of concept' scale.

Somebody needs to do proper charge/discharge measurements and come up with a
non-linear algorithm expressing estimated battery life based on present battery
terminal voltage.

This needs a lot of experimentation/measurement, as well as a couple of
calculations. Please provide me either with a patch against battvolt_scale(),
or give me a mathematical formula expressing battery lifetime percentage as
function of terminal voltage.

Attachments

discharge.txt.gz (24.4 KB) - added by philipp.zabel@… 6 years ago.
battemp/battvolt measurement during discharge
charge1.txt.gz (18.9 KB) - added by philipp.zabel@… 6 years ago.
charge1.txt.gz, charge log

Change History

comment:1 Changed 6 years ago by mail@…

  • blocked set to 212

Changed 6 years ago by philipp.zabel@…

battemp/battvolt measurement during discharge

comment:2 Changed 6 years ago by philipp.zabel@…

  • attachments.mimetype changed from text/plain to application/octet-stream

Changed 6 years ago by philipp.zabel@…

charge1.txt.gz, charge log

comment:3 Changed 6 years ago by tony_tu@…

  • Status changed from new to assigned

I will transfer to Tim later about this bug, and see if we could change
battery on next version and give right scale

comment:4 Changed 6 years ago by alphaone@…

  • blocked set to 193

comment:5 Changed 6 years ago by alphaone@…

  • blocked set to 341

Have you been able to obtain the correct scale yet?

comment:6 Changed 6 years ago by tony_tu@…

We have change the resister value on the PMU battery circuit, should give
better scale compare to Bv3 version, but we did not put all rest Bv4 SMT due
to holiday/quene issue. We will verify the boards rest SMT finished.

comment:7 Changed 6 years ago by speedevil@…

I'm doing a simple patch for this, based on a simple curve-fit. Just polishing ATM.

comment:8 Changed 5 years ago by willie_chen@…

  • Owner changed from tony_tu@… to michael@…
  • Status changed from assigned to new

comment:9 Changed 5 years ago by willie_chen@…

  • Cc speedevil@… added

Did this patch publish in kernel?So we can close this bug.

comment:10 Changed 5 years ago by andy

  • Status changed from new to closed
  • Resolution set to fixed

We took a patch for charging curve some months ago, on GTA02 we do have coulomb counter, so closing.

Note: See TracTickets for help on using tickets.