Ticket #1182 (closed defect: fixed)
Charging current measurement is incorrect (fix)
| Reported by: | andrew.paulsen@… | Owned by: | willie_chen@… |
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | kernel | Version: | 2007.2 |
| Severity: | normal | Keywords: | |
| Cc: | buglog@… | Blocked By: | |
| Blocking: | Estimated Completion (week): | ||
| HasPatchForReview: | PatchReviewResult: | ||
| Reproducible: |
Description
There are multiple problems with the charging current measurement that I have
fixed in this patch.
- Least significant bits of ADCIN1 were not shifted correctly
- A 6V reference voltage was assumed, but a 2.4V reference is actually used
- Math could result in negative values, but unsigned types were used
- Explicit type casting was added to the calculation to correct a possible overflow
Attachments
Change History
Note: See
TracTickets for help on using
tickets.
Fixes charge current measurement. Applies after the patch in bug 1181.