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

RFC: Rename flash_area_ calls in MCUboot with boot_storage_ #2123

Open
de-nordic opened this issue Nov 15, 2024 · 0 comments
Open

RFC: Rename flash_area_ calls in MCUboot with boot_storage_ #2123

de-nordic opened this issue Nov 15, 2024 · 0 comments

Comments

@de-nordic
Copy link
Collaborator

de-nordic commented Nov 15, 2024

Why?

I have been recently adding change to MCUboot where it allows to support devices without hardware required erase and found out that it is really hard to do the change in a graceful way as the MCUboot is entangled, for example, with Zephyr.

It also means that any changes to flash_area_ in Zephyr immediately affect MCUboot, without easy way to provide compatibility layer.

Why not redefine the API if we are at it?

Because it would take ages and while we may get blocked by any changes to Zephyr or other system we support. And too many changes at once.

So what would be the change?

All flash_area_ prefixed calls would be renamed to use boot_storage_ prefix and system specific headers will be altered to do inline calls to system native function or MCUboot defined compatibility functions.

What would be Flash cost?

Should be none at this point.

OK, lets do it. Is there something that may block us?

Yes. bootutil_public. It seems that the library shares some headers and flash area macros with callers. Need further investigation and fixing. It also picks flash definitions from sysflash and so on, while executing in context of application that should use system provided definitions for areas.

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

No branches or pull requests

1 participant