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

Issue 813286 link

Starred by 1 user
Status: WontFix
Owner:
philipchen@chromium.org
Closed: Nov 6
Cc:
philipchen@chromium.org
aaboagye@chromium.org
█ scollyer@chromium.org
adurbin@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

ec: battery/max17055: should return 'error' for battery params when battery is disconnected

Project Member Reported by philipchen@chromium.org, Feb 17 2018 
Fuel gauge max17055 is used in the systems w/o smart battery.
We read battery parameters from max17055, not directly from the battery.

When we boot on AC with battery disconnected, we can still read battery parameters from max17055 as long as max17055 is powered.
But the read reg values are nonsense.

We should return error in this case to match the behavior on the systems using smart battery.

Comment 1 by philipchen@chromium.org, Feb 17 2018

Summary: ec: battery/max17055: should return 'error' for battery params when battery is disconnected (was: ec: battery/max17055: return error when battery is disconnected)

Comment 2 by philipchen@chromium.org, Feb 17 2018 

 Issue 813287  has been merged into this issue.

Comment 3 by philipchen@chromium.org, Nov 6

Status: WontFix (was: Assigned)
We have an alternative solution:
https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1311513

So this can be closed now.
►

Sign in to add a comment