Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

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

Attached to Project: Rockbox
Opened by Andree Buschmann (Buschel) - Saturday, 26 May 2007, 20:07 GMT+2
Last edited by Peter D'Hoye (petur) - Saturday, 14 July 2007, 16:42 GMT+2
Task Type Patches
Category Battery/Charging
Status Closed
Assigned To No-one
Player Type iPod 5G
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

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.
   iPOD_Video_battery_v1.patch (4.4 KiB) 
 firmware/export/config-ipodnano.h  |   10 +++++-----
 firmware/export/config-ipodvideo.h |   19 ++++++++++++++++++-
 firmware/export/powermgmt.h        |   14 +++++++++++++-
 3 files changed, 36 insertions(+), 7 deletions(-)

This task depends upon

Closed by  Peter D'Hoye (petur)
Saturday, 14 July 2007, 16:42 GMT+2
Reason for closing:  Accepted
Comment by Dave Hooper (stripwax) - Tuesday, 29 May 2007, 23:34 GMT+2
Ideally the correct battery #define should be automatic based on what target you build for.
Comment by Andree Buschmann (Buschel) - Tuesday, 29 May 2007, 23:45 GMT+2
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.
Comment by Douglas Valentine (Dwyloc) - Thursday, 31 May 2007, 13:39 GMT+2
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.
Comment by Andree Buschmann (Buschel) - Thursday, 31 May 2007, 18:52 GMT+2
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?
Comment by Douglas Valentine (Dwyloc) - Thursday, 31 May 2007, 20:44 GMT+2
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.
Comment by Andree Buschmann (Buschel) - Thursday, 31 May 2007, 20:45 GMT+2
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.
   iPOD_Video_battery_v2.patch (7.6 KiB) 
 firmware/export/config.h           |    2 ++
 firmware/export/config-ipodnano.h  |   15 +++++++--------
 firmware/export/config-ipodvideo.h |   25 +++++++++++++++++--------
 firmware/export/powermgmt.h        |   14 +++++++++++++-
 firmware/powermgmt.c               |   20 ++++++++++++++++++++
 5 files changed, 59 insertions(+), 17 deletions(-)

Comment by Peter D'Hoye (petur) - Friday, 13 July 2007, 01:12 GMT+2
Can you remove all TABs and sync to latest svn?
Comment by Andree Buschmann (Buschel) - Saturday, 14 July 2007, 12:44 GMT+2
Sync'ed to latest SVN #13878 and removed TABs.
   iPOD_Video_battery_v3.patch (7.9 KiB) 
 firmware/export/config.h           |    2 ++
 firmware/export/config-ipodnano.h  |   15 +++++++--------
 firmware/export/config-ipodvideo.h |   28 ++++++++++++++++++++--------
 firmware/export/powermgmt.h        |   14 +++++++++++++-
 firmware/powermgmt.c               |   20 ++++++++++++++++++++
 5 files changed, 62 insertions(+), 17 deletions(-)

Loading...