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

Refactor Equinox PCB revisions #8266

Merged
merged 2 commits into from
Mar 10, 2020
Merged

Conversation

wilba
Copy link
Contributor

@wilba wilba commented Feb 29, 2020

Description

Minor refactoring of how the PCB revisions are handled, the specifics for the prototype (rev0) version were in proto_via keymap, which meant you couldn't build a non-VIA version, and also couldn't build both VIA builds using make all:via.

This also technically fixes a bug in the future that will happen when we switch on backlight support in VIA, since the proto_via build would not have the VIA support for backlight and VIA would get grumpy and blue screen. Now the rev0 build still has backlight enabled, it's just driving a disconnected pin.

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

@drashna drashna requested a review from a team March 1, 2020 00:28
@drashna drashna requested a review from a team March 4, 2020 09:12
Copy link
Member

@noroadsleft noroadsleft left a comment

Choose a reason for hiding this comment

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

Everything else looks good to me.

keyboards/ai03/equinox/rev0/rules.mk Outdated Show resolved Hide resolved
@drashna drashna requested a review from a team March 7, 2020 00:08
@wilba
Copy link
Contributor Author

wilba commented Mar 10, 2020

fixed

@noroadsleft noroadsleft merged commit 2a8ccaf into qmk:master Mar 10, 2020
@noroadsleft
Copy link
Member

Thanks!

Shinichi-Ohki added a commit to Shinichi-Ohki/qmk_firmware that referenced this pull request Mar 11, 2020
* 'master' of https://github.com/qmk/qmk_firmware: (107 commits)
  Add VIA support to Tokyo60. (qmk#8362)
  fix kira80 matrix mapping and add more preset layouts (qmk#8361)
  Rename qmk json-keymap to qmk json2c (qmk#8372)
  format code according to conventions [skip ci]
  Feature: RGBLight layers (qmk#7768)
  Refactor Equinox PCB revisions (qmk#8266)
  Replace tab indentation with spaces in is0 default keymap. (qmk#8368)
  Fix layer debug calls (qmk#8370)
  [Keymap] Added HHKB Keymap and user config (qmk#8119)
  Refactor rgblight_reconfig.h (qmk#7773)
  [Keyboard] Palette1202 (qmk#7736)
  [Keyboard] Helix add split common option (qmk#7718)
  Enable bootmagic lite for CU80 (qmk#8363)
  [Keymap] Update personal keymap (qmk#8354)
  Remove NO_UART defines from config.h for V-USB boards (qmk#8351)
  [Keyboard] Add Kira 80 support (qmk#8342)
  [Keyboard] Add PRKL30 keyboard (qmk#8319)
  Update Dvorak, Colemak and Workman keycode aliases (qmk#8217)
  Move splittest I2C (qmk#8346)
  [Keymap] planck layout for kifinnsson (qmk#8324)
  ...
@wilba wilba mentioned this pull request Mar 21, 2020
13 tasks
c0psrul3 pushed a commit to c0psrul3/qmk_firmware that referenced this pull request Mar 23, 2020
* Refactor Equinox PCB revisions

* typo
sowbug pushed a commit to sowbug/qmk_firmware that referenced this pull request Apr 2, 2020
* Refactor Equinox PCB revisions

* typo
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Apr 10, 2020
* Refactor Equinox PCB revisions

* typo
kylekuj pushed a commit to kylekuj/qmk_firmware that referenced this pull request Apr 21, 2020
* Refactor Equinox PCB revisions

* typo
jakobaa pushed a commit to jakobaa/qmk_firmware that referenced this pull request Jul 7, 2020
* Refactor Equinox PCB revisions

* typo
jakeisnt pushed a commit to jakeisnt/qmk_firmware that referenced this pull request Aug 20, 2020
* Refactor Equinox PCB revisions

* typo
@wilba wilba deleted the equinox_refactor_pr branch March 23, 2021 23:10
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.

4 participants