Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Battery/Charging
  • Assigned To No-one
  • Operating System iPod Nano
  • Severity Medium
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by the_winch - 2006-04-02
Last edited by jdgordon - 2007-07-31

FS#4998 - Ipod nano battery level.

Correct BATTERY_SCALE_FACTOR, at least as close as I can get
battery type
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.

Closed by  jdgordon
2007-07-31 12:39
Reason for closing:  Out of Date

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?

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.

could you please increase the maximum to 600mAh to cover the ipod video 60gb?

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.

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.

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!

@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?

yes, i think i was using the wrong one... i apologize for my error

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.

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

Available keyboard shortcuts

Tasklist

Task Details

Task Editing