| Message ID | 20260831-mtd-nand-new-chip-support-v1-1-dca9f63ac0f9@analog.com |
|---|---|
| State | New |
| Headers | show |
| Series | mtd: spinand: winbond: Add support for W25N08LW | expand |
On 31/08/2026 at 16:38:10 +01, Nuno Sá <nuno.sa@analog.com> wrote: > Add support for the W25N08LW, a 1.8V 8Gbit SPI-NAND made of two > 4Gbit LUNs, with 4096-byte pages and 256 bytes of spare area. > > The chip reuses the KV ECC status helper, but needs its own OOB > layout because the geometry of its spare area differs from the > existing Winbond parts. Only half of it is visible while the > internal ECC is enabled, so the ECC region is reported as > inaccessible and just the free bytes are exposed. > > Assisted-by: Claude:Opus-5 > Signed-off-by: Nuno Sá <nuno.sa@analog.com> > --- > drivers/mtd/nand/spi/winbond.c | 45 ++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 45 insertions(+) > > diff --git a/drivers/mtd/nand/spi/winbond.c b/drivers/mtd/nand/spi/winbond.c > index 9b78c1e6cbc9..c2c4cc306f05 100644 > --- a/drivers/mtd/nand/spi/winbond.c > +++ b/drivers/mtd/nand/spi/winbond.c > @@ -345,6 +345,41 @@ static const struct mtd_ooblayout_ops w25n02kv_ooblayout = { > .free = w25n02kv_ooblayout_free, > }; > > +static int w25n08lw_ooblayout_ecc(struct mtd_info *mtd, int section, > + struct mtd_oob_region *region) > +{ > + /* > + * With ecc enabled the parity bits are not accessible. So we can > + * only see page + 128. Without ecc the full page + 256 is accessible. > + * To make it simple just return the area as not accessible. > + */ > + return -ERANGE; > +} > + > +static int w25n08lw_ooblayout_free(struct mtd_info *mtd, int section, > + struct mtd_oob_region *region) > +{ > + if (section > 7) > + return -ERANGE; > + > + region->offset = (16 * section); > + /* Note that we include User Data 2 which is not protected by ECC */ The more I think about these free bytes, the more I get convinced that we should only expose ECC protected bytes. I honestly do not remember any guarantee regarding these officially stated, the common case (eg. UBI) there is no impact, but maybe it is best to only expose the bytes that are actually protected, since the thresholds are getting lower and lower (and chances to get bitflips there increase with the optimizations of the technology). Do you mind only exposing User Data 1? Thanks, Miquèl
On Thu, Sep 03, 2026 at 12:12:15PM +0200, Miquel Raynal wrote: > On 31/08/2026 at 16:38:10 +01, Nuno Sá <nuno.sa@analog.com> wrote: > > > Add support for the W25N08LW, a 1.8V 8Gbit SPI-NAND made of two > > 4Gbit LUNs, with 4096-byte pages and 256 bytes of spare area. > > > > The chip reuses the KV ECC status helper, but needs its own OOB > > layout because the geometry of its spare area differs from the > > existing Winbond parts. Only half of it is visible while the > > internal ECC is enabled, so the ECC region is reported as > > inaccessible and just the free bytes are exposed. > > > > Assisted-by: Claude:Opus-5 > > Signed-off-by: Nuno Sá <nuno.sa@analog.com> > > --- > > drivers/mtd/nand/spi/winbond.c | 45 ++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 45 insertions(+) > > > > diff --git a/drivers/mtd/nand/spi/winbond.c b/drivers/mtd/nand/spi/winbond.c > > index 9b78c1e6cbc9..c2c4cc306f05 100644 > > --- a/drivers/mtd/nand/spi/winbond.c > > +++ b/drivers/mtd/nand/spi/winbond.c > > @@ -345,6 +345,41 @@ static const struct mtd_ooblayout_ops w25n02kv_ooblayout = { > > .free = w25n02kv_ooblayout_free, > > }; > > > > +static int w25n08lw_ooblayout_ecc(struct mtd_info *mtd, int section, > > + struct mtd_oob_region *region) > > +{ > > + /* > > + * With ecc enabled the parity bits are not accessible. So we can > > + * only see page + 128. Without ecc the full page + 256 is accessible. > > + * To make it simple just return the area as not accessible. > > + */ > > + return -ERANGE; > > +} > > + > > +static int w25n08lw_ooblayout_free(struct mtd_info *mtd, int section, > > + struct mtd_oob_region *region) > > +{ > > + if (section > 7) > > + return -ERANGE; > > + > > + region->offset = (16 * section); > > + /* Note that we include User Data 2 which is not protected by ECC */ > > The more I think about these free bytes, the more I get convinced that > we should only expose ECC protected bytes. I honestly do not remember > any guarantee regarding these officially stated, the common case > (eg. UBI) there is no impact, but maybe it is best to only expose the > bytes that are actually protected, since the thresholds are getting > lower and lower (and chances to get bitflips there increase with the > optimizations of the technology). > > Do you mind only exposing User Data 1? Not at all! Will re-spin with that in mind! Thx! - Nuno Sá > > Thanks, > Miquèl > > ______________________________________________________ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/
diff --git a/drivers/mtd/nand/spi/winbond.c b/drivers/mtd/nand/spi/winbond.c index 9b78c1e6cbc9..c2c4cc306f05 100644 --- a/drivers/mtd/nand/spi/winbond.c +++ b/drivers/mtd/nand/spi/winbond.c @@ -345,6 +345,41 @@ static const struct mtd_ooblayout_ops w25n02kv_ooblayout = { .free = w25n02kv_ooblayout_free, }; +static int w25n08lw_ooblayout_ecc(struct mtd_info *mtd, int section, + struct mtd_oob_region *region) +{ + /* + * With ecc enabled the parity bits are not accessible. So we can + * only see page + 128. Without ecc the full page + 256 is accessible. + * To make it simple just return the area as not accessible. + */ + return -ERANGE; +} + +static int w25n08lw_ooblayout_free(struct mtd_info *mtd, int section, + struct mtd_oob_region *region) +{ + if (section > 7) + return -ERANGE; + + region->offset = (16 * section); + /* Note that we include User Data 2 which is not protected by ECC */ + region->length = 16; + + /* Extract BBM */ + if (!section) { + region->offset += 2; + region->length -= 2; + } + + return 0; +} + +static const struct mtd_ooblayout_ops w25n08lw_ooblayout = { + .ecc = w25n08lw_ooblayout_ecc, + .free = w25n08lw_ooblayout_free, +}; + static int w25n01jw_ooblayout_ecc(struct mtd_info *mtd, int section, struct mtd_oob_region *region) { @@ -781,6 +816,16 @@ static const struct spinand_info winbond_spinand_table[] = { SPINAND_ECCINFO(&w35n01jw_ooblayout, w25w35nxxjw_ecc_get_status), SPINAND_CONFIGURE_CHIP(w35n0xjw_vcr_cfg), SPINAND_CONT_READ(w35n0xjw_set_cont_read)), + /* 8G-bit densities */ + SPINAND_INFO("W25N08LW", /* 2x4G-bit 1.8V */ + SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0xb3, 0x24), + NAND_MEMORG(1, 4096, 256, 64, 2048, 40, 1, 2, 1), + NAND_ECCREQ(8, 512), + SPINAND_INFO_OP_VARIANTS(&read_cache_variants, + &write_cache_variants, + &update_cache_variants), + 0, + SPINAND_ECCINFO(&w25n08lw_ooblayout, w25n02kv_ecc_get_status)), }; static int winbond_spinand_init(struct spinand_device *spinand)
Add support for the W25N08LW, a 1.8V 8Gbit SPI-NAND made of two 4Gbit LUNs, with 4096-byte pages and 256 bytes of spare area. The chip reuses the KV ECC status helper, but needs its own OOB layout because the geometry of its spare area differs from the existing Winbond parts. Only half of it is visible while the internal ECC is enabled, so the ECC region is reported as inaccessible and just the free bytes are exposed. Assisted-by: Claude:Opus-5 Signed-off-by: Nuno Sá <nuno.sa@analog.com> --- drivers/mtd/nand/spi/winbond.c | 45 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 45 insertions(+) --- base-commit: 15a3cbce32994141252bb4ecfe3ff3a5d22d0b4f change-id: 20260831-mtd-nand-new-chip-support-44b10657243f -- Thanks! - Nuno Sá