-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Ensmallen 8266 firmware binary, switch to LittleFS for all #3457
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
pkendall64
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should rename the the src/elrs.flash.1m64.ld file as it's not 64k for the FS, it's 32k. Maybe just call it src/elrs.flash.ld.
Everything else looks sensible. Will approve after testing.
|
Yeah good idea. I went with elrs.flash.1m.ld since it is only used for the 1MB esp82xx chips and avoid confusion with it possibly being used for ESP32. Also tested on a RadioMaster ER8GV to verify the VSpd/BaroAlt looked good (ESP32 SX1280) |
|
@CapnBry I added another commit that changes the way we use ArduinoJson in |
pkendall64
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on 8285 and ESP32. All looks good here.
Many changes to shrink the firmware binary to allow ESP8285-based receivers to web flash again. WARNING: Change from SPIFFS to LittleFS means any custom hardware.json / options.json will be lost.
Porblem
Users who have flashed with 4.0.0-RC1 are unable to flash new binaries to their devices. Failing with "Not enough space". This is mostly LBT users as their binary is slightly larger than the ISM.
Ensmallen (verb) To Make Smaller
-DNDEBUG. Saves a couple KB flash and minor RAM.Overall size decreased by 27709 compressed bytes.