Skip to content

[nrf fromlist] bootutil: Unify app_max_size() implementations #463

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

Merged

Conversation

tomchy
Copy link
Contributor

@tomchy tomchy commented Jul 4, 2025

Remove redundant application size calculations in favor of a swap-specific function, implemented inside swap_<type>.c. In this way, slot sizes use the same restrictions as image validation.

@tomchy tomchy force-pushed the bugfix/swap/Unify_max_app_size_methods_nrf branch from 6ee54cb to db70f9d Compare July 4, 2025 09:53
@tomchy tomchy force-pushed the bugfix/swap/Unify_max_app_size_methods_nrf branch 6 times, most recently from b9678dd to 8a8dc91 Compare July 7, 2025 13:04
Remove redundant application size calculations in favor of
a swap-specific function, implemented inside swap_<type>.c.
In this way, slot sizes use the same restrictions as image validation.

Upstream PR #: 2318

Signed-off-by: Tomasz Chyrowicz <[email protected]>
@tomchy tomchy force-pushed the bugfix/swap/Unify_max_app_size_methods_nrf branch from 8a8dc91 to 08a7952 Compare July 10, 2025 07:33
@tomchy tomchy changed the title bootutil: Unify app_max_size() implementations [nrf fromlist] bootutil: Unify app_max_size() implementations Jul 10, 2025
Copy link

@tomchy tomchy requested review from nvlsianpu and michalek-no July 10, 2025 08:29
@de-nordic de-nordic requested a review from nordicjm July 11, 2025 14:25
@nordicjm nordicjm merged commit cf1f76c into nrfconnect:main Jul 14, 2025
7 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