qmk/qmk_firmware

[Bug] QMK-DFU exit key and bootmagic bootloader key don't work on the same key

Open

#8.302 aberto em 3 de mar. de 2020

Ver no GitHub
 (6 comments) (2 reactions) (0 assignees)C (43.867 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (20.368 stars)
Métricas de merge de PR
 (Mesclagem média 20d 9h) (27 fundiu PRs em 30d)

Description

Describe the Bug

By default, the key in the first row and first column is the key that triggers the bootloader jump if pressed when the usb cable is plugged in. However, if the QMK-DFU escape key is set using QMK_ESC_OUTPUT and QMK_ESC_INPUT and it is the same key, it works too well. You escape the bootloader, but then it detects the key being pressed and brings you right back to the bootloader, so you'll never exit the bootloader that way. This is usually the esc key on many boards.

The expected behavior would be that it actually goes back to the user program.

System Information

  • Keyboard: any that uses qmk-dfu and bootmagic (lite)
    • Revision (if applicable):
  • Operating system: Windows 10 Pro 1903
  • AVR GCC version: 4.9.2
  • ARM GCC version: 4.9.3
  • QMK Firmware version: 0.7.119
  • Any keyboard related software installed?
    • AutoHotKey
    • Karabiner
    • Other:

Additional Context

I don't know if it's fixed already, I tried searching but I know my qmk version is a bit older

Guia do colaborador

Direção de pesquisa
Investigue a interação entre bootmagic e a tecla de saída do QMK DFU. Examine a lógica do bootmagic em `quantum/bootmagic.c` e o tratamento de saída do QMK DFU em `quantum/dfu.c`. O problema menciona que, quando a mesma tecla é usada para ambos, ocorre um loop de retorno. Verifique a lógica que lê a tecla após sair do bootloader. Considere adicionar um debounce ou verificação de estado para evitar reentrada. Procure por correções semelhantes no repositório do QMK.
Pilha de tecnologia
c
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
CEmbedded systemsQMK firmware basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40