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

Fix the foe code of the rtl_slavedemo application #110

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bnjmnp
Copy link

@bnjmnp bnjmnp commented Dec 15, 2021

With d9ebdc9 the FOE part of the API changed a bit, but the application was not updated.

With d9ebdc9 the FOE part of the API changed a bit, but the application was not updated.
@nakarlsson
Copy link
Contributor

For this particular example we could also add that it will abort with Error if not in BOOTSTRAP state.
Makes the sample more clear if some might wonder why it is missing. Of course, as a static variable it would be 0'ed by crt as part of .bss otherwise.

.write_only_in_boot = 1;

To demonstrate the use of of the `write_only_in_boot` FoE file configuration.
@bnjmnp
Copy link
Author

bnjmnp commented Dec 16, 2021

Good point, I just added what you suggested.

@bnjmnp
Copy link
Author

bnjmnp commented Jan 19, 2025

Hi there! I was wondering if there are any updates or specific reasons why this hasn't been merged yet. Is there anything I can do to help move it forward?

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.

2 participants