-
-
Notifications
You must be signed in to change notification settings - Fork 39.6k
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.
I will review all the affected board makers and boards in VIA to be sure that nothing slipped through and got merged accidentally w/o due diligence. Any violation of the licenses and attempts to bypass the license requirements will result in removal from the VIA repo. |
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 one is complicated. In theory, they could have properly reverse engineered it. But if they took via.c, and retooled it to work with whatever firmware, then the code is still covered by GPL (2+) and source code still needs to be produced or to be in voliation. https://www.gnu.org/licenses/gpl-faq.en.html#GPLRequireSourcePostedPublic
In this case, releasing the binaries/firmware is releasing to the public. And: https://www.gnu.org/licenses/gpl-faq.en.html#TranslateCode
|
For clarity's sake, the modified code isn't the only code that needs to be produced -- the entire firmware is considered a derived work, and thus the source for the entire firmware needs to be produced, even if it's overall not based on QMK. Any use of a modified |
if the wireless firmware runs on a seperate mcu(do not use any QMK related source codes) and communicated the other mcu (runs QMK based firmware) through peripherals such as spi, uart (like the adafruit uart friends used in many keyboard). I'd thought in such cases, only released the source codes in the MCU which runs QMK should be enough. Is this right? |
Correct. In the case of via stuff, taking the GPL licensed code, and modifying it to work with a different firmware/code would require disclosure of that source too. |
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 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 was marked as off-topic.
This comment was marked as off-topic.
#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 |
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: