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

Support QMK firmware 0.19 #182

Closed
wants to merge 4 commits into from

Conversation

choplin
Copy link
Contributor

@choplin choplin commented Jan 17, 2023

This PR updates the firmware codes to build it with the latest QMK firmware (0.19.9).

This consists of the following changes:

  • Remove deprecated configuration from config.h
    • NO_ACTION_MACRO, NO_ACTION_FUNCTION
  • Use info.json to define USB define descriptor parameters
    • VENDOR_ID, PRODUCT_ID, DEVICE_VER, MANUFACTURER, PRODUCT
    • Since QMK accepts only x.y.z format for usb.device_version, their values are slightly changed.
  • Replace the obsolete keycodes
    • RESET -> QK_BOOT, EEP_RST -> EE_CLR, KC_LANG -> KC_LNG
  • Replace the obsolete API
    • register_button -> register_mouse

@Yowkees Yowkees requested a review from koron January 22, 2023 03:17
@koron
Copy link
Collaborator

koron commented Jan 22, 2023

Thank you for your contribution.

It looks good to merge. But we will postpone to merge this to main branch.

Because we consider that most of Keyball's users use Remap to configure keymap.
And as you may know, current Remap doesn't support QMK 0.19 yet.
However, they shall plan to shifting to QMK 0.19 (Japanese post, and Issue).
We will merge this after Remap supported QMK 0.19.

I'll create a pending branch for QMK 0.19 based on this PR.
Thanks again.

@koron
Copy link
Collaborator

koron commented Jan 22, 2023

今Github Actionsの結果を確認してたら、0.16.3 でもビルドできてるみたい。
仮にそうならmainにマージしてしまっても良い気はする。

1点気になるのは、Keyball 46のボール構成でPIDが変わるところか。
手元でも試験してみる予定。

@koron
Copy link
Collaborator

koron commented Jan 22, 2023

0.19.10を使ってGHAでビルドしてるやつ。
https://github.com/Yowkees/keyball/actions/runs/3978515730/jobs/6820340169

@koron koron mentioned this pull request Jan 22, 2023
6 tasks
@koron
Copy link
Collaborator

koron commented Jan 22, 2023

One47 も別途対応が必要そう。

@koron
Copy link
Collaborator

koron commented Jan 22, 2023

0.16.3 でもビルドできてるみたい。
仮にそうならmainにマージしてしまっても良い

0.16.3において、以下の変更点がどう機能するかについて検討中。

  • Remove deprecated configuration from config.h
    • NO_ACTION_MACRO, NO_ACTION_FUNCTION
  • 0.16.3 で quantum/ から既に消えていることを確認
  • Use info.json to define USB define descriptor parameters
    • VENDOR_ID, PRODUCT_ID, DEVICE_VER, MANUFACTURER, PRODUCT
    • Since QMK accepts only x.y.z format for usb.device_version, their values are slightly changed.
  • Keyball46 及び One47 で意図しないIDが付与される可能性があるので要修正
  • Replace the obsolete keycodes
    • RESET -> QK_BOOT, EEP_RST -> EE_CLR, KC_LANG -> KC_LNG
  • 問題無く利用できることを確認済み
  • Replace the obsolete API
    • register_button -> register_mouse

@koron
Copy link
Collaborator

koron commented Jan 22, 2023

QMK 0.16.3に向けた先行マージは諦めます。

0.19用のブランチは support-qmk_firmware-0.19-or-above #183 として続けます。
当該ブランチは、このPRの内容をベースに、そのまま含んでいます。

こちらのPRはいったん閉じて、 #183 で続きをやるということにします。
ありがとうございました。

@koron koron closed this Jan 22, 2023
@choplin
Copy link
Contributor Author

choplin commented Jan 24, 2023

了解しました!レビューありがとうございます。keyball44と61は手元にあるので、0.19対応に向けて検証など必要なことがあれば、お声がけ頂ければご協力できるかと思います。

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

Successfully merging this pull request may close these issues.

None yet

2 participants