• Status Closed
  • Percent Complete
  • 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 funman - 2010-05-29
Last edited by funman - 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  funman
2010-07-30 17:31
Reason for closing:  Rejected
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

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

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+.

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.

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.


Available keyboard shortcuts


Task Details

Task Editing