-
-
Notifications
You must be signed in to change notification settings - Fork 8.3k
ports/alif/boards/ALIF_ENSEMBLE: Add deploy instructions. #17435
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
ports/alif/boards/ALIF_ENSEMBLE: Add deploy instructions. #17435
Conversation
5069017
to
f435c5b
Compare
@dpgeorge I'm not sure how to add the firmware to the hosted files, we probably need to add the entire build dir. Could you please add it? Also, I don't have a devkit to test with, but I did test these instructions with AE3.
I'm not really sure what the purpose of the "features" tag is, but I can just remove these. |
I think we should add just the files that are needed for deployment over serial. To do that, we should create a target in the |
91683fb
to
52b9214
Compare
This is fine with me, but it's probably best if you do this step. We could merge the build instructions and board images contained in this PR or feel free to push to the branch/cherry-pick the commits here. |
I've now pushed 2 commits to this PR:
I didn't test the zip yet! |
Code size report:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #17435 +/- ##
=======================================
Coverage 98.57% 98.57%
=======================================
Files 169 169
Lines 21968 21968
=======================================
Hits 21654 21654
Misses 314 314 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Tested the build and everything works. |
@dpgeorge Is the readme/deploy here clear enough? Should we move forward with this PR? |
The resulting `firmware.zip` file is self contained with everything needed to deploy the firmware, eg over SE UART. Signed-off-by: Damien George <[email protected]>
Signed-off-by: Damien George <[email protected]>
The firmware for this board will now be built and available automatically. Signed-off-by: iabdalkader <[email protected]>
a8d3ef5
to
5b0fb29
Compare
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.
I tested this on an Alif dev kit by following the instructions here, and the system package update works, along with the firmware install.
Signed-off-by: iabdalkader <[email protected]>
5b0fb29
to
9b38cf9
Compare
Summary
Add deploy/build instructions for Ensemble E7 DevKit.
Testing
I've tested these instructions, with another board, and they work fine. Testing with the devkit itself would be helpful.