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 support for XD004 macro keyboard #6337

Merged
merged 3 commits into from
Jul 15, 2019
Merged

Add support for XD004 macro keyboard #6337

merged 3 commits into from
Jul 15, 2019

Conversation

SidneyBovet
Copy link
Contributor

@SidneyBovet SidneyBovet commented Jul 15, 2019

Description

The default TMK firmware for this board seems to favor underglow lighting instead of tap or oneshot actions. This is why this pull request proposes an alternative to fit on the small flash size (12KB) a firmware that does tapping and oneshot and has no RGB lighting support.

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

  • None

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

The only warning I get when building is that the firmware just fits in the flash, which I assume can't be helped and is ok:

* The firmware size is approaching the maximum - 12098/12288 (98%, 190 bytes free)

@SidneyBovet
Copy link
Contributor Author

Force-pushed to include my actual email address, sorry about that.

const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {

// 0: Base Layer
LAYOUT_all(KC_L, KC_O, KC_V, KC_E),
Copy link
Member

Choose a reason for hiding this comment

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

❤️

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's actually the default layout it shipped with. Mechanical keyboards, spreading the love 😄

keyboards/xd004/keymaps/system_and_media/keymap.c Outdated Show resolved Hide resolved
keyboards/xd004/readme.md Show resolved Hide resolved
keyboards/xd004/v1/rules.mk Show resolved Hide resolved
keyboards/xd004/v1/rules.mk Outdated Show resolved Hide resolved
keyboards/xd004/v1/rules.mk Outdated Show resolved Hide resolved
SidneyBovet and others added 3 commits July 15, 2019 18:38
Also applying the following suggested edits:

Add hardware availability link in readme
Co-Authored-By: Drashna Jaelre <[email protected]>

Enable lite bootmagic
Co-Authored-By: Drashna Jaelre <[email protected]>

Remove commented out MCU
Co-Authored-By: fauxpark <[email protected]>
Correcting usage of tap_code_16 for modified key, thanks to @drashna
@drashna
Copy link
Member

drashna commented Jul 15, 2019

Thanks!

@drashna drashna merged commit a200f19 into qmk:master Jul 15, 2019
doughsay pushed a commit to doughsay/qmk_firmware that referenced this pull request Aug 31, 2019
* Add support for XD004

Also applying the following suggested edits:

Add hardware availability link in readme
Co-Authored-By: Drashna Jaelre <[email protected]>

Enable lite bootmagic
Co-Authored-By: Drashna Jaelre <[email protected]>

Remove commented out MCU
Co-Authored-By: fauxpark <[email protected]>

* Add more ellaborate keymap

Correcting usage of tap_code_16 for modified key, thanks to @drashna

* Add information about bootloader type

Co-Authored-By: Drashna Jaelre <[email protected]>
swanmatch pushed a commit to swanmatch/qmk_firmware that referenced this pull request Sep 3, 2019
* Add support for XD004

Also applying the following suggested edits:

Add hardware availability link in readme
Co-Authored-By: Drashna Jaelre <[email protected]>

Enable lite bootmagic
Co-Authored-By: Drashna Jaelre <[email protected]>

Remove commented out MCU
Co-Authored-By: fauxpark <[email protected]>

* Add more ellaborate keymap

Correcting usage of tap_code_16 for modified key, thanks to @drashna

* Add information about bootloader type

Co-Authored-By: Drashna Jaelre <[email protected]>
ripxorip pushed a commit to ripxorip/qmk_firmware that referenced this pull request Dec 3, 2019
* Add support for XD004

Also applying the following suggested edits:

Add hardware availability link in readme
Co-Authored-By: Drashna Jaelre <[email protected]>

Enable lite bootmagic
Co-Authored-By: Drashna Jaelre <[email protected]>

Remove commented out MCU
Co-Authored-By: fauxpark <[email protected]>

* Add more ellaborate keymap

Correcting usage of tap_code_16 for modified key, thanks to @drashna

* Add information about bootloader type

Co-Authored-By: Drashna Jaelre <[email protected]>
ridingqwerty pushed a commit to ridingqwerty/qmk_firmware that referenced this pull request Jan 10, 2020
* Add support for XD004

Also applying the following suggested edits:

Add hardware availability link in readme
Co-Authored-By: Drashna Jaelre <[email protected]>

Enable lite bootmagic
Co-Authored-By: Drashna Jaelre <[email protected]>

Remove commented out MCU
Co-Authored-By: fauxpark <[email protected]>

* Add more ellaborate keymap

Correcting usage of tap_code_16 for modified key, thanks to @drashna

* Add information about bootloader type

Co-Authored-By: Drashna Jaelre <[email protected]>
swamp09 pushed a commit to swamp09/qmk_firmware that referenced this pull request Mar 11, 2020
* Add support for XD004

Also applying the following suggested edits:

Add hardware availability link in readme
Co-Authored-By: Drashna Jaelre <[email protected]>

Enable lite bootmagic
Co-Authored-By: Drashna Jaelre <[email protected]>

Remove commented out MCU
Co-Authored-By: fauxpark <[email protected]>

* Add more ellaborate keymap

Correcting usage of tap_code_16 for modified key, thanks to @drashna

* Add information about bootloader type

Co-Authored-By: Drashna Jaelre <[email protected]>
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
* Add support for XD004

Also applying the following suggested edits:

Add hardware availability link in readme
Co-Authored-By: Drashna Jaelre <[email protected]>

Enable lite bootmagic
Co-Authored-By: Drashna Jaelre <[email protected]>

Remove commented out MCU
Co-Authored-By: fauxpark <[email protected]>

* Add more ellaborate keymap

Correcting usage of tap_code_16 for modified key, thanks to @drashna

* Add information about bootloader type

Co-Authored-By: Drashna Jaelre <[email protected]>
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.

3 participants