Skip to content
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

boot: bootutil: loader.c: Add check if has upgrade before pushing state change #2142

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

jtlas19
Copy link

@jtlas19 jtlas19 commented Dec 10, 2024

Before pushing MCUBOOT_STATUS_UPGRADING, check if has_upgrade is true to ensure this is not pushed at every boot and only when needed.

@jtlas19 jtlas19 requested a review from davidvincze as a code owner December 10, 2024 13:59
@jtlas19 jtlas19 force-pushed the jtlas19-check-before-state-change branch 2 times, most recently from 92c7cd8 to 475fc32 Compare December 11, 2024 08:47
@nordicjm
Copy link
Collaborator

@jtlas19 can you fix compliance issue?

@nordicjm
Copy link
Collaborator

@jtlas19 would you be able to fix this today or on Monday? Would like to pull latest changes into Zephyr with a manifest update soon

@jtlas19
Copy link
Author

jtlas19 commented Dec 13, 2024

@jtlas19 would you be able to fix this today or on Monday? Would like to pull latest changes into Zephyr with a manifest update soon

I'll get it fixed today, apologize for taking so long. Thankyou for your patience!

…te change

Before pushing MCUBOOT_STATUS_UPGRADING, check if has_upgrade is true to ensure this is not pushed at every boot and only when needed.

Signed-off-by: Joseph Todd <[email protected]>
@jtlas19 jtlas19 force-pushed the jtlas19-check-before-state-change branch from 475fc32 to ad915c6 Compare December 16, 2024 10:29
@jtlas19
Copy link
Author

jtlas19 commented Dec 16, 2024

@nordicjm All fixed :)

@nordicjm nordicjm merged commit f5454f1 into mcu-tools:main Dec 16, 2024
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants