Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#4998 - Ipod nano battery level.

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

Details

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.
   nanobatt.patch (3.9 KiB) 
 apps/settings_menu.c              |    7 ++++++-
 firmware/powermgmt.c              |    5 +++++
 firmware/export/config-ipodnano.h |    5 +++--
 firmware/export/config.h          |    1 +
 firmware/export/powermgmt.h       |    4 ++++
 5 files changed, 19 insertions(+), 3 deletions(-)

This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Tuesday, 31 July 2007, 14:39 GMT+2
Reason for closing:  Out of Date
Comment by Douglas Valentine (Dwyloc) - Saturday, 19 August 2006, 14:52 GMT+2
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, 19:52 GMT+2
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.
   ipodnano_battery.patch (3 KiB) 
 firmware/powermgmt.c              |    7 +++++++
 firmware/export/config-ipodnano.h |   13 +++++++------
 firmware/export/config.h          |    1 +
 3 files changed, 15 insertions(+), 6 deletions(-)

Comment by robin (robin0800) - Saturday, 21 April 2007, 19:07 GMT+2
could you please increase the maximum to 600mAh to cover the ipod video 60gb?
Comment by Douglas Valentine (Dwyloc) - Saturday, 21 April 2007, 20:23 GMT+2
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, 19:34 GMT+2
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.
   battery_bench.txt (10.1 KiB)
Comment by Matt M (Chesteta) - Friday, 18 May 2007, 00:24 GMT+2
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, 13:34 GMT+2
@Matt

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, 07:47 GMT+2
yes, i think i was using the wrong one... i apologize for my error
Comment by Matt M (Chesteta) - Saturday, 02 June 2007, 20:18 GMT+2
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, 20:40 GMT+2
My patch has now been superseded by http://www.rockbox.org/tracker/task/7216 which includes my patch and also displays more correct information for time remaining.

Loading...