Skip to content

drivers: flash: jesd216: improve mode support checks #90857

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

natto1784
Copy link
Contributor

The only information for 4s-4d-4d, 8s-8s-8s and 8d-8d-8d in the basic flash parameter table is in DWORD20 with one byte for each of the aforementioned modes. This one byte is split into two fields that contain that mode's maximum operation speed with and without data strobe. Unsupported fields have the value of 0xF. For the mode to be supported, at least one of the two fields must not be 0xF, so we check the byte against 0xFF.

@natto1784
Copy link
Contributor Author

the failed checks are unrelated it seems

@de-nordic de-nordic requested a review from anangl June 3, 2025 15:13
anangl
anangl previously approved these changes Jun 4, 2025
The only information for 4s-4d-4d, 8s-8s-8s and 8d-8d-8d in the basic flash
parameter table is in DWORD20 with one byte for each of the aforementioned
modes. This one byte is split into two fields that contain that mode's
maximum operation speed with and without data strobe. Unsupported fields
have the value of 0xF. For the mode to be supported, at least one of the
two fields must not be 0xF, so we check the byte against 0xFF.

Signed-off-by: Amneesh Singh <[email protected]>
Copy link

sonarqubecloud bot commented Jun 4, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants