On Fri, Jul 14, 2023 at 9:30 PM Michael Walle michael@walle.cc wrote:
Am 2023-07-14 20:29, schrieb Linus Walleij:
The Winbond "w25q128" (actual vendor name W25Q128JV) has exactly the same flags as the sibling device "w25q128jv". The devices both require unlocking to enable write access.
The actual product naming between devices vs the Linux strings in winbond.c:
0xef4018: "w25q128" W25Q128JV-IM/JM 0xef7018: "w25q128jv" W25Q128JV-IN/IQ/JQ
This is still wrong btw. According to: https://www.winbond.com/resource-files/W25Q128JV%20RevI%2008232021%20Plus.pd...
This should be 0xef4018: "w25q128" W25Q128JV-IN/IQ/JQ 0xef7018: "w25q128jv" W25Q128JV-IM/JM
Otherwise, looks good. Either Tudor can change that in place while picking up the patch or you could send a new version.
Ooops you're right, I'll just pick up your tag and re-send, sorry!
Yours, Linus Walleij