Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Battery/Charging
  • Assigned To No-one
  • Operating System iPod 5G
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes 4
  • Private
Attached to Project: Rockbox
Opened by Andree Buschmann - 2007-05-26
Last edited by Peter D'Hoye - 2007-07-14

FS#7216 - Correction for iPOD Video/Nano battery type and consumption

This patch is very tiny one. It defines the correct battery capacities for iPOD Nano and iPOD Video and defines realistic current consumptions. for both types.
For the two different battery types of iPOD Video (400mAh for 30GB and 600mAh for 60/80GB) there was inserted a new define which is configured to use 30GB-model as a default.
The current consumptions are calculated from the IpodRuntime wiki page.

Closed by  Peter D'Hoye
2007-07-14 14:42
Reason for closing:  Accepted
Dave Hooper commented on 2007-05-29 21:34

Ideally the correct battery #define should be automatic based on what target you build for.

Andree Buschmann commented on 2007-05-29 21:45

Please help me if missed it. Is there already a define to seperate between 30GB- and 60/80GB-iPods?

Otherwise I can simply delete this additional default-capacity for iPOD Videos and let the user set
the correct capacity for the current model with rockbox settings. At least the correction for iPOD
Nano battery capacity and the power consumption of Nano and Video would be helpful.

Douglas Valentine commented on 2007-05-31 11:39

Sounds goods to me.

I could the information in this related patch (http://www.rockbox.org/tracker/task/4998) for the ipod nano battery could also be merged into your patch?

I am not the original author of the patch but I believe he made the patch after making battery voltage level measurements with a multi meter.

Andree Buschmann commented on 2007-05-31 16:52

Yes, this can be merged. I also have tried to configure a more correct battery-type (LIION400) for iPOD Video 30GB.
Btw, as far as I know the batteries are Li-Ion, not Li-Poly. And the nano battery is 300mAh, not 340mAh.
Do you think the correction for Battery Scale factor can be also used for the iPOD Video?

Last question: Can you give any hint how to differ between 30GB and 60GB iPODs at compiletime?

Douglas Valentine commented on 2007-05-31 18:44

Yes you are correct, if the info on this web site is right http://ipodbatteryfaq.com/ipodbatteryandpower.html the nano has a 300mAh Li-Ion battery.

As for your other two question sadly I don’t have the answers, maybe one of the core rockbox developers will step forward with some more info for you.

Andree Buschmann commented on 2007-05-31 18:45

Ok, added main parts of patch #4998 into this one. So, this patch contains:
- correct battery capacity for iPOD nano (300mAh), iPOD Video 30GB (400mAh) and iPOD Video 60/80GB (600mAh)
- new battery types for iPOD nano (BATT_LIION300) and for iPOD Video 30GB (BATT_LIION400)
- battery charge/discharge curves as well as level_shutoff- and level_dangerous-definitions (for iPOD nano took from patch #4998, for iPOD Video took from measurements upon battery_benchs)
- better values for current consumption for iPOD nano and iPOD Video (calculated from IpodRuntime-wiki)

ToDo’s:
- to check the current consumption of nano and Video when backlight is on
- to check charge/discharge/level_shutoff/level_dangerous for 60/80GB Video’s (but maybe the assumption of similar behaviour to 30GB Video’s is ok)

So, this patch isn’t the 100% solution, but nevertheless hits reality much more than the actual Trunk-version.

Peter D'Hoye commented on 2007-07-12 23:12

Can you remove all TABs and sync to latest svn?

Andree Buschmann commented on 2007-07-14 10:44

Sync’ed to latest SVN #13878 and removed TABs.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing