On 10/15/26 07:56, Jamin Lin wrote:
The W25Q32DW and W25Q33PW use the same JEDEC ID (0xef6016), but
have different part names. Since the JEDEC ID is used to uniquely
identify a flash model in QEMU, both parts cannot be added as separate
entries with the same ID.

According to Winbond, the W25Q32DW has been end-of-life for many years
and the W25Q33PW is its next-generation replacement. The W25Q33PW is
compatible with the W25Q32DW and uses the same JEDEC ID.

Rename the existing w25q32dw model to w25q33pw to reflect the currently
supported part.

Signed-off-by: Jamin Lin <[email protected]>
---
  hw/block/m25p80.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
index 545e0b5728..e88efd913b 100644
--- a/hw/block/m25p80.c
+++ b/hw/block/m25p80.c
@@ -353,7 +353,7 @@ static const FlashPartInfo known_devices[] = {
      { INFO("w25x16",      0xef3015,      0,  64 << 10,  32, ER_4K) },
      { INFO("w25x32",      0xef3016,      0,  64 << 10,  64, ER_4K) },
      { INFO("w25q32",      0xef4016,      0,  64 << 10,  64, ER_4K) },
-    { INFO("w25q32dw",    0xef6016,      0,  64 << 10,  64, ER_4K) },
+    { INFO("w25q33pw",    0xef6016,      0,  64 << 10,  64, ER_4K) },


Instead please add a new "w25q33pw" device and keep "w25q32dw".

Thanks,

C.

      { INFO("w25x64",      0xef3017,      0,  64 << 10, 128, ER_4K) },
      { INFO("w25q64",      0xef4017,      0,  64 << 10, 128, ER_4K) },
      { INFO("w25q80",      0xef5014,      0,  64 << 10,  16, ER_4K) },


Reply via email to