Skip to content
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

drivers: display: st7789v: Add ready time to startup #85023

Merged

Conversation

petejohanson
Copy link
Contributor

Add additional property to allow a configurable delay to the display initialization to allow the st7789v to be ready to receive commands.

Encountered this while working on a keyboard that incorporates an ST7789 display. Initial power up would fail to initialize the display, but upon reset it would initialize fine. Tracked it down to a necessary delay for the device to be fully powered and ready to use. This implementation matches the one found in the SSD1306 display driver.

jilaypandya
jilaypandya previously approved these changes Feb 2, 2025
faxe1008
faxe1008 previously approved these changes Feb 3, 2025
@kartben
Copy link
Collaborator

kartben commented Feb 5, 2025

@petejohanson this needs a rebase, thanks!

Add additional property to allow a configurable delay to the display
initialization to allow the st7789v to be ready to receive commands.

Signed-off-by: Peter Johanson <[email protected]>
@petejohanson petejohanson dismissed stale reviews from faxe1008 and jilaypandya via 18445b3 February 5, 2025 00:29
@petejohanson petejohanson force-pushed the st7789-ready-delay-upstream-pr branch from 23307f7 to 18445b3 Compare February 5, 2025 00:29
@petejohanson
Copy link
Contributor Author

@petejohanson this needs a rebase, thanks!

Done. Thanks.

@kartben kartben merged commit 16b1d3c into zephyrproject-rtos:main Feb 5, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants