DZ has uploaded this change for review.

View Change

flashchips: Remove Macronix MX25U25635F from chip list

The MX25U25643G has same ID as MX25U25635F, and the former one is the
next generation of the latter. Therefore, we replace it with MX25U25643G
and backward compatible.

We have tested read/write/erase and write protection function for
MX25U25643G.

MX25U25643G datasheet is available at the following URL:
https://www.mxic.com.tw/en-us/products/NOR-Flash/Serial-NOR-Flash/Pages/spec.aspx?p=MX25U25643G&m=Serial%20NOR%20Flash&n=PM2832

MX25U25635F datasheet is available at the following URL:
https://www.macronix.com/en-us/products/NOR-Flash/Serial-NOR-Flash/Pages/spec.aspx?p=MX25U25635F&m=Serial%20NOR%20Flash&n=PM1712

Change-Id: Ief3fd7641bed817066692c4abffff6d3b0df16b9
Signed-off-by: danielzhang <danielzhang@mxic.com.cn>
---
M flashchips.c
M include/flashchips.h
2 files changed, 2 insertions(+), 51 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/33/79633/1
diff --git a/flashchips.c b/flashchips.c
index a4605bd..8ae2680 100644
--- a/flashchips.c
+++ b/flashchips.c
@@ -10257,59 +10257,10 @@

{
.vendor = "Macronix",
- .name = "MX25U25635F",
- .bustype = BUS_SPI,
- .manufacture_id = MACRONIX_ID,
- .model_id = MACRONIX_MX25U25635F,
- .total_size = 32768,
- .page_size = 256,
- /* OTP: 512B total; enter 0xB1, exit 0xC1 */
- .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_QPI | FEATURE_4BA,
- .tested = TEST_OK_PREW,
- .probe = PROBE_SPI_RDID,
- .probe_timing = TIMING_ZERO,
- .block_erasers =
- {
- {
- .eraseblocks = { {4 * 1024, 8192} },
- .block_erase = SPI_BLOCK_ERASE_21,
- }, {
- .eraseblocks = { {4 * 1024, 8192} },
- .block_erase = SPI_BLOCK_ERASE_20,
- }, {
- .eraseblocks = { {32 * 1024, 1024} },
- .block_erase = SPI_BLOCK_ERASE_5C,
- }, {
- .eraseblocks = { {32 * 1024, 1024} },
- .block_erase = SPI_BLOCK_ERASE_52,
- }, {
- .eraseblocks = { {64 * 1024, 512} },
- .block_erase = SPI_BLOCK_ERASE_DC,
- }, {
- .eraseblocks = { {64 * 1024, 512} },
- .block_erase = SPI_BLOCK_ERASE_D8,
- }, {
- .eraseblocks = { {32 * 1024 * 1024, 1} },
- .block_erase = SPI_BLOCK_ERASE_60,
- }, {
- .eraseblocks = { {32 * 1024 * 1024, 1} },
- .block_erase = SPI_BLOCK_ERASE_C7,
- }
- },
- /* TODO: security register */
- .printlock = SPI_PRETTYPRINT_STATUS_REGISTER_BP3_SRWD, /* bit6 is quad enable */
- .unlock = SPI_DISABLE_BLOCKPROTECT_BP3_SRWD,
- .write = SPI_CHIP_WRITE256, /* Multi I/O supported */
- .read = SPI_CHIP_READ, /* Fast read (0x0B) and multi I/O supported */
- .voltage = {1650, 2000},
- },
-
- {
- .vendor = "Macronix",
.name = "MX25U25643G",
.bustype = BUS_SPI,
.manufacture_id = MACRONIX_ID,
- .model_id = MACRONIX_MX25U25635F,
+ .model_id = MACRONIX_MX25U25643G,
.total_size = 32768,
.page_size = 256,
/* OTP: 512B total; enter 0xB1, exit 0xC1 */
diff --git a/include/flashchips.h b/include/flashchips.h
index 17a6fc4..654196f 100644
--- a/include/flashchips.h
+++ b/include/flashchips.h
@@ -529,7 +529,7 @@
#define MACRONIX_MX25U3235E 0x2536 /* Same as MX25U6435F */
#define MACRONIX_MX25U6435E 0x2537 /* Same as MX25U6435F */
#define MACRONIX_MX25U12835E 0x2538 /* Same as MX25U12835F */
-#define MACRONIX_MX25U25635F 0x2539 /* Same as MX25U25643G */
+#define MACRONIX_MX25U25643G 0x2539
#define MACRONIX_MX25U51245G 0x253a
#define MACRONIX_MX25L3235D 0x5E16 /* MX25L3225D/MX25L3235D/MX25L3237D */
#define MACRONIX_MX25L6495F 0x9517

To view, visit change 79633. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: Ief3fd7641bed817066692c4abffff6d3b0df16b9
Gerrit-Change-Number: 79633
Gerrit-PatchSet: 1
Gerrit-Owner: DZ
Gerrit-MessageType: newchange