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.6
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by dtr - 2010-07-07
Last edited by b0hoon - 2010-10-18

FS#11462 - Philips GoGear HDD63x0 patch to disable / enable backlight

The current code only sets the backlight to the lowest setting. This patch actually turns the backlight off when the timeout occurs.

Closed by  b0hoon
2010-10-18 20:09
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

Commited in r28307. Thanks!

What happens on the other philips?

Does the backlight work correctly on them or is it set to the lowest setting too?

dtr commented on 2010-07-10 03:38

The altered file is only used by the philips hdd63x0 and hdd16x0 targets. I cannot comment on the current functionality of the backlight for the hdd16x0 as I do not own one. My assumption was that the code was correct for the hdd16x0 and as such, the patch only changes functionality for the hdd63x0.

mx03 commented on 2010-09-21 19:27

I use HDD1620 and when the backlight timeout occurs, the lcd blinks a little. (Turn backlights on, off, on, off, …)
Maybe this patch solves the problem, if i have time to compile i try it.

I’m not sure if this method is correct at all. I must look into OF and find it.

EDIT: Found it. Solution is correct, OF does it the same way but always with GPO32_ENABLE |= 0×400 before GPO32_VAL.

mx03: This patch has no influence on HDD16x0 at all.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing