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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

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.
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.