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 AL1 keyboard to QMK #3049
Add AL1 keyboard to QMK #3049
Conversation
olivia
commented
May 25, 2018
•
edited
edited
- Move over folder from AL1 Repo
- Add instructions to flash since default keymap is missing reset key
- Add better keymap with reset key
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think QMK already has support for all the custom code in matrix.c. You could try removing that file and see if things still work out for you. Unless of course the AL1 does indeed use some custom matrix stuff.
DDRD |= (1<<0) | (1<<1); | ||
|
||
|
||
#define KEYMAP( \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you rename KEYMAP
to LAYOUT_all
in this .h file and in the keymap.c files that reference it?
{ K500, K501, K502, K503, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO, KC_NO} \ | ||
} | ||
|
||
#define KEYMAP_SPLITBS( \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you rename this to LAYOUT_split_bs
and in the keymap.c files that reference it?
@@ -0,0 +1,56 @@ | |||
#include "al1.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please replace #include "all.h
with #include QMK_KEYBOARD_H
#include "al1.h" | ||
|
||
const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = { | ||
[0] = KEYMAP(\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change KEYMAP
to LAYOUT_all
and its other references in the file
@@ -0,0 +1,56 @@ | |||
#include "al1.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change to #include QMK_KEYBOARD_H
#include "al1.h" | ||
|
||
const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = { | ||
[0] = KEYMAP_SPLITBS(\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change to LAYOUT_split_bs
and its other references in this file too!
Is the custom matrix necessary here? If not, then it may be best to use the default one. As for the bootloader, if you have a "RESET" code in the keymap, you should be okay. |
|
||
OPT_DEFS += -DINTERRUPT_CONTROL_ENDPOINT | ||
|
||
OPT_DEFS += -DBOOTLOADER_SIZE=4096 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we know what controller is being used here?
Eg, is this a Pro Micro, Teensy 2.0, other, or SMD?
Could you make sure that the board has a readme? |
#3330 supersedes this |