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

Initial Direct XIP Support + Revert of Default Image Slot Value (back to 0) #139

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

dinesharjani
Copy link
Contributor

This commit represents the first time we were able to upload Direct XIP without having to stop the debugger, check something, modify something in the code, etc. Now, this doesn't mean that it's finished - I believe it can be made a lot tidier. But it's closer.

As for the revert on the Default Image Slot value, if we leave Direct XIP aside, what we want every time is for the image to be in slot 0. That's what our comparison for 'is this already uploaded?' is against. Hence, why the correct value, I think now after going back and forth a couple of times, is zero for how we've been doing things up until now.

… to 0)

This commit represents the first time we were able to upload Direct XIP without having to stop the debugger, check something, modify something in the code, etc. Now, this doesn't mean that it's finished - I believe it can be made a lot tidier. But it's closer.

As for the revert on the Default Image Slot value, if we leave Direct XIP aside, what we want every time is for the image to be in slot 0. That's what our comparison for 'is this already uploaded?' is against. Hence, why the correct value, I think now after going back and forth a couple of times, is zero for how we've been doing things up until now.
@dinesharjani dinesharjani self-assigned this Oct 11, 2023
@dinesharjani dinesharjani added the enhancement New feature or request label Oct 11, 2023
@dinesharjani dinesharjani merged commit 733588b into master Oct 11, 2023
1 check passed
@dinesharjani dinesharjani deleted the direct-xip branch October 11, 2023 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant