Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

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

Attached to Project: Rockbox
Opened by David Fowle (dtr) - Wednesday, 07 July 2010, 09:18 GMT+2
Last edited by Szymon Dziok (b0hoon) - Monday, 18 October 2010, 22:09 GMT+2
Task Type Patches
Category LCD
Status Closed
Assigned To No-one
Player Type Another
Severity Low
Priority Normal
Reported Version Release 3.6
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

The current code only sets the backlight to the lowest setting. This patch actually turns the backlight off when the timeout occurs.
   hdd6330-backlight.diff (0.6 KiB) 
 firmware/target/arm/philips/backlight-hdd.c |    8 ++++++++
 1 file changed, 8 insertions(+)

This task depends upon

Closed by  Szymon Dziok (b0hoon)
Monday, 18 October 2010, 22:09 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  Commited in r28307. Thanks!
Comment by Rafaël Carré (funman) - Friday, 09 July 2010, 22:29 GMT+2
What happens on the other philips?

Does the backlight work correctly on them or is it set to the lowest setting too?
Comment by David Fowle (dtr) - Saturday, 10 July 2010, 05:38 GMT+2
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.
Comment by Maximilian Belz (mx03) - Tuesday, 21 September 2010, 21:27 GMT+2
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.
Comment by Szymon Dziok (b0hoon) - Thursday, 07 October 2010, 12:34 GMT+2
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 |= 0x400 before GPO32_VAL.

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

Loading...