Ticket #255 (closed defect: fixed)
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
Change History
Changed 6 years ago by philipp.zabel@…
- Attachment discharge.txt.gz added
battemp/battvolt measurement during discharge
comment:2 Changed 6 years ago by philipp.zabel@…
- attachments.mimetype changed from text/plain to application/octet-stream
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: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.
