All of lore.kernel.org
 help / color / mirror / Atom feed
From: <Takahiro.Kuwano@infineon.com>
To: <miquel.raynal@bootlin.com>, <pratyush@kernel.org>,
	<mwalle@kernel.org>, <richard@nod.at>, <vigneshr@ti.com>,
	<nicolas.ferre@microchip.com>, <alexandre.belloni@bootlin.com>,
	<claudiu.beznea@tuxon.dev>, <corbet@lwn.net>,
	<skhan@linuxfoundation.org>
Cc: <STLin2@winbond.com>, <hsinyi@chromium.org>,
	<thomas.petazzoni@bootlin.com>, <linux-mtd@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>
Subject: RE: [PATCH v4 12/25] mtd: spi-nor: winbond: Add support for W25Q12RV-Q/N
Date: Fri, 4 Sep 2026 09:07:24 +0000	[thread overview]
Message-ID: <a6f90df0e0f742ddbbaea48ec5c0cc59@infineon.com> (raw)
In-Reply-To: <20260902-winbond-v7-1-spi-nor-rv-addition-v4-12-779d35303dff@bootlin.com>

Hi Miquel,

> There is an ID collision with the chip of same density from the JV
> family. Both chips are very similar in practice, it is mostly a matter
> of electrical differences (mostly power consumption being lower).
> 
> As a significant difference, RV chips identify themselves as supporting
> the new SFDP (rev F) field which forces an alternate write SR2
> opcode (0x31).
> 
> This density wronly advertizes its size and requires a fixup for

Typo? 'wrongly'
Same in patch #20.

> [...]
> + mtd_debug info /dev/mtd0
> mtd.type = MTD_NORFLASH
> mtd.flags = MTD_CAP_NORFLASH
> mtd.size = 8388608 (8M)

Is mtd0 a 8MB partition?

Thanks,
Takahiro


WARNING: multiple messages have this Message-ID (diff)
From: <Takahiro.Kuwano@infineon.com>
To: <miquel.raynal@bootlin.com>, <pratyush@kernel.org>,
	<mwalle@kernel.org>, <richard@nod.at>, <vigneshr@ti.com>,
	<nicolas.ferre@microchip.com>, <alexandre.belloni@bootlin.com>,
	<claudiu.beznea@tuxon.dev>, <corbet@lwn.net>,
	<skhan@linuxfoundation.org>
Cc: <STLin2@winbond.com>, <hsinyi@chromium.org>,
	<thomas.petazzoni@bootlin.com>, <linux-mtd@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>
Subject: RE: [PATCH v4 12/25] mtd: spi-nor: winbond: Add support for W25Q12RV-Q/N
Date: Fri, 4 Sep 2026 09:07:24 +0000	[thread overview]
Message-ID: <a6f90df0e0f742ddbbaea48ec5c0cc59@infineon.com> (raw)
In-Reply-To: <20260902-winbond-v7-1-spi-nor-rv-addition-v4-12-779d35303dff@bootlin.com>

Hi Miquel,

> There is an ID collision with the chip of same density from the JV
> family. Both chips are very similar in practice, it is mostly a matter
> of electrical differences (mostly power consumption being lower).
> 
> As a significant difference, RV chips identify themselves as supporting
> the new SFDP (rev F) field which forces an alternate write SR2
> opcode (0x31).
> 
> This density wronly advertizes its size and requires a fixup for

Typo? 'wrongly'
Same in patch #20.

> [...]
> + mtd_debug info /dev/mtd0
> mtd.type = MTD_NORFLASH
> mtd.flags = MTD_CAP_NORFLASH
> mtd.size = 8388608 (8M)

Is mtd0 a 8MB partition?

Thanks,
Takahiro

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

  reply	other threads:[~2026-09-04  9:07 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-02 16:59 [PATCH v4 00/25] mtd: spi-nor: QE handling cleanup + fixup reworks + Winbond RV chips addition Miquel Raynal
2026-09-02 16:59 ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 01/25] mtd: spi-nor: Rename BFPT_DWORD15_QER_SR2_BIT1_BUGGY Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 02/25] mtd: spi-nor: Create a structure containing the flash opcodes Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 03/25] mtd: spi-nor: Refactor Read Status/Write Status support Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 04/25] mtd: spi-nor: Add support for the new JESD216 rev F QER field Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 05/25] mtd: spi-nor: Create the concept of fixup table with match function Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 06/25] mtd: spi-nor: Move the fixup flags into the fixup list Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 07/25] mtd: spi-nor: Create an indirection on the part name Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 08/25] mtd: spi-nor: Move the SFDP header structure to a C header Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 09/25] mtd: spi-nor: winbond: Prepare introduction of W25QxxRV-Q/N parts Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 10/25] mtd: spi-nor: winbond: Add support for W25Q32RV-Q/N Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 11/25] mtd: spi-nor: winbond: Add support for W25Q64RV-Q/N Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 12/25] mtd: spi-nor: winbond: Add support for W25Q12RV-Q/N Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-04  9:07   ` Takahiro.Kuwano [this message]
2026-09-04  9:07     ` Takahiro.Kuwano
2026-09-02 16:59 ` [PATCH v4 13/25] mtd: spi-nor: winbond: Add support for W25Q25RV-Q/N Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 14/25] mtd: spi-nor: winbond: Add support for W25Q51RV-Q/N Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 15/25] mtd: spi-nor: winbond: Add support for W25Q01RV-Q/N Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 16/25] mtd: spi-nor: winbond: Add support for W25Q02RV-Q/N Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 17/25] mtd: spi-nor: winbond: Prepare introduction of W25QxxRV-M parts Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 18/25] mtd: spi-nor: winbond: Add support for W25Q32RV-M Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 19/25] mtd: spi-nor: winbond: Add support for W25Q64RV-M Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 20/25] mtd: spi-nor: winbond: Add support for W25Q12RV-M Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 21/25] mtd: spi-nor: winbond: Add support for W25Q25RV-M Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 22/25] mtd: spi-nor: winbond: Add support for W25Q51RV-M Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 23/25] mtd: spi-nor: winbond: Add support for W25Q01RV-M Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 24/25] mtd: spi-nor: winbond: Add support for W25Q02RV-M Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal
2026-09-02 16:59 ` [PATCH v4 25/25] mtd: spi-nor: debugfs: Expose SR opcodes and QE mask Miquel Raynal
2026-09-02 16:59   ` Miquel Raynal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=a6f90df0e0f742ddbbaea48ec5c0cc59@infineon.com \
    --to=takahiro.kuwano@infineon.com \
    --cc=STLin2@winbond.com \
    --cc=alexandre.belloni@bootlin.com \
    --cc=claudiu.beznea@tuxon.dev \
    --cc=corbet@lwn.net \
    --cc=hsinyi@chromium.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=miquel.raynal@bootlin.com \
    --cc=mwalle@kernel.org \
    --cc=nicolas.ferre@microchip.com \
    --cc=pratyush@kernel.org \
    --cc=richard@nod.at \
    --cc=skhan@linuxfoundation.org \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=vigneshr@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.