-
-
Notifications
You must be signed in to change notification settings - Fork 39.9k
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
Manufacturers distributing without source code -- wireless tri-mode boards, missing functionality such as HE matrix etc. #24085
Comments
For clarity, the vendor ID is Westberry Technology, which is also in use by Epomaker, KiiBOOM and others. Manufacturers other than Royal Kludge are also going to be put on hold for this reason -- the third-party firmware provider must supply source code for all QMK-based boards. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
#23982 |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment has been minimized.
This comment has been minimized.
This comment was marked as resolved.
This comment was marked as resolved.
Add Chosfox CF81 that slipped in last year. Clearly advertised as a tri-mode alexotos just did a review on their Fox65 and that will be the same if that comes up. |
In connection to this, Here are boards that are merged, w/o required code disclosure:
Then there are muddy cases that I'm not 100% sure about, since there are versions with added letters in the name that are wireless, but may not share the same base FW, like:
Comment for tracking purposes for the time being. |
https://switchandclick.com/chosfox-cf81/ suggests a non pro version exists, which is not wireless. |
You're correct. I suppose the question is: for transgressors now/future will their priors be removed? or rather, should they be to force compliance? |
This comment was marked as resolved.
This comment was marked as resolved.
MyKeyClub / IRISLAB may be another candidate for this list. I own the "QMK" versions (wired-only ones) of 2 of their boards: Their code is anywhere to be found in this repository: https://github.com/qmk/qmk_firmware/tree/master/keyboards/mykeyclub The only available code is for the JRIS65 model and its README file contains an interesting disclaimer:
I do not expect to see any code coming from MyKeyClub nor IRISLAB anytime soon |
This comment was marked as off-topic.
This comment was marked as off-topic.
Add (Ifkb / Ergokbd) to the list. As per https://www.ergokbd.com/products/if-corne-wireless2-4g-hotswap-split-mechanical-keyboard-pre-soldered-acrylic-vial-programmable-corne-crkb they are actively promoting their product with QMK and Vial but blatantly and outright refusing to provide ANY source code despite repeated attempts by some buyers. |
Here are the source code we received from the solution provider, please kindly review and let us know if it suffice to remove us from the list! thanks in advance. |
Hi @TweetyDaBird, FYI the source code for IFKB products is available at https://home.ifkb.tech/ which is also linked on the product page of Ergokbd which you reference. To be honest it's still far from ideal as I myself was a user who tried to get the source code by contacting the vendor on AliExpress I bought it from and was also stone walled. I only came across this when checking qmk toolbox for HID devices and seeing "IFKB sofle" and after a quick google search I found it.
|
Incorrect, the sources don't contain the wireless code. The vendor stays for this reason. |
I would have been very happy to be proven wrong, but no. No change. No complete source provided. |
I'm very new to all the QMK stuff so sorry for me asking but when you say there is no wireless code what are you referring to? I thought that all the code on the user manual page https://home.ifkb.tech/products/if-corne/user-manual would include everything needed. |
That code is the QMK/vial code for the MCU running the receiver. Not the code used to interface with QMK and running the wireless communication. As per the license that has to be supplied as well. Or at least a functional equivalent supplied if not the exact code. |
Only if it’s running QMK, or includes portions of QMK code. There’s some further investigation required for those boards as to whether or not they’re in breach. |
Yeah, very true, but at the very least it would be running code derivative of QMKs code as it's interfacing with the QMK MCU as a slave. (Although, I'm basically guessing it's using code similar to Mitosis, which I think this is based off). |
TL:DR; Vendors who submit PRs whilst not providing full sources for all of their shipping boards will be put on hold until source code for all violating keyboards are provided. Intentional deception may result in boards being removed from QMK and all future PRs for that manufacturer being denied outright.
This issue is limited to reports of non-compliant vendors -- asking for support or complaining about boards being unavailable (apart from reporting them) will be hidden as off-topic. For maintainers, once a vendor has been added to the list below, please mark the comment as "outdated".
If QMK identifies any licensing violation, all current and future PRs from that manufacturer will be put on hold until sources are provided. QMK's license requires full disclosure of source code for any firmware which is based on QMK, includes any part of QMK, or derives from QMK in any way. This source code disclosure is not negotiable and is automatically agreed to by any designer when they choose to use QMK.
Given the recent ramping-up of Chinese vendors disregarding QMK's license, QMK now actively chooses to prevent merges when violations are found.
Vendors may rectify the situation by providing full source code for their boards, even if it's in their own fork of QMK Firmware.
Current list of vendors who need to provide source code before any new PRs will be considered:
Initial post reporting Royal Kludge boards
Royal Kludge has issued a bunch of PRs recently with significant ambiguity as to whether or not submitted keyboards are tri-mode wireless or not.
So far there has been insufficient evidence supplied stating "no", they're wired-only boards -- comments on PRs when querying this fact have been ambiguous at best.
Additionally, with the fact that Royal Kludge is currently distributing VIA definitions for tri-mode keyboards based on QMK here -- including the rk839, also known as RK-R65, which shares the Product ID
0xE453
listed in PR #23559, we can only assume that Royal Kludge is submitting wired board definitions to QMK in order to satisfy VIA's requirement that keyboards must exist in QMK'smaster
branch, with no intention to supply source code for wireless boards as per QMK's licensing requirements.As of the posting of this issue, QMK is putting a hold on all PRs from vendor ID
0x342D
until all sources are made available for the corresponding QMK-based boards listed on the above page (inclusive of wireless bindings where relevant):0x6461
)0xE453
)0xE47F
)0xE453
)0xE480
)0xE484
)0xE483
)0xE487
)0xE485
)Additionally, the VIA definitions on the same page also list the following combinations which also need full sources provided, including wireless:
0xBB3F
, product ID0x0001
0x342D
, product ID0xE482
The text was updated successfully, but these errors were encountered: