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

Add new example keyboard, support for a new TRS socket, and minor fixes to the MCU generation. #20

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

coolsa
Copy link

@coolsa coolsa commented Jun 21, 2023

Thank you for this project! I used it last year to make a keyboard, and was always intending to add the changes to the upstream.

There was a minor bug with the MCU pin offset, but now that's accounted for and I can use my weird off brand pro-micro clones.

I also added support for a TRS socket, the PJ-324M, since that's what I had on hand at the time of construction.

And finally there's the new keyboard, which is a split preonic clone. I had to make some significant changes to the parameters.scad file, so I just moved the altered file to the example folder. I did keep the commented TRS socket settings in the original file, just for future reference.

Let me know if there are any changes I should make, and hopefully this PR is welcome.

Thanks again!

coolsa and others added 4 commits May 4, 2022 17:27
This socket may be only a TRS, but QMK does say that it should still work for split keyboards.
I've got a non-standard board, so need to use that parameter.
- New TRS socket support added
- MCU can now support offset pins
@50an6xy06r6n
Copy link
Owner

Hey thanks for the contribution! I'm starting to work on this a little bit, starting with some refactors to improve modularity. Once I have those done I'll probably come in and rebase your changes on top of that.

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