search

qmk / qmk_firmware

Open-source keyboard firmware for Atmel AVR and Arm USB families
https://qmk.fm
GNU General Public License v2.0
17.52k stars 37.8k forks source link

Clueboard: third press of a MO key hangs firmware #4692

Closed khogeland closed 2 years ago

khogeland commented 5 years ago

Using a Clueboard 60% v1.0.1. The third key up event of a MO key causes the firmware to hang. I'm using this simple keymap, everything seems to work fine including the MO key/function layer. I've tried putting the MO mapping on other physical keys, so I don't think it's a hardware problem. Compiled at QMK firmware v0.6.209 (https://github.com/qmk/qmk_firmware/commit/f3cbb7bd6e307186e5802fa0366a21a5f9f5e41c) using the Docker build, same bug also present in the firmware built by the QMK configurator (v0.6.203).

khogeland commented 5 years ago

6 presses on Caps Lock has the same effect

khogeland commented 5 years ago

Noticed both keys trigger led_set_kb in led.c. Commenting out the body of that function prevents the issue, so something's broken in there. I don't even have LEDs, so for my personal use I'm going to leave that commented and call it fixed. :stuck_out_tongue:

drashna commented 5 years ago

Do you know the last version that worked?

khogeland commented 5 years ago

This is a new keyboard, so it never worked for me. If the stock firmware was compiled from one of those keymaps, @skullydazed might know which version that was compiled with.

stale[bot] commented 4 years ago

This issue has been automatically marked as resolved because it has not had activity in the last 90 days. It will be closed in the next 30 days unless it is tagged properly or other activity occurs.

drashna commented 4 years ago

Was this still happening?

github-actions[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had activity in the last 90 days. It will be closed in the next 30 days unless it is tagged properly or other activity occurs. For maintainers: Please label with bug, in progress, on hold, discussion or to do to prevent the issue from being re-flagged.

tzarc commented 2 years ago

Closing due to inactivity.