Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#11597 - AMSv1: improve battery life

Attached to Project: Rockbox
Opened by Rafaël Carré (funman) - Thursday, 02 September 2010, 14:51 GMT+2
Last edited by Rafaël Carré (funman) - Sunday, 05 September 2010, 17:34 GMT+2
Task Type Patches
Category Battery/Charging
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.6
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Gives ~ +20% battery life

Changes cpu freq by CGU_PROC instead of using fastbus/synchronous
Also removes setting for asynchronous

Tested on clipv1/fuzev1

Problem: long home press seems to be broken on fuzev1?
   set_cpu_freq.diff (3.6 KiB) 
 b/firmware/target/arm/as3525/clock-target.h  |   13 -----------
 b/firmware/target/arm/as3525/system-as3525.c |   30 ++++++---------------------
 2 files changed, 8 insertions(+), 35 deletions(-)

This task depends upon

Closed by  Rafaël Carré (funman)
Sunday, 05 September 2010, 17:34 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  committed v4 patch with some comments: r28000
Comment by Rafaël Carré (funman) - Thursday, 02 September 2010, 15:06 GMT+2
This is more correct, in synchronous mode cpu clock must be > pclk/hclk (and an integer multiple of it)
   set_cpu_freq-v2.diff (3.4 KiB) 
 b/firmware/target/arm/as3525/clock-target.h  |   13 +------------
 b/firmware/target/arm/as3525/system-as3525.c |   19 ++++++++-----------
 2 files changed, 9 insertions(+), 23 deletions(-)

Comment by Rafaël Carré (funman) - Thursday, 02 September 2010, 16:11 GMT+2
My home button has a problem with SVN too so it's not related
Comment by Rafaël Carré (funman) - Friday, 03 September 2010, 08:31 GMT+2
Clipv1: 10h runtime
Comment by Michael Chicoine (mc2739) - Friday, 03 September 2010, 13:25 GMT+2
e200v2: 19:22:49 runtime
Comment by Jack Halpin (FlynDice) - Friday, 03 September 2010, 14:13 GMT+2
fuzev1 SVN r27965 12:23, with patch 16:35
Comment by Rafaël Carré (funman) - Friday, 03 September 2010, 17:07 GMT+2
I got 22hours on fuzev1 but something definitely went wrong

2 consecutive lines are:

01:08:12, 04092, 092%, 13:40, 4088, -, -, -
14:15:46, 51346, 091%, 13:36, 4075, -, -, -

It seems somehow the player was sleeping in some state all night until I looked at the battery level in the morning (I didn't press resume, and the config has nothing weird)

   fuze.png (4.4 KiB)
   config.cfg (0.3 KiB)
Comment by Rafaël Carré (funman) - Friday, 03 September 2010, 17:57 GMT+2
- Base PCLK off PLLA and not FCLK, so PCLK is not divided by 4 when unboosted

Not sure if the battery gains will still be there
   set_cpu_freq-v3.diff (3.9 KiB) 
 b/firmware/target/arm/as3525/clock-target.h  |   23 ++++++++++-------------
 b/firmware/target/arm/as3525/system-as3525.c |   19 ++++++++-----------
 2 files changed, 18 insertions(+), 24 deletions(-)

Comment by Rafaël Carré (funman) - Friday, 03 September 2010, 20:33 GMT+2
Another idea to try:

- reduce FCLK to maximum (use maximum dividers and the 24MHz crystal as source) when unboosted

Also it might be interesting to try a build with PCLK set to 15.5MHz to see if it compare equally to previous results
   set_cpu_freq-v4.diff (3.8 KiB) 
 b/firmware/target/arm/as3525/clock-target.h  |   23 ++++++++++-------------
 b/firmware/target/arm/as3525/system-as3525.c |   17 ++++++-----------
 2 files changed, 16 insertions(+), 24 deletions(-)

Comment by Bertrik Sikken (bertrik) - Friday, 03 September 2010, 23:33 GMT+2
I got 13h30 with the v2 patch on clipv1. An older benchmark for this player gave me 10h59m.
Comment by Rafaël Carré (funman) - Saturday, 04 September 2010, 14:41 GMT+2
With v4 patch:
- fuzev1 15h30
- clipv1 8h50
Comment by Michael Chicoine (mc2739) - Saturday, 04 September 2010, 21:45 GMT+2
e200v2 with v4 patch: 16:45:56 runtime

Loading...