[PATCH 6/6] mtd: spi-nor-ids: Add support for Winbond W25Q256JV/W25Q512JV

[Weijie Gao]

On Tue, 2026-04-28 at 09:45 +0200, Miquel Raynal wrote:
> On 28/04/2026 at 11:32:13 +08, Weijie Gao <weijie.gao at mediatek.com>
> wrote:
> 
> > Datasheets can be found here:
> > 
https://urldefense.com/v3/__https://www.winbond.com/resource-files/W25Q256JV*20DTR*20RevK*2010202025*20Plus.pdf__;JSUlJQ!!CTRNKA9wMg0ARbw!hPu0SggaVM2sVfZIIpmauqFfldb_a2CUsdY8EXtlYmCrlPsgfbAHiUVQjb7RsCxDyHKNzQ_IeQino7kbe_I8YtAtzN5pOGJalR6BUZ0$
> >  
> > 
https://urldefense.com/v3/__https://www.winbond.com/resource-files/W25Q512JV*20DTR*20RevD*2006292020*20133.pdf__;JSUlJQ!!CTRNKA9wMg0ARbw!hPu0SggaVM2sVfZIIpmauqFfldb_a2CUsdY8EXtlYmCrlPsgfbAHiUVQjb7RsCxDyHKNzQ_IeQino7kbe_I8YtAtzN5pOGJaaukqJvM$
> >  
> > 
> > Signed-off-by: Weijie Gao <weijie.gao at mediatek.com>
> > ---
> >  drivers/mtd/spi/spi-nor-ids.c | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> > 
> > diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-
> > nor-ids.c
> > index 492411a90f5..dc62a0e034a 100644
> > --- a/drivers/mtd/spi/spi-nor-ids.c
> > +++ b/drivers/mtd/spi/spi-nor-ids.c
> > @@ -609,11 +609,21 @@ const struct flash_info spi_nor_ids[] = {
> >  			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ
> > |
> >  			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
> >  	},
> > +	{
> > +		INFO("w25q256jv", 0xef7019, 0, 64 * 1024, 512,
> > +			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ
> > |
> > +			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
> > +	},
> >  	{
> >  		INFO("w25q256jw", 0xef6019, 0, 64 * 1024, 512,
> >  			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ
> > |
> >  			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
> >  	},
> > +	{
> > +		INFO("w25q512jv", 0xef7020, 0, 64 * 1024, 1024,
> 
> This is JV-M (here and above).

Of course. I didn't test the -Q version but I already saw related parts
like w25q512jvq existed.

Perhaps naming these -M series with 'm' suffix is better.

> 
> These chips also happen to conflict with W25QxxRV-M chips. Is this
> support for these new chips that you are targeting?

I didn't use RV series now. w25q256jv/w25q512jv are tested on
MediaTek's filogic platform.

As winbond doesn't provide the actual contents of the SFDP table, I'm
not sure if there's a way to distinguish between these two families.

> 
> > +			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ
> > |
> > +			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
> > +	},
> >  	{
> >  		INFO("w25q512nwq", 0xef6020, 0, 64 * 1024, 1024,
> >  			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ
> > |
> 
> I would rather group them by family (ef60xx then ef70xx).

It's OK.

> 
> Thanks,
> Miquèl