FS#8871 - battery runningtime eta range checking

Attached to Project: Rockbox
Opened by Tomasz Wasilczyk (tomkiewicz) - Friday, 11 April 2008, 11:23 GMT
Last edited by Nils Wallménius (nls) - Saturday, 12 April 2008, 07:14 GMT
Task Type Patches
Category Battery/Charging
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


When battery voltage drops under battery_level_shutoff (about 1-10mV - it depends from round off), device estimated runtime loops over unsigned int range and displays 35 millions hours left instead of zero. It can be noticed when using battery benchmark or System->Rockbox info->Battery.

I've added voltage range checking, so that bug is now fixed. I've tested it (i.e. by modifying values in /firmware/target/.../powermgmt-c200.c) and works OK.

example battery benchmark files, that shows issue (look at the end of the file):
This task depends upon

Closed by  Nils Wallménius (nls)
Saturday, 12 April 2008, 07:14 GMT
Reason for closing:  Accepted
Additional comments about closing:  Thanks!