• Status Closed
  • Percent Complete
  • Task Type Bugs
  • Category LCD
  • Assigned To No-one
  • Operating System iPod 5G
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by dreamlayers - 2009-04-14
Last edited by dreamlayers - 2009-04-21

FS#10129 - LCD sleep timer should start after PWM fadeout

Currently, the LCD sleep timer starts at the beginning of backlight PWM fade-out. This means that if LCD “Sleep (After Backlight Off)” is set to “Always” and “Backlight Fade Out” isn’t off, the LCD will go white at the beginning of the fade. This is ugly and unpleasant. A simple workaround is to set “Sleep (After Backlight Off)” to “5” (seconds) instead of “Always”. This attached patch fixes the problem.

This is a known issue with  FS#9890  iPod 5G LCD sleep. The patch is “lcd_sleep_after_pwm_fadeout.patch” from that task. However, it is actually a separate issue in platform independent code, so I guess it needs its own FS entry.

Note that a PWM fade can end in 3 different ways:
1) The LCD fades and the timer ISR handles the end.
2) Other code calls timer_register(), so the fade is interrupted, and backlight_release_timer() is called.
3) timer_register() fails.
Of these 1) is the only case which normally happens, but the patch handles all three.

(Behaviour confirmed and patch tested with r20696)

Closed by  dreamlayers
2009-04-21 22:01
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


sadur commented on 2009-04-18 13:49

Applied to r20728 and works well. Now the LCD fades out as expected.


Available keyboard shortcuts


Task Details

Task Editing