- Status Closed
- Percent Complete
- 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
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.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
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.