Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category LCD
  • Assigned To No-one
  • Operating System Another
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.4
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Rafaël Carré - 2010-05-29
Last edited by Rafaël Carré - 2010-07-30

FS#11330 - Clip*: disable peripherals in lcd_enable()

Switch off DBOP (or SSP for Clip+) clocks when LCD is not in use.

7 minutes more runtime on Clipv1
10 minutes more runtime on Clip+
6 minutes more runtime on Clipv2

These numbers are really low so we need more benches to be sure there is an effect at all.

Closed by  Rafaël Carré
2010-07-30 17:31
Reason for closing:  Rejected
Additional comments about closing:  

Clipv1 LCD sometimes stay off / (or crashes?) after unplugging USB

Bertrik Sikken commented on 2010-07-25 21:18

Attached in a resynced version of the patch (funman, please verify my sync). Don’t we also need something like dbop_init in lcd-clipv2.c ?

I’ll benchmark this in the clip+.

Rafaël Carré commented on 2010-07-25 21:28

Your sync looks alright

I added dbop_init() for clipv1 because the screen wouldn’t go back on, but it worked on clipv2 with only toggling the clock enable bit.

Disabling the DBOP clock might reset some registers, we should verify what happens on clipv2 too, the registers might have a defaut reset value which works anyway.

Bertrik Sikken commented on 2010-07-26 16:07

I got 17h21m with the patch compared to 17h17m earlier with plain SVN (benchmarked with the same album), so 4 minutes improvement for the Clip+ I tested on (a COMBO4E variant with new OLED controller and new SD controller). This was done with default settings (except repeat turned on), so most of the benchmark was done with the display OFF anyway.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing