-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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 support for the Preonic rev3 board to ZMK #228
Conversation
…irmware#202 before it can be tested.
…irmware#202 before it can be tested.
…irmware#202 before it can be tested.
…lly written using the Zephyr macros, when a simple #if !defined() would have sufficed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple minor comments, and I believe the clang-format CI job may be failing due to issue with when you forked main
. Can you rebase or merge up main
to try to get that green again? Thanks!
@@ -51,6 +51,7 @@ static int kscan_gpio_config_interrupts(struct device **devices, | |||
} | |||
#endif | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I follow the question. Can you elaborate?
Add ZMK copyright headers where missing. Add BT_ bindings for testing purposes in keymap.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One last lingering keymap question, then we should get this in!
// | | | | | | | | | | | | | | ||
bindings = < | ||
&kp F1 &kp F2 &kp F3 &kp F4 &kp F5 &kp F6 &kp F7 &kp F8 &kp F9 &kp F10 &kp F11 &kp F12 | ||
&bt BT_SEL 0 &bt BT_SEL 1 &bt BT_SEL 2 &bt BT_SEL 3 &bt BT_SEL 4 &trans &trans &trans &trans &trans &trans &kp DEL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding this... At the same time... There's no way to get to this adjust layer (yet, I'm still brainstorming how best to handle this common pattern). Can we put this on raise
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shurikai Did you see this comment? TIA.
Is this PR dead? |
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into #228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
@shurikai If still interested in getting this in, can you please see the one comment, and add a ZMK metadata file, so we can get this merged? Closing for now, but feel free to reopen. |
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
My Planck rev 6.1 would not register any keys without "col2row". I also had to apply a matrix_transform in order to move them to the correct position. Peeking into zmkfirmware#228 helped for finding this. I also add transforms for the 2u spacebar layouts.
This pull request is to add support for the Preonic rev 3. The preonic is a board with built in MCU, and not a shield. The included default keymap is based on the one found on the QMK configurator, but will not be fully functional until shifted keycodes are available.
To build:
west build -b preonic_rev3
Note that there is one file touched below (kscan_gpio_matrix.c) that was strictly a change to line formatting in the C file. Nothing changed functionally.