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

void W25qxx_EraseSector(uint32_t SectorAddr) BUG For W25Q256 #6

Open
hfbuhun opened this issue Nov 26, 2019 · 3 comments
Open

void W25qxx_EraseSector(uint32_t SectorAddr) BUG For W25Q256 #6

hfbuhun opened this issue Nov 26, 2019 · 3 comments

Comments

@hfbuhun
Copy link

hfbuhun commented Nov 26, 2019

This code is very good,but There is a problem for W25Q256, The instruction is 21h instead of 20h for a 32-bit address

@AlexKeisy
Copy link

AlexKeisy commented Nov 26, 2019

https://www.winbond.com/resource-files/w25q256fv_revg1_120214_qpi_website_rev_g.pdf
I specially check... function is 20h, but I havent got a w25q256 to test...

I know code contains bugs on writeBytes function....
this library is good as first point, but need to prepare to use....

@hfbuhun
Copy link
Author

hfbuhun commented Nov 26, 2019

https://www.winbond.com/resource-files/w25q256fv_revg1_120214_qpi_website_rev_g.pdf
我专门检查了...功能是20小时,但是我没有w25q256来测试...

我知道代码中包含关于writeBytes函数的错误。...
该库虽然很好,但是需要准备使用。

Sorry, my model is W25Q256JV。
https://www.winbond.com/resource-files/w25q256jv%20spi%20revb%2009202016.pdf
page55

@nimaltd
Copy link
Owner

nimaltd commented Nov 26, 2019

hello. thank you for your feedback. But i think it is work correctly.

The Sector Erase instruction sets all memory within a specified sector (4K-bytes) to the erased state of all
1s (FFh). A Write Enable instruction must be executed before the device will accept the Sector Erase
Instruction (Status Register bit WEL must equal 1). The instruction is initiated by driving the /CS pin low
and shifting the instruction code “20h” followed a 24/32-bit sector address (A23/A31-A0). The Sector
Erase instruction sequence is shown in Figure 36.

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

3 participants