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

st7789 config and parallel support #821

Merged
merged 4 commits into from
Jan 15, 2024

Conversation

Daft-Freak
Copy link
Collaborator

Mostly because I said you could configure the pins from the config, but that patch wasn't merged yet... Also includes the parallel support and the tufty config and input code so that has a user.

I think I have a refactor of this code into a generic DBI driver somewhere though it may be multiple somewheres... and in other projects 🤔

@Gadgetoid
Copy link
Contributor

Replaced by #827?

@Daft-Freak
Copy link
Collaborator Author

The first commit was, the rest is... less important.

@Gadgetoid
Copy link
Contributor

Hmm, conflicted on some pio headers...

@Daft-Freak
Copy link
Collaborator Author

Addition of SD support probably... My priorities are elsewhere (if anywhere), but I do actually have my tufty now...

@Daft-Freak
Copy link
Collaborator Author

Eh, rebased anyway. (Not tested but nothing should've changed to break it...)

Whoops, was sure I'd done this already
Based on some old hacks
This is what the rotation/8bit changes were for
@Daft-Freak
Copy link
Collaborator Author

Turns out this (and anything else without a config.h) was broken... now it isn't (and has a config.h anyway)

Also I just ran pico3d on it 😄

@Daft-Freak
Copy link
Collaborator Author

Hmm, now I have other changes that conflict with this, but only because they delete a file that this adds 🤔

(Daft-Freak/32blit-beta@pico-st7789-config-parallel...Daft-Freak:32blit-beta:pico-gpio-input)

@Daft-Freak Daft-Freak merged commit 5a2d9ec into 32blit:master Jan 15, 2024
8 checks passed
@Daft-Freak Daft-Freak deleted the pico-st7789-config-parallel branch January 15, 2024 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants