Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Battery/Charging
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.6
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by funman - 2010-09-02
Last edited by funman - 2010-09-05

FS#11597 - AMSv1: improve battery life

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)
Closed by  funman
2010-09-05 15:34
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

committed v4 patch with some comments: r28000

funman commented on 2010-09-02 13:06

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)
funman commented on 2010-09-02 14:11

My home button has a problem with SVN too so it’s not related

funman commented on 2010-09-03 06:31

Clipv1: 10h runtime

mc2739 commented on 2010-09-03 11:25

e200v2: 19:22:49 runtime

FlynDice commented on 2010-09-03 12:13

fuzev1 SVN r27965 12:23, with patch 16:35

funman commented on 2010-09-03 15:07

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)
funman commented on 2010-09-03 15:57

- 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)
funman commented on 2010-09-03 18:33

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)
bertrik commented on 2010-09-03 21:33

I got 13h30 with the v2 patch on clipv1. An older benchmark for this player gave me 10h59m.

funman commented on 2010-09-04 12:41

With v4 patch:
- fuzev1 15h30
- clipv1 8h50

mc2739 commented on 2010-09-04 19:45

e200v2 with v4 patch: 16:45:56 runtime

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing