forks/RadioLibSmol
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed Persistent Storage Issue for RP2040 with Arduino Pico Framework (#868)

Browse source 
RP2040 does not have an EEPROM but always uses the last 4K chunk of the flash for a software EEPROM - if used. It is exactly handled as ESP32 "SoftEEPROMs", meaning it does copy the "flashEEPROM" to memory on .begin(); and does need to commit(); to write it back. We saw in the past that a node could successfully get an OTAA on an RP2040, but could never join - due to the missing commit and wrong init, this was the reason. As the "SoftEEPROM" is always written at the end of the flash, it also survives an Arduino Sketch reflash if not wiped afterwards by node.wipe(); More info and documentation here: https://arduino-pico.readthedocs.io/en/latest/eeprom.html
This commit is contained in:
Nico Maas 2023-11-04 16:09:13 +01:00 • committed by GitHub
parent aca1d78a97
commit f691b11c38
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/ArduinoHal.cpp
View file

@ -104,7 +104,7 @@ void inline ArduinoHal::spiEnd() {
void ArduinoHal::readPersistentStorage(uint32_t addr, uint8_t* buff, size_t len) { void ArduinoHal::readPersistentStorage(uint32_t addr, uint8_t* buff, size_t len) {
#if !defined(RADIOLIB_EEPROM_UNSUPPORTED) #if !defined(RADIOLIB_EEPROM_UNSUPPORTED)
#if defined(RADIOLIB_ESP32) #if defined(RADIOLIB_ESP32) || defined(ARDUINO_ARCH_RP2040)
EEPROM.begin(RADIOLIB_HAL_PERSISTENT_STORAGE_SIZE); EEPROM.begin(RADIOLIB_HAL_PERSISTENT_STORAGE_SIZE);
#elif defined(ARDUINO_ARCH_APOLLO3) #elif defined(ARDUINO_ARCH_APOLLO3)
EEPROM.init(); EEPROM.init();
@ -112,7 +112,7 @@ void ArduinoHal::readPersistentStorage(uint32_t addr, uint8_t* buff, size_t len)
for(size_t i = 0; i < len; i++) { for(size_t i = 0; i < len; i++) {
buff[i] = EEPROM.read(addr + i); buff[i] = EEPROM.read(addr + i);
} }
#if defined(RADIOLIB_ESP32) #if defined(RADIOLIB_ESP32) || defined(ARDUINO_ARCH_RP2040)
EEPROM.end(); EEPROM.end();
#endif #endif
#endif #endif
@ -120,7 +120,7 @@ void ArduinoHal::readPersistentStorage(uint32_t addr, uint8_t* buff, size_t len)
void ArduinoHal::writePersistentStorage(uint32_t addr, uint8_t* buff, size_t len) { void ArduinoHal::writePersistentStorage(uint32_t addr, uint8_t* buff, size_t len) {
#if !defined(RADIOLIB_EEPROM_UNSUPPORTED) #if !defined(RADIOLIB_EEPROM_UNSUPPORTED)
#if defined(RADIOLIB_ESP32) #if defined(RADIOLIB_ESP32) || defined(ARDUINO_ARCH_RP2040)
EEPROM.begin(RADIOLIB_HAL_PERSISTENT_STORAGE_SIZE); EEPROM.begin(RADIOLIB_HAL_PERSISTENT_STORAGE_SIZE);
#elif defined(ARDUINO_ARCH_APOLLO3) #elif defined(ARDUINO_ARCH_APOLLO3)
EEPROM.init(); EEPROM.init();
@ -128,7 +128,7 @@ void ArduinoHal::writePersistentStorage(uint32_t addr, uint8_t* buff, size_t len
for(size_t i = 0; i < len; i++) { for(size_t i = 0; i < len; i++) {
EEPROM.write(addr + i, buff[i]); EEPROM.write(addr + i, buff[i]);
} }
#if defined(RADIOLIB_ESP32) #if defined(RADIOLIB_ESP32) || defined(ARDUINO_ARCH_RP2040)
EEPROM.commit(); EEPROM.commit();
EEPROM.end(); EEPROM.end();
#endif #endif