qmk/qmk_firmware

Support more than 256 leds

Open

#24.675 aperta il 4 dic 2024

Vedi su GitHub
 (6 commenti) (2 reazioni) (0 assegnatari)C (43.867 fork)batch import
discussionhelp wantedquestion

Metriche repository

Star
 (20.368 star)
Metriche merge PR
 (Merge medio 20g 9h) (27 PR mergiate in 30 g)

Descrizione

I made a split keyboard with 4 leds under each key (10 for the 2 2U keys) and the results are pretty spectacular. This sums up to 162 leds per half, or 324 leds on both. QMK currently uses a uint_8 in most places to iterate over leds, so this required some patches which I want to polish up and contribute back.

I'm not intimately familiar with the codebase or nuances of QMK, so it would be great with some guidance on the best way to implement it for this project. I assume RAM is enough of a concern for the AVR platform that replacing all relevant uint_8 with uint_16 would be problematic? Would a typedef uintX_t num_leds_t then be best setting the type to uint16_t if RGB_MATRIX_LED_COUNT > 255, and using that type everywhere instead? Anything else I should know before putting time into implementing this properly?

Guida contributor

Direzione di ricerca
Indaga tutti gli usi di `uint8 t` per i conteggi LED nel codice QMK, in particolare nei file della matrice RGB e dei driver LED. Propone una soluzione che utilizzi `uint16 t` o un typedef come `led count t` definito condizionalmente in base a `RGB MATRIX LED COUNT`. Valuta l'impatto sulla memoria sulla piattaforma AVR.
Tech stack
c
Dominio
Nessuno
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Cembedded programmingQMK basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50