RE: [PATCH v1 8/8] mtd: spi-nor: rework flash parameter initialization
| Newsgroups | org.infradead.lists.linux-mtd,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
Hi, > Rework how the flash parameters are initialized. It used to make a > difference whether a flash is in the in-kernel database, whether it > supports multi I/O and so on. Recently, that makes more and more > problems because flash IDs are reused among legacy flashes (w/o SFDP > support) and newer flashes [1,2,3,4]. > > Simplify the whole parameter initialization, by starting with the > parameters we have in the in-kernel database and the try to parse SFDP > (and let it override any settings). > > This has the consequence that all the flashes now get a RDSFPD opcode > which might be an unknown opcode from the perspective of the flash. With > the generic SPI NOR flash driver, this was already the case for any > flashes not listed in the in-kernel database. So far, there was not a > single complaint. > > [1] https://lore.kernel.org/linux-mtd/CAAyq3SYX9UPwhC_Ume_S2yxhQwimRvB=Y6O_+FFqokhNmw7jQg@mail.gmail.com/ > [2] https://lore.kernel.org/linux-mtd/[email protected]/ > [3] https://lore.kernel.org/linux-mtd/[email protected]/ > [4] https://lore.kernel.org/linux-mtd/[email protected]/ > > Signed-off-by: Michael Walle <[email protected]> > Tested-by: Cheng Ming Lin <[email protected]> > Tested-by: Miquel Raynal <[email protected]> # w/ W25Q02JVFIM > --- Tested-by: Takahiro Kuwano <[email protected]> Test log is here (I forgot a tag). https://lore.kernel.org/linux-mtd/[email protected]/ ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/