Closed
Bug 712228
Opened 14 years ago
Closed 13 years ago
Fix battery status impl for maguro device
Categories
(Core :: Hardware Abstraction Layer (HAL), defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: cjones, Unassigned)
Details
The gonk hal impl expects to find /sys/class/power_supply/battery/charging_source, and AFAICT that's used to determine if the battery is charging. On the maguro however, we don't have charging_source. Here's what we do have
# for i in `ls`; do echo "information \"$i\": `cat $i`"; done
for i in `ls`; do echo "information \"$i\": `cat $i`"; done
information "uevent": POWER_SUPPLY_NAME=battery
POWER_SUPPLY_STATUS=Charging
POWER_SUPPLY_HEALTH=Overheat
POWER_SUPPLY_PRESENT=1
POWER_SUPPLY_TECHNOLOGY=Li-ion
POWER_SUPPLY_VOLTAGE_MAX_DESIGN=4200
POWER_SUPPLY_VOLTAGE_MIN_DESIGN=3300
POWER_SUPPLY_VOLTAGE_NOW=3970
POWER_SUPPLY_CAPACITY=69
subsystem: invalid length
information "subsystem":
device: invalid length
information "device":
information "status": Charging
information "health": Overheat
information "present": 1
information "technology": Li-ion
information "voltage_max_design": 4200
information "voltage_min_design": 3300
information "voltage_now": 3970
information "capacity": 69
information "type": Battery
power: invalid length
information "power":
So looks like status == "Charging" will get us the same thing. Does /sys/class/power_supply/battery/status exist on the sgs2?
If we run into a few more of these discrepancies, we should look at pulling in the upower code that deals with this.
| Reporter | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•