RE: [PATCH v3 11/23] mtd: spi-nor: winbond: Add support for W25Q12RV-Q/N
| Newsgroups | gmane.linux.documentation,gmane.linux.drivers.mtd,gmane.linux.ports.arm.kernel,gmane.linux.kernel |
|---|---|
| Message-ID | <[email protected]> |
Hi Miquel, > There is an ID collision with the chip of same density from the JV > family. Both chips are very similar in practice, it is mostly a matter > of electrical differences (mostly power consumption being lower). > > As a significant difference, RV chips identify themselves as supporting > the new SFDP (rev F) field which forces an alternate write SR2 > opcode (0x31). > > Signed-off-by: Miquel Raynal <[email protected]> > --- > + cat /sys/bus/spi/devices/spi0.0/spi-nor/jedec_id > ef4018 > + cat /sys/bus/spi/devices/spi0.0/spi-nor/manufacturer > winbond > + xxd -p /sys/bus/spi/devices/spi0.0/spi-nor/sfdp > 534644500a0100ff00080117800000ffffffffffffffffffffffffffffff > ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff > ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff > ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff > ffffffffffffffffe520f9ffffffff0344eb086b083b42bbfeffffffffff ^^^^^^^^ Just noticed size is wrong like patch #18. BFPT DWORD2 advertises memory density in bits (0-based). In 128Mb parts, this should be 0x07FFFFFF. Need post_bfpt_fixup. Thanks, Takahiro