-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix display character handling #4
Comments
I've fixed the alignment/wrapping on the display. Whoever implemented the LCD code did not appear to understand the internal VRAM mapping of these displays. Also fixed some performance problems that suppressed a ridiculous frequency of display redraws. Now it actually only updates if it is flagged as dirty. |
That looks so cool! Well done! |
@vintagepc how do you plan to add brightness? |
I'll fade the background color to black as brightness goes down. Doesn't have to be perfect, just a visual indication so we can test it in sim. :) |
Brightness has been implemented. (in theory) It's wired to the pin but probably not passing the initial brightness check yet. I think we can close this initial hardware bringup task. |
Tie the pin low during detection and it will always pass as Backlight enabled. |
Hmmm... what about that last custom character? It is the wrong one. Are these characters generated at runtime or is it just a font file that includes them? |
They are based on what we write to CG RAM (i.e. I implemented CGRAM support). I think there's still some unimplemented/wrong behaviour with the display somewhere in regards to blanking and screen updates. I haven't dived in as that's less of a roadblock to further activity; but I figured a 95% functional display is pretty critical until we resolve #3 . Quite possible It's kicking the cursor on an update somewhere and clobbering that character when we write a new one. |
There are a few issues with the stock display implementation:
The text was updated successfully, but these errors were encountered: