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

PAGE_LOCKS are not ECC #3

Open
polhenarejos opened this issue Sep 7, 2024 · 0 comments
Open

PAGE_LOCKS are not ECC #3

polhenarejos opened this issue Sep 7, 2024 · 0 comments

Comments

@polhenarejos
Copy link

polhenarejos commented Sep 7, 2024

From the datasheet:

Lock bits in OTP are triple-redundant with a majority vote. They can’t be ECC-protected, because they may be mutated
bit-by-bit over multiple programming operations.

which makes sense.

But in the main.c code, ECC is used to lock the page 48, albeit 3 bytes are provided (which is clearly raw). If it is ECC, only 2 bytes should be provided.

cmd.flags = OTP_DATA_PAGE48_LOCK1_ROW | OTP_CMD_ECC_BITS | OTP_CMD_WRITE_BITS;

If I understood correctly, OTP_CMD_ECC_BITS should not be used.

kilograham added a commit that referenced this issue Sep 8, 2024
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