-
Notifications
You must be signed in to change notification settings - Fork 6
Add config struct to Flash #116
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #116 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 18 18
Lines 2102 2102
=========================================
Hits 2102 2102 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
is there a way to write to flash using nrfutil ? nrfjprog is deprecated and should not be used. |
Yes there is. I am preparing a script for that. I will use nrfutil to flash. On the other hand, I found that we can read the device id with So I will make a script that requires nrfutil, and, in case nrfjprog is available, also dumps the device id (that way we don't need a custom firmware speaking uart in the robot). |
9f12405 to
6c8de12
Compare
How to test this changeJust follow the README: https://github.com/geonnave/swarmit/blob/a55d5b387edc4bff38f92bb6831b17884cc71724/swarmit/provision/README.md During the first step ( swarmit-provision fetch --fw-version test-provision |
|
For the moment I am using nrfjprog, since there was already a base script created by Martina that uses nrfjprog, so it was just faster to integrate it that way. We should be able to easily replace it with the more modern nrfutils, perhaps in a future PR. |
ddca253 to
6e4538f
Compare
|
Nice and useful addition. I know it's annoying and time is missing but it took me quite some effort to cover all Python code with tests. Would you mind adding tests for the new provision modules? |
|
I see, however timing is unfortunate to add tests right now. |
6e4538f to
cc8ffe7
Compare
|
Ok, removed all python changes, and kept only the C change for the config struct. The python provisioning script now lives at https://github.com/DotBots/dotbot-provision |
The idea is to introduce a set of self-contained changes to firmware and python, that enable DotBots and gateways to be easily provisioned at deployment time.
initial comment about writing to config memory
With this change we will be able to change the network id after flashing the hex, simplifying deployment.The network id can be changed as follows: