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

keyball61: extend delay after unselect pin #114

Merged
merged 2 commits into from
Apr 22, 2022
Merged

keyball61: extend delay after unselect pin #114

merged 2 commits into from
Apr 22, 2022

Conversation

koron
Copy link
Collaborator

@koron koron commented Apr 21, 2022

to stabilize duplex matrix.

This may mitigate #112

@koron koron self-assigned this Apr 21, 2022
@koron
Copy link
Collaborator Author

koron commented Apr 21, 2022

左手基板でCtrl+CやA+Vの同時押しがゴーストを生んでしまう可能性
(もしくは右手基板でも3つ以上のキーを押すとゴーストが生まれる)
を減らすための対策を作りました。

必ずしも同じではないのですがRP2040で似たような現象に遭遇した際に、
キースキャン中に電圧をHiで維持する時間を延伸することで症状が緩和できましたので
それを移植したものになります。

RP2040を用いた実験では65us以上であれば大丈夫そうでしたが、少しマージンをみて70usにしました。
もともとは30usだったので比較で言えばかなり長時間になりますが、
これだけでduplex matrixを安定化させられるのであれば、十分な対価であると判断しました。

少し実用してからマージする予定です。

@Yowkees
Copy link
Owner

Yowkees commented Apr 22, 2022

koron様
ソフトウェアによる対策のご検討、誠にありがとうございます!
現在進めている一体型KeyballにもDuplex-Matrixを採用していたため、作り直しか~Keyball61も設計しなおしかな~とショボーンとしていましたがこのような対策案があるのですね。
私も手持ちのKeyball61や組立サービスで作った数台にて再現を試みましたが、ゴーストが出ず苦慮していました。
半田付け不良等の諸条件が重なったときの症状で、それをさらにソフト的に緩和できていれば、かなり安心できるのではないでしょうか。

ここ数日気が気でなかったですが、ちょっと落ち着きました。。
まもなく、新作Keyballの試作品が準備できる予定です。

対策案をご用意いただき本当にありがとうございました!

@koron
Copy link
Collaborator Author

koron commented Apr 22, 2022

@Yowkees

なかなか難しいですね。

ダイオード以外にLEFT判定のハンダジャンパのできも関わってるかもしれません。
今度それで悩んでる人がいたらLEFTジャンパもやり直すように言った方が良いかもです。

@Yowkees Yowkees merged commit 8404bd4 into Yowkees:main Apr 22, 2022
@Yowkees
Copy link
Owner

Yowkees commented Apr 22, 2022

ソフトウェアでのご対応、誠にありがとうございました!了解しました。
私サイドから出来ることとして、はんだ付けの確実なやり方的な説明写真や資料を準備したいと思います!

@koron koron deleted the keyball61-stabilize-duplex-matrix branch May 8, 2022 03:01
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