qmk/qmk_firmware

[Bug] RGBLED_NUM needs one more than actual number of LEDs

Offen

#14.483 geöffnet am 17.09.2021

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C (43.867 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (20.368 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Describe the Bug

The last LED in the row of addressable RGB LEDs on my keyboard fails to light up when using the standard firmware. Increasing the number of LEDs defined in config.h by one solves the problem. The value then is 5 though, while I have only 4 LEDs on the keyboard.

System Information

  • Keyboard: Keebio BDN9
    • Revision (if applicable): Rev. 1
  • Operating system: Windows 10
  • AVR GCC version: 5.4.0
  • ARM GCC version: 9.2.1 20191025
  • QMK Firmware version: 0.14.16
  • Any keyboard related software installed?
    • AutoHotKey
    • Karabiner
    • Other:

Additional Context

The LEDs use SK6812 drivers, which should be drop-in clones of the WS2812B.

Contributor Guide

Research-Richtung
Untersuche die Verwendung von RGBLED NUM in der Firmware. Überprüfe den Code, der für die Adressierung der LEDs verantwortlich ist, wahrscheinlich in quantum/rgb.c oder ähnlichem. Suche nach Off by one Fehlern in Schleifen, die über LEDs iterieren. Überlege, ob die erste LED als 0 oder 1 adressiert wird. Die Problemumgehung des Benutzers deutet darauf hin, dass die Anzahl um eins höher sein sollte als die tatsächliche Anzahl.
Tech Stack
c
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50