POC of a dynamic loading system for encoders to save on flash size#540
POC of a dynamic loading system for encoders to save on flash size#540GL-PROXY wants to merge 2 commits intoNext-Flip:devfrom
Conversation
|
interesting idea, thanks for the pr. would be nice to have some numbers here, i have a feeling that most encoders rely on the same helpers functions as the decoders and if thats the case moving the encoders only wouldnt remove the bulk of the code from flash. i dont have numbers to back that up but thats something to test before moving forward with this |
Hi WillyJL, I’ll try to gather some numbers for you, though I might need to do it later since I’m currently busy. I’ll also look into which protocols are feasible, as you mentioned, it might not be possible for all of them. |
|
Hi WillyJL, here are the numbers you asked for. Baseline (no patch): With lazy-loading patch (ONLY SECPLUSV2): |
|
thanks for the numbers. not bad, but definitely diminishing for the amount of work to convert the protocols. ill keep this open as a draft and possibly come back to it later. thank you |
What's new
Overview
The goal is to help with the limited flash space by moving most of the encoder parts to the SD card and loading them dynamically when needed instead of keeping everything in flash. Currently this is more of a POC then a finished improvement there are many parts i could have done better and cut down on to save even more flash but i don't really have the time so i am just making this pr as a example of what could be done to help.
For the reviewer