Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#6860 - prepared correct sansa e200 battery status handling

Attached to Project: Rockbox
Opened by Matthias Wientapper (mattzz) - Monday, 19 March 2007, 21:56 GMT+2
Last edited by Marc Guay (Marc_Guay) - Monday, 31 March 2008, 21:23 GMT+2
Task Type Patches
Category Battery/Charging
Status Closed
Assigned To No-one
Player Type Sansa e200
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

- prepared a first version of correct battery status handling
- improved battery status accuracy
- this is a first shot, more measurements have to be done
   sansa-battery.diff (2.9 KiB) 
 firmware/export/config.h      |    1 +
 firmware/export/config-e200.h |    2 +-
 firmware/powermgmt.c          |   10 ++++++++++
 3 files changed, 12 insertions(+), 1 deletion(-)

This task depends upon

Closed by  Marc Guay (Marc_Guay)
Monday, 31 March 2008, 21:23 GMT+2
Reason for closing:  Out of Date
Additional comments about closing:  Power consumption improvements have made this obsolete.
Comment by Matthias Wientapper (mattzz) - Thursday, 22 March 2007, 00:35 GMT+2
- corrected shutdown voltage to 3,3V
- warning level 3,4V
- discharging curve linear between 3,3V and 4,2V
   sansa-battery_2.diff (2.8 KiB) 
 firmware/export/config.h      |    1 +
 firmware/export/config-e200.h |    2 +-
 firmware/powermgmt.c          |   10 ++++++++++
 3 files changed, 12 insertions(+), 1 deletion(-)

Comment by Barry Wardell (barrywardell) - Friday, 23 March 2007, 11:59 GMT+2
sansa-battery_2.diff committed to SVN. I'll leave the task open for now as mattz wanted to improve on the linear discharge curve between 3.3V and 4.2V.
Comment by Matthias Wientapper (mattzz) - Tuesday, 03 April 2007, 21:31 GMT+2
Added first complete discharge curve (had to patch batter_bench) on
http://www.rockbox.org/twiki/bin/view/Main/SandiskE200BatteryMeasurements
Comment by Dagni McPhee (donutman25) - Tuesday, 10 April 2007, 06:34 GMT+2
my battery_bench file
   battery_bench.txt (4.4 KiB)
Comment by powers (kpowers11) - Thursday, 12 April 2007, 04:13 GMT+2
Here is my battery_bench file. It ran from just about full charge to low battery shutdown.
   battery_bench.txt (14 KiB)
Comment by Matthias Wientapper (mattzz) - Saturday, 14 April 2007, 12:50 GMT+2
Please make sure that you use a patched battery_bench plugin - otherwise the battery_bench.txt will not be completely written.
(Refer to http://www.rockbox.org/twiki/bin/view/Main/SandiskE200BatteryMeasurements for the information about the one line that has to be patched)
Comment by Alex (testdasi) - Sunday, 06 May 2007, 23:01 GMT+2
Recalibrate the voltage based on the latest batter_bench on the wiki page.
(application/octet-stream)    battery_cal01.patch (1 KiB) 
 powermgmt.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comment by Michael Hahn (disorganizer) - Wednesday, 20 June 2007, 11:29 GMT+2
maybe a new idea to this:
instead of patching rockbox with new battery "profiles", wouldnt it be better to patch rockbox to use a profile which we can put into a normal textfile?
as far as i understand the textfile only needs to hold a conversion table from Voltage to % and runtime left. The rockbox battery indication then needs to look into this file to get the correct readings.
this way everyone could use a battery profile which fits his/her needs.

if we could then update the battery-benchmark plugin to write exactly this textfile for the individual sansa we use, every user can use the plugin to create a battery profile matching his/her exact use of the sansa (as i believe the differen usages, graphics, games, settings influence the possible runtime of a battery).


Comment by Michael Sevakis (MikeS) - Wednesday, 20 June 2007, 12:40 GMT+2
In general that might be helpful with the use of custom, non-OEM replacement batteries as well without having to have a custom build. Other profile information besides a conversion table should probably be included for charging purposes.
Comment by Eddy (bascule) - Thursday, 01 November 2007, 09:47 GMT+2
Does battery_bench still require to be patched? I can't see a bug report or patch sitting in Flyspray and without it working properly, there is much less chance of an average user running it and adding to the base of knowledge...

EDIT - yes it does need to be patched.

I've raised a proper Flyspray bug report for this specific battery_bench plugin error here: http://www.rockbox.org/tracker/task/8070
Comment by William Rowell (rowell) - Tuesday, 06 November 2007, 19:07 GMT+2
My battery_bench.txt from a full charge to an automatic shutdown.
   battery_bench.txt (71.2 KiB)

Loading...