Skip to content
This repository has been archived by the owner on Mar 24, 2024. It is now read-only.

gw_flash: Add support for Winbond W25Q128JWSIQ 16MB Flash #169

Closed
wants to merge 1 commit into from
Closed

gw_flash: Add support for Winbond W25Q128JWSIQ 16MB Flash #169

wants to merge 1 commit into from

Conversation

unhold
Copy link
Contributor

@unhold unhold commented Dec 5, 2021

W25Q128JWSIQ is widely available in SOIC-8.
Some testing done.

@unhold
Copy link
Contributor Author

unhold commented Dec 5, 2021

It worked flawlessly, I could flash it multiple times without any issue. In fact, it also worked with the default configuration. But the first try after the pull request failed, I have to figure out why.

@unhold unhold closed this Dec 5, 2021
@kbeckmann
Copy link
Owner

Thanks. Hope you can find what’s wrong. The pr looks good otherwise and the commands seem to be correct according to the data sheet. Perhaps some init sequence might be needed to reset the chip?

@unhold
Copy link
Contributor Author

unhold commented Jan 5, 2022

Thank you for checking it!
Finally had the inspiration to figure it out: I had protection bits in the status registers set, because I accidentally executed init_mx_issi() on the Winbond. The pull-requested code was fine.

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

Successfully merging this pull request may close these issues.

2 participants