Rockbox

Tasklist

FS#9569 - fix for e200v2 lcd_update_rect

Attached to Project: Rockbox
Opened by Michael Chicoine (mc2739) - Saturday, 22 November 2008, 01:22 GMT
Last edited by Dave Chapman (linuxstb) - Tuesday, 16 December 2008, 07:52 GMT
Task Type Patches
Category LCD
Status Closed
Assigned To No-one
Operating System Another
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch corrects an issue with lcd_update_rect in lcd-e200v2.c

Also, an unnecessary setting for the button light is removed since this function is in backlight-e200v2-fuze.c
This task depends upon

Closed by  Dave Chapman (linuxstb)
Tuesday, 16 December 2008, 07:52 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed as r19453
Comment by Thomas Martitz (kugel.) - Saturday, 22 November 2008, 16:34 GMT
Do the same for fuze as well.

The OF seems to set that GPIO at LCD init though. My guess is now, that due to the inability to configure the button light in the OF, that the OF just turns on the button light always with the lcd and/or backlight.
Comment by Thomas Martitz (kugel.) - Saturday, 22 November 2008, 16:39 GMT
Oops, forgot the lcd_update_rect part.

linuxstb should have a look at that before committing, as he wrote the fuze lcd driver which seems to have some similarities with the e200v2 one.
Comment by Dave Chapman (linuxstb) - Saturday, 22 November 2008, 17:48 GMT
Michael,

What "issue" is the use of lcd_framebuffer[x][y] fixing? It doesn't make sense to access the lcd framebuffer that way - it's defined to be [y][x].

Comment by Dave Chapman (linuxstb) - Saturday, 22 November 2008, 18:04 GMT
I've just committed the GPIOD part of this patch (for both Fuze and e200v2) to SVN.
Comment by Michael Chicoine (mc2739) - Saturday, 22 November 2008, 18:20 GMT
Dave, without this change, on display of the main menu, the display is shifted right when the status bar is on instead of being shifting down. This was the only change that I found that would display the menu properly.
Comment by Michael Chicoine (mc2739) - Saturday, 29 November 2008, 17:32 GMT
Do not modify the CCU_IO register since we don't use GPIOD - same change as made in r19257 for Sansa Fuze
Comment by Jonathan Gordon (jdgordon) - Sunday, 30 November 2008, 00:36 GMT
the above patch has been commited.
Comment by Michael Chicoine (mc2739) - Monday, 01 December 2008, 19:54 GMT
This patch corrects lcd_update_rect properly. Tested with credits.rock which uses the lcd_update_rect function.

Loading...