-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
treewide: update thingy91x bootloader setup #18137
Conversation
Use production setup of thingy91x bootloader configuration. Signed-off-by: Maximilian Deubel <[email protected]>
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 9cd9fa55b5c6c02044d79d55c52f54eeca88c6df more detailssdk-nrf:
mcuboot:
Github labels
List of changed files detected by CI (15)
Outputs:ToolchainVersion: 3dd8985b56 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
@@ -14,3 +14,8 @@ CONFIG_UART_CONSOLE=n | |||
CONFIG_MCUBOOT_SERIAL=y | |||
CONFIG_MCUBOOT_SERIAL_DIRECT_IMAGE_UPLOAD=y | |||
CONFIG_BOOT_SERIAL_IMG_GRP_IMAGE_STATE=y | |||
|
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.
Seems ok. @tony-le-24 can you check if the various additional nightly and weekly tests still work with this change -- any of them that run on the Thingy:91X?
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 will, thanks
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 ran it on my PR for Thingy91:X testing, things seem green. LGTM!
sdk-mcuboot PR has been merged, have brought in manifest update and commit here into #17567 to save on CI |
Use production setup of thingy91x bootloader configuration.