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

PicoSystem: Add DisplayIO to board config #5398

Merged
merged 1 commit into from
Oct 6, 2021

Conversation

Gadgetoid
Copy link

This PR adds DisplayIO bringup to the Pimoroni PicoSystem.

This is raised as a draft with a line in mpconfigboard stubbed out (so checks will hopefully pass), pending the addition of a PicoSystem "stage" and "ugame" to https://github.com/python-ugame/circuitpython-stage/ and a bump to the submodule.

Looks like circuitpython-stage submodule is currently tracking master, so there shouldn't be any implications in bumping it?

@ladyada
Copy link
Member

ladyada commented Sep 27, 2021

cc @deshipu pretty sure we dont want to use gamepad or gamepad shift anymore, replaced with keypad

https://learn.adafruit.com/key-pad-matrix-scanning-in-circuitpython

@Gadgetoid
Copy link
Author

I think "gamepad" is used by ugame- but I'm not quite sure how this all hangs together and what the dependencies are.

It looks like "gamepad shift" is not needed though, as it deals with shift-register based inputs which PicoSystem doesn't have! I'll rebase that out.

@tannewt
Copy link
Member

tannewt commented Sep 27, 2021

I think stage was already switched to keypad: python-ugame/circuitpython-stage@a895b3a

@tannewt tannewt added board New board or update to a single board enhancement labels Sep 27, 2021
* Bring up PicoSystem's display with DisplayIO
* Add stage game library
* Add supporting modules - AUDIOIO and KEYPAD
@Gadgetoid Gadgetoid force-pushed the patch-picosystem-displayio branch from 6612e2d to 2fc6a72 Compare September 30, 2021 12:57
@tannewt
Copy link
Member

tannewt commented Oct 5, 2021

@Gadgetoid Don't forget to mark this as ready when it is.

@Gadgetoid Gadgetoid marked this pull request as ready for review October 6, 2021 09:38
@Gadgetoid
Copy link
Author

@Gadgetoid Don't forget to mark this as ready when it is.

I had checked in a couple of times and completely failed to notice it was still in draft form, thank you!

Copy link
Member

@ladyada ladyada left a comment

Choose a reason for hiding this comment

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

looks ok to me!

@ladyada ladyada merged commit 2186673 into adafruit:main Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board New board or update to a single board enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants