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

Adding support for the Katana60revision2 PCB #10229

Closed
wants to merge 11 commits into from
Closed

Adding support for the Katana60revision2 PCB #10229

wants to merge 11 commits into from

Conversation

Manuel-Indlekofer
Copy link

Description

Currently @rominronin's Katana60rev2 PCB is sold but there is no QMK support yet.
In order to support the new revisions base layout it was required to change the pin assignement of the microcontroller to the key matrix and the diode direction.
To do this i serperated the current folder structure into two subfolders for each revision.
While doing this i tried to modify as little existing config files and code as possible.
I tested the resulting binary with the original Katana60rev2 PCB.

Types of Changes

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

Issues Fixed or Closed by This PR

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).

noroadsleft and others added 11 commits September 2, 2020 21:46
splitting folder structure to support new revisions and adding new config for new pcb layout
As it turns out the katana60rev2 pcb has different column/row key assignments in comparison to rev1
Reverted removal of newlines in original file and removed two empty makefiles because they are not needed
During final testing i noticed that each revision folder needs a makefile even if it is empty
@rominronin
Copy link
Contributor

Hey, thank you for taking the time to put this together.

The revision 2 model supports slightly different bottom rows, so I'm not sure if it's possible to limit Katana60 to a single project folder in qmk (certainly not with my knowledge of it).

I am preparing a pull request for a katana60rev2 keyboard folder.

@Manuel-Indlekofer Manuel-Indlekofer marked this pull request as draft September 6, 2020 09:53
@rominronin
Copy link
Contributor

I created a new pull request with the updated firmware and a couple of layouts, you can find it here: #10255

@Manuel-Indlekofer
Copy link
Author

I created a new pull request with the updated firmware and a couple of layouts, you can find it here: #10255

alrigth,
thanks

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.

3 participants