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

MEN-7554: Move zephyr-image update module and merge mender-flash into it #171

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

vpodzime
Copy link
Collaborator

@vpodzime vpodzime commented Mar 12, 2025

Two related changes plus a small follow-up.

@vpodzime vpodzime force-pushed the master-zephyr_um_move branch 7 times, most recently from c449f1c to 23c7622 Compare March 14, 2025 10:55
After the headers and sources reorganization there's no need to
keep it in the `src/core` directory where it quite obviously
doesn't belong.

Ticket: MEN-7554
Changelog: none
Signed-off-by: Vratislav Podzimek <[email protected]>
The POSIX implementation is just an artifical one that has no
real usage (and the *weak* one as well, of course). The only real
flashing implementation is the Zephyr/MCUBoot one. By only
keeping the latter, we can get rid of the unnecessary abstraction
and sources and simplify things.

Ticket: MEN-7554
Changelog: none
Signed-off-by: Vratislav Podzimek <[email protected]>
@vpodzime vpodzime force-pushed the master-zephyr_um_move branch from 23c7622 to deb5c50 Compare March 14, 2025 11:30
Now that the code is not attempting to be abstract and platform
independent, we can use the `struct flash_img_context` type directly.

Ticket: MEN-7554
Changelog: none
Signed-off-by: Vratislav Podzimek <[email protected]>
@vpodzime vpodzime changed the title chore: Move the zephyr-image update module under src/platform MEN-7554: Move zephyr-image update module and merge mender-flash into it Mar 14, 2025
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.

4 participants