> drivers/mtd/spi-nor/Makefile | 1 + > drivers/mtd/spi-nor/core.c | 1 + > drivers/mtd/spi-nor/core.h | 1 + > drivers/mtd/spi-nor/renesas.c | 57 +++++++++++++++++++++++++++++++++++ Please put the flash in atmel.c. There are all the Atmel/Adesto/now Renesas flashes. > +static const struct flash_info renesas_nor_parts[] = { > + { "at25ql128a", INFO(0x1f4218, 0, 64 * 1024, 256) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > +}; I suppose this flash supports SFDP. Therefore, { "at25ql128a", PARSE_SFDP }, should be enough. I see that the 0x1f42xx IDs are also used for the at25sl series. Could you shed some light to what the differences are? Are they compatible, do we have to distiguish between them? Last, for a new flash submission we require a SFDP dump. See [1]. [1] https://lore.kernel.org/linux-mtd/4304e19f3399a0a6e856119d01ccabe0@xxxxxxxx/ > +static void renesas_nor_default_init(struct spi_nor *nor) > +{ > + nor->flags |= SNOR_F_HAS_LOCK; Did you test locking? -michael