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

Kscan cleanup #1269

Merged
merged 2 commits into from Apr 26, 2022
Merged

Kscan cleanup #1269

merged 2 commits into from Apr 26, 2022

Conversation

joelspadin
Copy link
Collaborator

Switched the matrix kscan driver over to use the GPIO DT spec macros and updated Kconfig to automatically enable each driver based on DT nodes.

I didn't update the direct and demux drivers similarly because those are still defining functions with macros, and I figured it would be a lot easier to review as separate PRs that align them with how the matrix driver now works.

The key scanning drivers are now automatically enabled when a DT node
with the matching "compatible" property is present and enabled, so they
no longer need to be manually set for each board.
@joelspadin
Copy link
Collaborator Author

I've tested this on a board that uses the composite, matrix, and direct GPIO drivers. I don't have anything that uses the demux driver though.

Copy link
Contributor

@petejohanson petejohanson left a comment

Choose a reason for hiding this comment

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

Perfect. Thanks for the clean up.

@petejohanson petejohanson merged commit 6d6efa3 into zmkfirmware:main Apr 26, 2022
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request May 2, 2022
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request May 2, 2022
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request May 2, 2022
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request May 3, 2022
…or Functions

Updates behaviors to compile only those explicitly used in a keymap, similarly to zmkfirmware#1269.
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request May 3, 2022
…or Functions

Updates behaviors to compile only those explicitly used in a keymap, similarly to zmkfirmware#1269.
@cgoates cgoates mentioned this pull request May 12, 2022
4 tasks
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request Jun 12, 2022
…or Functions

Updates behaviors to compile only those explicitly used in a keymap, similarly to zmkfirmware#1269.
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request Jun 18, 2022
Updates behaviors to compile only those explicitly used in a keymap, similarly to zmkfirmware#1269.
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request Jun 18, 2022
Updates behaviors to compile only those explicitly used in a keymap, similarly to zmkfirmware#1269.
kurtis-lew added a commit to kurtis-lew/zmk that referenced this pull request Jun 18, 2022
Updates behaviors to compile only those explicitly used in a keymap, similarly to zmkfirmware#1269.
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.

None yet

2 participants