New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 795991 link

Starred by 1 user
Status: Untriaged
Owner: ----
Cc:
█ vpalatin@chromium.org
█ rspangler@chromium.org
█ sha...@chromium.org
adurbin@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

EC: battery cycle count information is assumed to be static

Project Member Reported by drinkcat@chromium.org, Dec 19 2017 
The battery cycle count information is only updated in common/charge_state_v2.c:update_static_battery_info, which is only called once per EC boot, or whenever the battery is disconnected (that is, very rarely on production systems).

To quote vpalatin (https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/670380/13/include/ec_commands.h#4682): "this means our cycle_count is partially wrong most of the time...
we can live with it since mostly nobody uses it, but we need to be careful when reading feedbacks & co"

We might want to consider moving this field to battery_get_params (then passed to AP via common/charge_state_v2.c:update_dynamic_battery_info), if we think there is a use for this value.
Project Member

Comment 1 by sheriffbot@chromium.org, Dec 19

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
►

Sign in to add a comment