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

Added keyboard KPRepublic BM60V2EC #16866

Closed
wants to merge 13 commits into from
Closed

Conversation

LuisGit9
Copy link

@LuisGit9 LuisGit9 commented Apr 17, 2022

Added support for the KPRepublic BM60 with Encoder, Revision 2 based on the code for the BM60HSRGB Rev 2

Description

Copied the kprepublic/bm60hsrgb directory and added support for the encoder as a button and rotation in both direction, assigning it as a mouse wheel and volumen control

Types of Changes

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

Issues Fixed or Closed by This PR

Checklist

  • [ x ] My code follows the code style of this project: C, Python
  • [ x ] 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.
  • [ x ] I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • [ x ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Copy link
Author

@LuisGit9 LuisGit9 left a comment

Choose a reason for hiding this comment

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

Some questions about two of the suggested changes.

LuisGit9 and others added 3 commits April 21, 2022 16:14
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Copy link
Author

@LuisGit9 LuisGit9 left a comment

Choose a reason for hiding this comment

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

Changes applied.

@LuisGit9 LuisGit9 requested a review from drashna April 22, 2022 01:11
Co-authored-by: Drashna Jaelre <drashna@live.com>
Copy link
Author

@LuisGit9 LuisGit9 left a comment

Choose a reason for hiding this comment

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

Accepted change.

@LuisGit9 LuisGit9 requested a review from drashna April 24, 2022 00:25
Tested, accepted.

Co-authored-by: Drashna Jaelre <drashna@live.com>
Copy link
Author

@LuisGit9 LuisGit9 left a comment

Choose a reason for hiding this comment

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

Change accepted.

@LuisGit9 LuisGit9 requested a review from drashna April 29, 2022 01:16
@Didel
Copy link
Contributor

Didel commented May 24, 2022

It looks like this PR is similar to (the older) #15104 . As #15104 adds both v1 and v2 support, I looks like it will add the same functionality as this PR with support for more revisions of the board...

@drashna drashna requested review from a team and drashna July 11, 2022 20:01
@github-actions
Copy link

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@github-actions github-actions bot added the stale Issues or pull requests that have become inactive without resolution. label Aug 26, 2022
Copy link
Contributor

@keyboard-magpie keyboard-magpie left a comment

Choose a reason for hiding this comment

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

Changes to bring in line with current data driven status, and recent breaking changes

KC_LCTL, KC_LGUI, KC_LALT, KC_SPC, KC_RALT, MO(1), KC_LEFT, KC_DOWN, KC_RGHT
),
[1] = LAYOUT_60_ansi_arrow(
_______, KC_F1, KC_F2, KC_F3, KC_F4, KC_F5, KC_F6, KC_F7, KC_F8, KC_F9, KC_F10, KC_F11, KC_F12, RESET,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
_______, KC_F1, KC_F2, KC_F3, KC_F4, KC_F5, KC_F6, KC_F7, KC_F8, KC_F9, KC_F10, KC_F11, KC_F12, RESET,
_______, KC_F1, KC_F2, KC_F3, KC_F4, KC_F5, KC_F6, KC_F7, KC_F8, KC_F9, KC_F10, KC_F11, KC_F12, QK_BOOT,

Comment on lines +21 to +25
#define VENDOR_ID 0x4B50
#define PRODUCT_ID 0x1124
#define DEVICE_VER 0x0002
#define MANUFACTURER KP Republic
#define PRODUCT BM60V2EC
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
#define VENDOR_ID 0x4B50
#define PRODUCT_ID 0x1124
#define DEVICE_VER 0x0002
#define MANUFACTURER KP Republic
#define PRODUCT BM60V2EC

"keyboard_name": "BM60V2EC",
"url": "https://kprepublic.com/products/bm60ec-bm60-ec-rgb-60-gh60-hot-swappable-custom-mechanical-keyboard-pcb-programmed-qmk-via-rgb-switch-type-c-rotary-knob",
"maintainer": "luisgit9",
"layouts": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"layouts": {
"usb": {
"vid": "0x4B50",
"pid": "0x1124",
"device_version": "0.0.2"
},
"layouts": {

@@ -0,0 +1,79 @@
{
"keyboard_name": "BM60V2EC",
"url": "https://kprepublic.com/products/bm60ec-bm60-ec-rgb-60-gh60-hot-swappable-custom-mechanical-keyboard-pcb-programmed-qmk-via-rgb-switch-type-c-rotary-knob",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"url": "https://kprepublic.com/products/bm60ec-bm60-ec-rgb-60-gh60-hot-swappable-custom-mechanical-keyboard-pcb-programmed-qmk-via-rgb-switch-type-c-rotary-knob",
"manufacturer": "kprepublic",
"url": "https://kprepublic.com/products/bm60ec-bm60-ec-rgb-60-gh60-hot-swappable-custom-mechanical-keyboard-pcb-programmed-qmk-via-rgb-switch-type-c-rotary-knob",

@github-actions github-actions bot removed the stale Issues or pull requests that have become inactive without resolution. label Sep 1, 2022
@github-actions
Copy link

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@github-actions github-actions bot added the stale Issues or pull requests that have become inactive without resolution. label Oct 17, 2022
@github-actions
Copy link

Thank you for your contribution!
This pull request has been automatically closed because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, or re-open when it's ready.
// [stale-action-closed]

@github-actions github-actions bot closed this Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keyboard keymap stale Issues or pull requests that have become inactive without resolution.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants