-
Notifications
You must be signed in to change notification settings - Fork 688
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
AT+SYSMFG: Improve with error code and HASH #790
Comments
@MaJerle Thanks for the feedback.
~ |
Is your feature request related to a problem?
When using
AT+SYSMFG
, you get an error if you ask for more data to read than available, or if offset is larger than length of data. The same error is also if you simply put wrong parameters.Describe the solution you'd like.
It would be good to add 2 things in this command (and potentially
AT+SYSFLASH
, too):HASH/CRC32/...
of the MSG/FLASH content. In this case, we can check if flash content is the one we want, before we load new one. With this, we save the flash durability. A CRC32 is fast to compute on an MCU (even this HASH would be better tho, from security perspective) and easier to handle than to read full data back to MCU and compare if values are identiacal.Describe alternatives you've considered.
There are no other alternatives, unfortunately. If command returns ERROR, you have to think what the problem is, usually end us w/o solution.
Additional context.
No response
The text was updated successfully, but these errors were encountered: