Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Drivers
  • Assigned To No-one
  • Operating System Another
  • Severity Very Low
  • Priority Low
  • Reported Version Rbutil git
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by funman - 2010-05-06
Last edited by funman - 2010-05-08

FS#11246 - as3525: Stop clock when SDRAM is idle

With attached patch I have 40 minutes more battery life on Clipv1.

I’ll test again on Fuzev1 and Clip+ to confirm.

SDRAM setup is done by the bootloader in sdram_init(), so I moved this code into system_init() so there is no need to update the bootloaders to benefit from this : are you OK with that ?

More testing and on other models would be fine too.

Closed by  funman
2010-05-08 13:59
Reason for closing:  Accepted
Additional comments about closing:  

r25900

Nice! I l like how this is basically an automatic setting, without having to add explicit enable/disable code.
I can test tonight on the c200v2 and my own clip v1.

Seems to give a bit more battery life on Clipv1 / Fuzev2: http://forums.rockbox.org/index.php?topic=14064.msg166475#msg166475

Move the setting to the bootloader for AS3525v2 since we will need new bootloaders.

Keep it in rockbox.sansa for AS3525v1 so we can keep using the existing, well-tested bootloader.

I tested the first patch on my e260v2 with r25836 and got the following results:

without  FS#11246  - 14h 17min (results file on SansaRuntime wiki page)
with  FS#11246  - 16h 26min (results attached)

I will now recharge and test with the second patch.

+1h10 on fuzev2
+30min on clipv1
+2h on e200v2
+1h on clipv2 when boosted (else it crashed too early)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing