Skip to content

My keyball rp2040 - #881

Open
Gon-Kane wants to merge 10 commits into
Yowkees:mainfrom
Gon-Kane:MyKeyball_RP2040
Open

Gon-Kane wants to merge 10 commits into
Yowkees:mainfrom
Gon-Kane:MyKeyball_RP2040

Conversation

@Gon-Kane

@Gon-Kane Gon-Kane commented Sep 5, 2026

Copy link
Copy Markdown

No description provided.

@Yowkees

Yowkees commented Sep 25, 2026

Copy link
Copy Markdown
Owner

まずKeyball61向けのSparkFun Pro Micro RP2040対応、ありがとうございます。RP2040移植自体は需要があると思うので前向きに検討したいのですが、マージ前に以下を直していただけますか。

  1. keyball61/rules.mk を無条件に書き換えている点
    現状の変更は MCU = atmega32u4 をコメントアウトし、CONVERT_TO = sparkfun_pm2040 を常時有効にした上で、EXTRAKEY_ENABLE / CONSOLE_ENABLE / COMMAND_ENABLE / BACKLIGHT_ENABLE を全員 yes に変更する内容になっています。これだと既存のPro Micro(AVR)でビルドしている全ユーザーに影響してしまいます。
    Add optional RP2040 CE support for Keyball44 #870 (keyball44) と fix: port rp2040 support for keyball39 #882 (keyball39) で採用したパターン — config.h 側で #ifdef MCU_RP を使ってRP2040の時だけ追加設定を差し込み、rules.mk は CONVERT_TO の指定のみに留める、または keymaps/ 配下でオプトインする — に合わせていただけると、AVRビルドに影響を与えずに取り込めます。

  2. CONVERT_TO = sparkfun_pm2040 のタイポ疑い
    QMKの converters にこの名前のターゲットが実在するか確認をお願いします(見慣れない綴りです。sparkfun_pro_micro_rp2040 等の既存ターゲットと混同していないか確認してください)。

  3. 個人キーマップ keymaps/GonKaneMap と手順書 ChangeToSparkFunRP2040.md について
    これらはこのPRの本題(RP2040ポート自体)とは別の内容に見えます。個人設定を配布する意図であれば別PRに分けていただくか、リポジトリに含める意図がなければ削っていただけますか。

上記を直していただければ再度レビューします。よろしくお願いします。

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.

2 participants