New issue
Advanced search Search tips

Issue 786968 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Feb 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature



Sign in to add a comment

Chrome OS Can't estimate battery runtime with multiple batteries

Reported by craig_co...@outlook.com, Nov 20 2017

Issue description

Chrome Version: 60.0.3112.117
Chrome OS Version: 60.2.93
Chrome OS Platform: Lenovo X250
Network info: n/a


Chromium OS cannot provide estimated runtimes on DC (battery) on systems with multiple batteries. Lenovo Thinkpad X250 with two batteries constantly saying "100% - Estimating..." on DC power. (Battery levels shown correctly in Linux/Windows)
 

Comment 1 by vsu...@chromium.org, Jan 16 2018

Components: OS>Kernel>Power

Comment 2 by derat@chromium.org, Jan 16 2018

Labels: -Type-Bug Type-Feature
Does https://crrev.com/c/853295 help? (You'll need to echo "1" into /var/lib/power_manager/multiple_batteries.)

Comment 3 by tbroch@chromium.org, Jan 19 2018

Owner: derat@chromium.org
Status: Assigned (was: Unconfirmed)
It doesn't, unfortunately. 

Comment 5 by derat@chromium.org, Jan 22 2018

Can you attach the output from print_sysfs_power_supply_data?

Comment 7 by derat@chromium.org, Feb 8 2018

Status: WontFix (was: Assigned)
Closing due to lack of info, but I'm happy to reopen this bug and look into it further if you can paste the output from print_sysfs_power_supply_data while both batteries are present.
Project Member

Comment 8 by bugdroid1@chromium.org, Mar 8 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/2a5d89ec74397c572657b858afd8e2be9b9af655

commit 2a5d89ec74397c572657b858afd8e2be9b9af655
Author: Daniel Erat <derat@chromium.org>
Date: Thu Mar 08 08:37:40 2018

power: Read status of multiple batteries by default.

Set powerd's multiple_batteries pref to 1 by default.

BUG=b:71491303, chromium:786968 
TEST=did a build and verified that
     /usr/share/power_manager/multiple_batteries is 1

Change-Id: I6fac6a6a0869f757fb89e58e45000ff5047fa19b
Reviewed-on: https://chromium-review.googlesource.com/954053
Commit-Ready: Dan Erat <derat@chromium.org>
Tested-by: Dan Erat <derat@chromium.org>
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>

[add] https://crrev.com/2a5d89ec74397c572657b858afd8e2be9b9af655/power_manager/default_prefs/multiple_batteries

Hi guys, apologies, this completely slipped off my radar. How can i progress this?

Comment 10 by derat@chromium.org, Mar 29 2018

#9: See the request in #5 and #7.

Sign in to add a comment