Skip to content

Conversation

TychoVrahe
Copy link

This PR adds a check of model identifier in app image TLV against the one provided during mcuboot build

@TychoVrahe TychoVrahe self-assigned this Aug 27, 2025
@github-project-automation github-project-automation bot moved this to 🔎 Needs review in Firmware Aug 27, 2025
Copy link

@matejcik matejcik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM from my side but i'm not a good reviewer for this, someone with better familiarity with mcuboot should check it

@TychoVrahe TychoVrahe requested a review from cepetr August 27, 2025 10:37
Copy link

@cepetr cepetr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@TychoVrahe TychoVrahe merged commit 439e4f2 into trezor-v2.1.0-ncs3 Aug 27, 2025
1 of 60 checks passed
@github-project-automation github-project-automation bot moved this from 🔎 Needs review to 🤝 Needs QA in Firmware Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🤝 Needs QA
Development

Successfully merging this pull request may close these issues.

3 participants