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

Initial commit for Prime_E w/ VIA config support #5285

Merged
merged 23 commits into from
Mar 2, 2019
Merged

Initial commit for Prime_E w/ VIA config support #5285

merged 23 commits into from
Mar 2, 2019

Conversation

holtenc
Copy link
Contributor

@holtenc holtenc commented Mar 1, 2019

Description

Initial commit for Prime_E with support for VIA configurator. Additionally added VIA configurator support to /handwired/prime_exl for personal use.

Discussed this addition with @Wilba6582 and he has appoved it. (https://github.com/olivia/via-config/pull/52)

Honestly not sure why the files in lib/lufa and ...projects are part of this PR. I've not made any changes to them. Is there a way to exclude them?

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

Copy link
Member

@drashna drashna left a comment

Choose a reason for hiding this comment

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

Mostly, it looks fine.

However, it looks like you have a number of lib files that have been changed. This should take care of thos files:

git checkout - lib

Also, there are a number of things that have changed recently.

  • We've set the default for IS_COMMAND, so there no need to specify it, if you're using the default.
  • GPIO commands are hardware agnostic, and work on AVR and ARM. And we're trying to switch to these, on all boards, so it's consistent.

If you need any help with these changes, or any clarification about them, don't hesitate to ask.

Also, I know VIA support has been put on hold. If @Wilba6582 signs off on this, here, then we'll merge it.

@@ -0,0 +1 @@
7e58a720d521047f73ccc9b7cabe3901380feb6d
Copy link
Member

Choose a reason for hiding this comment

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

Could this be reverted?

You can run:

git checkout -- lib/

That should fix all of these.

drashna and others added 15 commits March 1, 2019 11:48
@holtenc
Copy link
Contributor Author

holtenc commented Mar 1, 2019

I think that should resolve all your requested changes.

@drashna
Copy link
Member

drashna commented Mar 1, 2019

Looks good. Though, the lib stuff is still there, unfortunately.

I'm not 100% sure how to fix that (I'm not git expert, either).... but I think you can do git reset upstream/master lib to remove those changes. --hard will remove them completely, but since I'm not 100% sure, so without should be safer.

That's assuming you have an upstream set. If not git remote add upstream https://github.com/qmk/qmk_firmware.git will add it, and then you should be able to do this.

@holtenc
Copy link
Contributor Author

holtenc commented Mar 2, 2019

OK. got those Lib files removed. Sorry.

@drashna
Copy link
Member

drashna commented Mar 2, 2019

Awesome! Thanks!

And no worries. Stuff like that happens from time to time. Git is some serious dark magic sometimes. :)

@drashna
Copy link
Member

drashna commented Mar 2, 2019

And @Wilba6582, just wanted to get your okay before merging this.

@wilba
Copy link
Contributor

wilba commented Mar 2, 2019

@drashna yes, this was in the works before the cutoff

@drashna
Copy link
Member

drashna commented Mar 2, 2019

Awesome, just wanted to make sure!

@drashna drashna merged commit 6e48ea0 into qmk:master Mar 2, 2019
bl1nk added a commit to bl1nk/qmk_firmware that referenced this pull request Mar 3, 2019
* upstream/master: (58 commits)
  [Keymap] layout/community/ortho4x12/symbolic update (qmk#5274)
  [Keyboard] matrix_init_user shouldn't call itself (qmk#5298)
  [Keymap] Update keymap.c (qmk#5290)
  Update Atreus layout to match the Niu layout. Fix Niu comment typo. (qmk#5297)
  [Keymap] Fix comment in the KBD67 default keymap. (qmk#5296)
  [Keyboard] update Hecomi/alpha's info.json  (qmk#5295)
  [Keyboard] Adding Numeric Keypad //e converter (qmk#5294)
  [Keyboard] Add support for matrix_scan_user to usb-usb converter (qmk#5291)
  handwired/terminus_mini: refactor, Configurator support, and readme cleanup (qmk#5292)
  [Keyboard] Initial commit for Prime_E w/ VIA config support (qmk#5285)
  [Keyboard] add dz60rgb ansi and hhkb support; add dz40rgb (qmk#5231)
  [Keyboard] Convert mf68_ble to keymap (qmk#5287)
  [Keymap] Added INSERT to my keymaps (qmk#5284)
  HS60 v2: Instructions how to reset and flash (qmk#5241)
  [Keymap] unausgeschlafen quefrency keymap (qmk#5273)
  [Keymap] Fix rgblight_enable ifdefs (qmk#5283)
  [Keymap] DCompact Layout Updates pt. 3 (qmk#5209)
  [Keyboard] Add Dozen0 (qmk#5272)
  [Keymap] update: add extrakey support for macOS media keys (qmk#5263)
  [Keymap] My TMO50 layout (qmk#5116)
  ...
slugger7 pushed a commit to slugger7/qmk_firmware that referenced this pull request Mar 7, 2019
* Initial Prime_E commit

* Via support for Prime_EXL

* Backlight testing

* update libs

* Clean up and addition of VIA compatibility.

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* removed IS_COMMAND from config.h. updated GPIO commands

* clean up libs
zer09 pushed a commit to zer09/qmk_firmware that referenced this pull request Mar 18, 2019
* Initial Prime_E commit

* Via support for Prime_EXL

* Backlight testing

* update libs

* Clean up and addition of VIA compatibility.

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* removed IS_COMMAND from config.h. updated GPIO commands

* clean up libs
chie4hao pushed a commit to chie4hao/qmk_firmware that referenced this pull request Mar 28, 2019
* Initial Prime_E commit

* Via support for Prime_EXL

* Backlight testing

* update libs

* Clean up and addition of VIA compatibility.

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* removed IS_COMMAND from config.h. updated GPIO commands

* clean up libs
slugger7 pushed a commit to slugger7/qmk_firmware that referenced this pull request Apr 3, 2019
* Initial Prime_E commit

* Via support for Prime_EXL

* Backlight testing

* update libs

* Clean up and addition of VIA compatibility.

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* removed IS_COMMAND from config.h. updated GPIO commands

* clean up libs
dlhextall pushed a commit to dlhextall/qmk_firmware that referenced this pull request May 24, 2019
* Initial Prime_E commit

* Via support for Prime_EXL

* Backlight testing

* update libs

* Clean up and addition of VIA compatibility.

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* removed IS_COMMAND from config.h. updated GPIO commands

* clean up libs
Timbus pushed a commit to Timbus/qmk_firmware that referenced this pull request Jun 23, 2019
* Initial Prime_E commit

* Via support for Prime_EXL

* Backlight testing

* update libs

* Clean up and addition of VIA compatibility.

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/handwired/prime_exl/keymaps/via/keymap.c

Co-Authored-By: holtenc <[email protected]>

* Update keyboards/primekb/prime_e/keymaps/default/keymap.c

Co-Authored-By: holtenc <[email protected]>

* removed IS_COMMAND from config.h. updated GPIO commands

* clean up libs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants