David Hendricks submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved Edward O'Callaghan: Looks good to me, approved Objections: David Hendricks: I would prefer that you didn't submit this
flashchips: Add support for Boya Microelectronics BY25Q128AS

Tested on Buspirate.

Signed-off-by: Jack Olsen <omegasec@tutanota.com>
Change-Id: I881ba86cfaa82e43c73360135d47c74d896cc191
Reviewed-on: https://review.coreboot.org/c/flashrom/+/44308
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M flashchips.c
M flashchips.h
2 files changed, 42 insertions(+), 0 deletions(-)

diff --git a/flashchips.c b/flashchips.c
index 48efb85..c4ba4c6 100644
--- a/flashchips.c
+++ b/flashchips.c
@@ -3443,6 +3443,44 @@
},

{
+ .vendor = "Boya Microelectronics",
+ .name = "BY25Q128AS",
+ .bustype = BUS_SPI,
+ .manufacture_id = BOYA_ID,
+ .model_id = BOYA_BY25Q128AS,
+ .total_size = 16384,
+ .page_size = 256,
+ .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_QPI,
+ .tested = TEST_OK_PREW,
+ .probe = probe_spi_rdid,
+ .probe_timing = TIMING_ZERO,
+ .block_erasers =
+ {
+ {
+ .eraseblocks = { {4 * 1024, 4096} },
+ .block_erase = spi_block_erase_20,
+ }, {
+ .eraseblocks = { {32 * 1024, 512} },
+ .block_erase = spi_block_erase_52,
+ }, {
+ .eraseblocks = { {64 * 1024, 256} },
+ .block_erase = spi_block_erase_d8,
+ }, {
+ .eraseblocks = { {16 * 1024 * 1024, 1} },
+ .block_erase = spi_block_erase_60,
+ }, {
+ .eraseblocks = { {16 * 1024 * 1024, 1} },
+ .block_erase = spi_block_erase_c7,
+ }
+ },
+ .printlock = spi_prettyprint_status_register_plain,
+ .unlock = spi_disable_blockprotect_at25fs040,
+ .write = spi_chip_write_256,
+ .read = spi_chip_read,
+ .voltage = {2700, 3600},
+ },
+
+ {
.vendor = "Bright",
.name = "BM29F040",
.bustype = BUS_PARALLEL,
diff --git a/flashchips.h b/flashchips.h
index 87aedc2..e9c0432 100644
--- a/flashchips.h
+++ b/flashchips.h
@@ -207,6 +207,10 @@
#define ATMEL_AT49F080 0x23
#define ATMEL_AT49F080T 0x27

+/* Boya Microelectronics Inc.*/
+#define BOYA_ID 0x68
+#define BOYA_BY25Q128AS 0x4018
+
/* Bright Microelectronics has the same manufacturer ID as Hyundai... */
#define BRIGHT_ID 0xAD /* Bright Microelectronics */
#define BRIGHT_BM29F040 0x40

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I881ba86cfaa82e43c73360135d47c74d896cc191
Gerrit-Change-Number: 44308
Gerrit-PatchSet: 9
Gerrit-Owner: Jack Olsen <omegasec@tutanota.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Edward O'Callaghan <edward.ocallaghan@koparo.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: merged