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

[Bug] Encoder on slave continues to trigger forever once activated #23191

Closed
2 tasks
jasonish opened this issue Mar 1, 2024 · 11 comments · Fixed by #23195
Closed
2 tasks

[Bug] Encoder on slave continues to trigger forever once activated #23191

jasonish opened this issue Mar 1, 2024 · 11 comments · Fixed by #23195
Assignees

Comments

@jasonish
Copy link

jasonish commented Mar 1, 2024

Describe the Bug

After updating from an older version to latest git and flashing both halves of my keyboard, the non-primary side encoder, once rotated continues to trigger forever at a rate that causes the keyboard to become unresponsive.

The primary side operates as you might expect.

I've narrowed it down to commit 9d9cdaa.
The commit before, 2eb9ff8 functions OK.

Keyboard Used

sofle_choc

Link to product page (if applicable)

Linux

Operating System

No response

qmk doctor Output

QMK Doctor is checking your environment.
Ψ CLI version: 1.1.2
Ψ QMK home: /home/jason/keyboards/qmk/master
Ψ Detected Linux (Fedora Linux 39 (Workstation Edition)).
Ψ Userspace enabled: False
Ψ Git branch: master
Ψ Repo version: 0.24.0
Ψ - Latest master: 2024-02-28 21:47:37 +1100 (4e369d405af) -- Merge branch 'develop'
Ψ - Latest upstream/master: 2024-02-28 21:47:37 +1100 (4e369d405af) -- Merge branch 'develop'
Ψ - Latest upstream/develop: 2024-02-28 19:13:04 -0700 (4e953f11696) -- Fixup mechlovin/octagon (#23179)
Ψ - Common ancestor with upstream/master: 2024-02-28 21:47:37 +1100 (4e369d405af) -- Merge branch 'develop'
Ψ - Common ancestor with upstream/develop: 2024-02-28 21:47:37 +1100 (4e369d405af) -- Merge branch 'develop'
Ψ All dependencies are installed.
Ψ Found arm-none-eabi-gcc version 13.1.0
Ψ Found avr-gcc version 13.1.0
⚠ We do not recommend avr-gcc newer than 8. Downgrading to 8.x is recommended.
Ψ Found avrdude version 7.3
Ψ Found dfu-programmer version 0.7.2
Ψ Found dfu-util version 0.11
Ψ Submodules are up to date.
Ψ Submodule status:
Ψ - lib/chibios: 2023-04-15 13:48:04 +0000 --  (11edb1610)
Ψ - lib/chibios-contrib: 2023-11-27 18:15:44 +0100 --  (9d7a7f90)
Ψ - lib/googletest: 2021-06-11 06:37:43 -0700 --  (e2239ee6)
Ψ - lib/lufa: 2022-08-26 12:09:55 +1000 --  (549b97320)
Ψ - lib/vusb: 2022-06-13 09:18:17 +1000 --  (819dbc1)
Ψ - lib/printf: 2022-06-29 23:59:58 +0300 --  (c2e3b4e)
Ψ - lib/pico-sdk: 2023-02-12 20:19:37 +0100 --  (a3398d8)
Ψ - lib/lvgl: 2022-04-11 04:44:53 -0600 --  (e19410f)
Ψ QMK is ready to go, but minor problems were found

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

@yottta
Copy link

yottta commented Mar 3, 2024

➕ Same issue here.
Keyboard: Sofle
OS: Mac/Win
Issue visible on the primary instead.

@haunt98
Copy link

haunt98 commented Mar 4, 2024

Same issue,
Keyboard: Sofle V2 with converter rp2040
OS: Mac/Win
Both primary, slave rotated got trigger

haunt98 added a commit to haunt98/qmk_keymaps that referenced this issue Mar 4, 2024
@Coko7
Copy link

Coko7 commented Mar 5, 2024

Same issue as well.
Keyboard: sofle/rev1
OS: Linux/Win
Primary encoder (left) is fine, but secondary (right) gets triggered repeatedly, as described by OP.

@tzarc
Copy link
Member

tzarc commented Mar 5, 2024

@jasonish @yottta @Coko7 are you using RP2040 converters? Or just normal 32U4-based?

@tzarc tzarc self-assigned this Mar 5, 2024
@Coko7
Copy link

Coko7 commented Mar 5, 2024

@tzarc I believe I am using an ATmega32U4, although I need to confirm it.

@yottta
Copy link

yottta commented Mar 5, 2024

@tzarc liatris converter.

qmk compile -kb splitkb/aurora/sofle_v2/rev1 -km custom -e CONVERT_TO=liatris

@jasonish
Copy link
Author

jasonish commented Mar 5, 2024

@jasonish @yottta @Coko7 are you using RP2040 converters? Or just normal 32U4-based?

ATmega32U4

@tzarc
Copy link
Member

tzarc commented Mar 6, 2024

Have updated #23195 with some potential ways forward, would be good to get some confirmation as to whether it's headed in the right direction. Still can't reproduce locally, so basically this is just some defensive coding.

@yottta
Copy link

yottta commented Mar 6, 2024

@tzarc installed it. Will keep you posted. Thanks a lot for the great effort! 👏

@Coko7
Copy link

Coko7 commented Mar 7, 2024

@tzarc Thank you for the work!
I have tried it out just now, and it seems to fix it on my keyboard. I have set up my secondary encoder for page up/down motion, and now it behaves as it should (without infinitely triggering).
I will test it more today and come back here if I run into some issues.

Coko7 added a commit to Coko7/sofle that referenced this issue Mar 7, 2024
@yottta
Copy link

yottta commented Mar 7, 2024

@tzarc installed it. Will keep you posted. Thanks a lot for the great effort! 👏

After 1 day I see no other issues. Thanks a lot for it.

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

Successfully merging a pull request may close this issue.

5 participants