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

BM40HSRGB Support[Feature Request] #9463

Closed
Fenlair opened this issue Jun 18, 2020 · 4 comments · Fixed by #9860 or #10044
Closed

BM40HSRGB Support[Feature Request] #9463

Fenlair opened this issue Jun 18, 2020 · 4 comments · Fixed by #9860 or #10044

Comments

@Fenlair
Copy link

Fenlair commented Jun 18, 2020

I would like to add support for the KPRepublic Hotswap BM40RGB PCB.

Description

The BM40RGB has kbfirmware support. I'd like to port it to qmk firmware. My plan was to use https://noroadsleft.github.io/kbf_qmk_converter/, if needed fix it up so it passes the coding guidlines and add a default keymap.

Would that be acceptable?

@sigprof
Copy link
Contributor

sigprof commented Jun 19, 2020

This is a perfectly fine way to add support for a PCB to QMK — I did exactly this when adding the ID80 support (although that PR is probably more complicated than usual, because it also had some changes to optimize the somewhat unusual matrix layout, which hopefully won't be needed for BM40RGB).

One part which could bring some difficulties is that, unlike my ID80 example, BM40RGB has per-switch RGB backlight, for which the RGB Matrix feature should be used (more common underglow LEDs are usually handled using RGB Lighting instead). Configuring the rgbmatrix code could be complicated; you can look at this example of adding the BM60RGB support.

@Fenlair
Copy link
Author

Fenlair commented Jun 21, 2020

Thanks for the info. I set everything up and it compiles, but especially for the RGB Matrix I need the PCB first to test it.
I will get a bm40rgb in a month or two and I plan to do a pull request when I tested my patch.

@sigprof
Copy link
Contributor

sigprof commented Jun 25, 2020

I just realized that the Reddit link about BM60RGB that I provided does not point to the actual video (it is just a link to the Twitch channel, and that particular video is not even available in archive). See Porting with Ports: BM60 RGB on Youtube instead.

@Fenlair
Copy link
Author

Fenlair commented Jun 25, 2020

no worries I found it without an issue after I knew it existed :)

Btw if someone owns a bm40rgb and doesn't want to wait for 2 months, send me a message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants