-
Notifications
You must be signed in to change notification settings - Fork 21
Initial Working Driver #1
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 8000 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
Conversation
Ok, now that my changes were merged into CP, this driver is ready to be merged (after review of course). :) |
Thanks for testing @deshipu, I'll take another look at this tonight. |
@deshipu, I just tested on Beta 6 and it worked fine. I don't have an ItsyBitsy M0 Express, but I hooked up my ItsyBitsy M4 Express the same way you did. Here's the code I used (with the IO pins adjusted)
|
I just noticed this has been stale for almost 3 weeks. @deshipu, could I please get you to take another look at this? Thanks |
I tried again with your pin assignment, and this time it worked. |
import displayio | ||
from adafruit_st7789 import ST7789 | ||
|
||
spi = board.SPI() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you configure the SPI to higher speed than the default? It's painfully slow. I think it would be best to configure it to display's maximum.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. It looks like that's been added since I made this. Could you remind me of the syntax to change it and the speed you think would work better? Were you thinking 24MHz?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found something on discord and used that to set it to 24MHz.
Looks and works great, thank you! |
Here's the driver for the ST7789. However, it is dependent on a new feature in CircuitPython provided by adafruit/circuitpython#1685.