Dunfell+oe4t deploy scripts + cboot default on R32.4.3 #80
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Since uboot slot alignment issues are tricky, unless you know what you are doing you probably want cboot as your bootloader for TX2/mender See discussion at OE4T/meta-mender-community#8 (comment) Signed-off-by: Dan Walkes <danwalkes@trellis-logic.com>
Move the deploy script out of meta mender community, clean it up to have arguments and not require running under sudo. Move the steps from the https://github.com/OE4T/meta-tegra/wiki/Flashing-the-Jetson-Dev-Kit page into a `setup-udev-rules` script and reference this if deploy fails. Signed-off-by: Dan Walkes <danwalkes@trellis-logic.com>
|
Looks good to me. |
ichergui
approved these changes
Mar 28, 2021
|
I tested a Mender update on a TX2 from a build before this change to a build after this change was applied, and the reboot was successful. After reboot, the It's probably not worth the effort to clean that up here. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Thanks for getting the repo up to date for the mender community upstream @madisongh
There are two other changes here I was thinking we may want to backport to dunfell r32.4.3 since that's our stable mender branch. They may make sense in r32.5.0 and/or gatesgarth too.
cboot-t18xthe default bootloader for thetegrademo-menderdistro. It's maybe not typical to put a change like this on a stable branch but since this shouldn't impact anyone's existing custom distribution and is documented clearly in the README I'd hope it would be safe. Also we've found from experience you can get into confusing situations if you use uboot or cboot-prebuilt, so making one less thing someone needs to get right in their demo setup for those still usingjetson-tx2could be useful.