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:  

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