qmk/qmk_firmware

[Bug] Large external EEPROM fails on AVR

Open

#8742 aperta il 9 apr 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)C (43.867 fork)batch import
bughelp wanted

Metriche repository

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

Descrizione

Describe the Bug

All the eeprom function signatures provided by avr-libc accept a void* address parameter for the EEPROM offset. This causes problems with AVR when attempting to address 32kB (using signed offsets) or 64kB (when using unsigned).

ChibiOS/ARM reuse the same function signatures to ensure compatibility across the rest of QMK -- these have no issue with addressing as the pointer size is 32-bit.

Suggested fix

  • Rework the persistent storage as a separate subsystem, perhaps called nvram_read_byte() and similar, which provides the same set of functions, but with uint32_t offsets.
  • Relocate things like the Teensy SmartEEPROM to match the current drivers/eeprom/ structure.
  • Relocate the emulated flash to the same location.

Guida contributor

Direzione di ricerca
Analizza le attuali firme delle funzioni EEPROM in avr libc e QMK, comprendi come avviene il troncamento del puntatore per EEPROM di grandi dimensioni e valuta l'approccio proposto del sottosistema nvram.
Tech stack
c
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 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
CGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40