Skip to content

Rework displays#1476

Merged
dhalbert merged 20 commits into
adafruit:masterfrom
tannewt:rework_displays
Jan 21, 2019
Merged

Rework displays#1476
dhalbert merged 20 commits into
adafruit:masterfrom
tannewt:rework_displays

Conversation

@tannewt
Copy link
Copy Markdown
Member

@tannewt tannewt commented Jan 18, 2019

This makes displays much more accessible by allowing them to be dynamically created. They have unusual lifetimes because they live through reloads so they can display error messages. To release them you must call displayio.release_displays().

This PR also splits the display bus mechanics from the display commands and adds 8 bit parallel bus support for faster pixel transmission.

@tannewt tannewt added this to the 4.0 Beta milestone Jan 18, 2019
@tannewt tannewt requested a review from dhalbert January 18, 2019 22:00
@siddacious
Copy link
Copy Markdown

👍 👍 👍 📺

Copy link
Copy Markdown
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lotsa comments but looks great! Thanks for cleaning up FourWire to use standard SPI, and great job refactoring to shared code!

Comment thread ports/nrf/common-hal/displayio/ParallelBus.c
Comment thread ports/nrf/common-hal/displayio/ParallelBus.c Outdated
Comment thread shared-bindings/displayio/Display.c Outdated
Comment thread shared-bindings/displayio/Display.c Outdated
Comment thread shared-bindings/displayio/Display.c
Comment thread shared-module/displayio/__init__.c
Comment thread supervisor/shared/board_busses.c
Comment thread supervisor/shared/board_busses.c
Comment thread supervisor/shared/board_busses.c
Comment thread shared-bindings/displayio/Display.c
@tannewt
Copy link
Copy Markdown
Member Author

tannewt commented Jan 19, 2019

Ok, please take another look.

Copy link
Copy Markdown
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just the one width/height thing.

Comment thread shared-bindings/displayio/Display.c
@tannewt
Copy link
Copy Markdown
Member Author

tannewt commented Jan 21, 2019

@dhalbert Build is green now.

Copy link
Copy Markdown
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Travis bingo!

------ ------ ------ ------ ------
❌ 2426.1 ❌ 2426.2 ❌ 2426.3 ❌ 2426.4 ❌ 2426.5
- - - - -
- - FREE - -
- - - - -
------ ------ ------ ------ ------

@dhalbert dhalbert merged commit 6b88d0f into adafruit:master Jan 21, 2019
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.

3 participants