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 splitkb Kyria rev. 2 #16178

Merged
merged 1 commit into from
Feb 21, 2022
Merged

Add splitkb Kyria rev. 2 #16178

merged 1 commit into from
Feb 21, 2022

Conversation

Crote
Copy link

@Crote Crote commented Feb 3, 2022

This commit adds the code to support the newest revision of splitkb's Kyria.

Description

Compared to the Kyria 1.x, the 2.x series features the following significant changes:

  • Handedness by matrix pin
  • Reworked matrix wiring
  • Reworked WS2812B ordering

This means a new software revision is required as those make it backwards incompatible.
The code is pretty much a copy/paste from rev1, with minor changes where neccesary.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).
    Note: make keyboard:all returned firmware size errors, but those were already present before this commit.
    Note: Proton-C support has not been tested as we do not have a pair of controllers at hand.

@drashna drashna requested a review from a team February 11, 2022 15:24
@splitkb
Copy link
Contributor

splitkb commented Feb 15, 2022

Is there anything we can do to get this pull request merged into the main branch?

@fauxpark
Copy link
Member

Let's wait for #16377 to be merged, and if it works, the same change can be applied here.

@splitkb
Copy link
Contributor

splitkb commented Feb 20, 2022

Requested changes have been applied, please let us know if there's anything else that should be patched up :)

@fauxpark fauxpark requested a review from a team February 20, 2022 05:25
@drashna drashna merged commit 14fb3cc into qmk:master Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants