RE: [PATCH v2] mtd: spi-nor: macronix: add support for mx77l51250f

From: Usyskin, Alexander
Date: Mon Jan 09 2023 - 09:09:17 EST



> > --- a/drivers/mtd/spi-nor/macronix.c
> > +++ b/drivers/mtd/spi-nor/macronix.c
> > @@ -100,6 +100,8 @@ static const struct flash_info macronix_nor_parts[]
> > = {
> > { "mx66u2g45g", INFO(0xc2253c, 0, 64 * 1024, 4096)
> > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ |
> SPI_NOR_QUAD_READ)
> > FIXUP_FLAGS(SPI_NOR_4B_OPCODES) },
> > + { "mx77l51250f", INFO(0xc2751a, 0, 64 * 1024, 4096)
> > + PARSE_SFDP },
>
> With the newest generic spi nor driver [1] this patch shouldn't
> be needed at all anymore. Could you verify, that your flash will work
> without it?
>
> -michael
>
> [1]
> https://elixir.bootlin.com/linux/v6.2-rc3/source/drivers/mtd/spi-
> nor/core.c#L1637

Tested now, the v6.2-rc3 recognizes the chip.
Does this mean that all SFDP-supporting chips will be recognized automatically?

Tomas, we can abandon upstreaming effort, I'll save the patch if it will needed by older kernel versions.

--
Thanks,
Sasha