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

Add OTP flash memory configuration #567

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

Conversation

rmja
Copy link
Contributor

@rmja rmja commented Feb 13, 2025

This PR restores the OTP configuration removed in #460.
I hope that I got the configuration right for all the series. The regions vary in size from 528 bytes (yes 16*32+16) to 512 bytes and 1k and 2k with many different locations. The data is based on the information in the header files and extracted with a grep OTP *.h.

Note that I had to make a new mem macro variant because of the odd sizes. Don't know if that is the best approach.

I have verified that it is correct for stm32f429 that I am on myself.

@embassy-ci
Copy link

embassy-ci bot commented Feb 13, 2025

@rmja rmja marked this pull request as draft February 13, 2025 13:25
@rmja rmja marked this pull request as ready for review February 13, 2025 13:32
@embassy-ci
Copy link

embassy-ci bot commented Feb 13, 2025

@rmja
Copy link
Contributor Author

rmja commented Feb 13, 2025

I have updated so the flash settings now come with an erase_size: 0.
It should now be ready for review.

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.

1 participant