Notice: A non well formed numeric value encountered in /sites/ on line 96 Notice: A non well formed numeric value encountered in /sites/ on line 96 Notice: A non well formed numeric value encountered in /sites/ on line 96 Deprecated: Function create_function() is deprecated in /sites/ on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/ on line 845 Deprecated: Function create_function() is deprecated in /sites/ on line 111 FS#4998 : Ipod nano battery level.



FS#4998 - Ipod nano battery level.

Attached to Project: Rockbox
Opened by David Dent (the_winch) - Sunday, 02 April 2006, 16:30 GMT
Last edited by Jonathan Gordon (jdgordon) - Tuesday, 31 July 2007, 12:39 GMT
Task Type Patches
Category Battery/Charging
Status Closed
Assigned To No-one
Operating System iPod Nano
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


Correct BATTERY_SCALE_FACTOR, at least as close as I can get it.
New battery type BATT_LIPOL340
Changed battery capacity setting to step in 20 instead of 50 when BATTERY_CAPACITY_MAX <= 1000

The battery level now works correctly but the estimated remaining time is way off.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Tuesday, 31 July 2007, 12:39 GMT
Reason for closing:  Out of Date
Comment by Douglas Valentine (Dwyloc) - Saturday, 19 August 2006, 12:52 GMT
Your patch sounds useful but sadly it no longer applies properly to current builds.
Do you have any plans to update it and see if you can get it committed in to CVS now the feature freeze is over?
Comment by Douglas Valentine (Dwyloc) - Sunday, 15 April 2007, 17:52 GMT
I have updated the patch to apply to the current version of SVN and made a few small additional changes.

The patch now sets the default battery capacity to 340mAh, the minimum battery size to 330mAh and the maximum to 400mAh
with increments of 10 instead of 50.
Comment by robin (robin0800) - Saturday, 21 April 2007, 17:07 GMT
could you please increase the maximum to 600mAh to cover the ipod video 60gb?
Comment by Douglas Valentine (Dwyloc) - Saturday, 21 April 2007, 18:23 GMT
robin: My patch dose more than just add a new battery type to the list for selection in the menu.

So an equivalent patch for the ipod video would have to be created by someone with an ipod video who
dose not mind taking their player apart to make some voltage measurements

As my patch dose not make any changes to the ipod video's code it should make no difference a build
for the ipod video 60GB.
Comment by Douglas Valentine (Dwyloc) - Monday, 07 May 2007, 17:34 GMT
I have attached the results from running the battery bench plugin on my ipod nano with the above patch, my ipod battery lasted just over 7 hours and 30 minuets.

I fully charged my player under the retail OS with my USB charger then rebooted into rockbox and started the battery bench plugin.

Then I just started my player playing in file browser mode and left it to play through as much of my music as possible with "Auto-Change Directory" enabled before the battery run out at my normal playback volume and with my normal headphones connected.

As I have a mixture of mp3 and ogg vorbis files encoded at different bit rates some I feel that this gives a more accurate idea of real world battery live than a more clinical bench mark would playing only files encoded in the same way with the same encoder.

My MP3 are mostly ripped with CDex and encoded with ether latest stable release version of the LAME DLL I could find at the time or the latest stable of the Ogg Vorbis DLL available at the time I encoded the files.
Comment by Matt M (Chesteta) - Thursday, 17 May 2007, 22:24 GMT
I cannot seem to apply this patch, giving an error... I have even gone into the patch and updated the settings_menu.c location with no luck. An update or fix would be appreciated. Thank you!
Comment by Douglas Valentine (Dwyloc) - Thursday, 31 May 2007, 11:34 GMT

If my updated version of the patch still applies fine here.

Are you sure you are trying my updated patch and not the old one at the top of the page?
Comment by Matt M (Chesteta) - Saturday, 02 June 2007, 05:47 GMT
yes, i think i was using the wrong one... i apologize for my error
Comment by Matt M (Chesteta) - Saturday, 02 June 2007, 18:18 GMT
its working now, can I use battery_bench.txt values as my scale for the battery time remaining? (so it displays the correct remaining time) and yes, i realize that you mentioned that the time remaining issue in the first post however I assume you posted the battery_bench.txt to give people a scale? Thanks again.
Comment by Douglas Valentine (Dwyloc) - Saturday, 02 June 2007, 18:40 GMT
My patch has now been superseded by which includes my patch and also displays more correct information for time remaining.