April 2022 - flashrom-gerrit - mail.coreboot.org

Change in flashrom[master]: flashchips: Add f25l16pa
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Attention is currently required from: Paul Menzel. Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63912 ) Change subject: flashchips: Add f25l16pa ...................................................................... Patch Set 1: (1 comment) File flashchips.h: https://review.coreboot.org/c/flashrom/+/63912/comment/77811ce9_2e769fb8 PS1, Line 235: #define ESMT_F25L16PA 0x2015 > Should that be sorted? Yes. Currently, I am just going through the GitHub PRs, look what's useful and push it to Gerrit. -- To view, visit https://review.coreboot.org/c/flashrom/+/63912 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: If80e2bdacc14e94cd70bb0f9dff5f38c9af62ef3 Gerrit-Change-Number: 63912 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Comment-Date: Thu, 28 Apr 2022 06:19:03 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-MessageType: comment

1 0

Change in flashrom[master]: flashchips: Add f25l16pa
by Paul Menzel (Code Review) 28 Apr '22

28 Apr '22

Attention is currently required from: Felix Singer. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63912 ) Change subject: flashchips: Add f25l16pa ...................................................................... Patch Set 1: (1 comment) File flashchips.h: https://review.coreboot.org/c/flashrom/+/63912/comment/90ce710e_73c2d42d PS1, Line 235: #define ESMT_F25L16PA 0x2015 Should that be sorted? -- To view, visit https://review.coreboot.org/c/flashrom/+/63912 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: If80e2bdacc14e94cd70bb0f9dff5f38c9af62ef3 Gerrit-Change-Number: 63912 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Comment-Date: Thu, 28 Apr 2022 06:16:47 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: flashchips: Add Shanghai Fudan Microelectronics Group Company Lmt. FM...
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63914 ) Change subject: flashchips: Add Shanghai Fudan Microelectronics Group Company Lmt. FM25F04 ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: From https://github.com/flashrom/flashrom/pull/51 -- To view, visit https://review.coreboot.org/c/flashrom/+/63914 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I83575ea3a3a8136cf127557add4a08ec17b69445 Gerrit-Change-Number: 63914 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-Comment-Date: Thu, 28 Apr 2022 06:14:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: flashchips: Add Fremont Micro Devices FT25H04
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63913 ) Change subject: flashchips: Add Fremont Micro Devices FT25H04 ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: From https://github.com/flashrom/flashrom/pull/51 -- To view, visit https://review.coreboot.org/c/flashrom/+/63913 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ic27bd2598bf6adb0d83d8005214bd04f03a06c2d Gerrit-Change-Number: 63913 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-Comment-Date: Thu, 28 Apr 2022 06:14:03 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: flashchips: Add Shanghai Fudan Microelectronics Group Company Lmt. FM...
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Felix Singer has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/63914 ) Change subject: flashchips: Add Shanghai Fudan Microelectronics Group Company Lmt. FM25F04 ...................................................................... flashchips: Add Shanghai Fudan Microelectronics Group Company Lmt. FM25F04 Change-Id: I83575ea3a3a8136cf127557add4a08ec17b69445 Signed-off-by: Tomas kovacik <nail(a)nodomain.sk> Signed-off-by: Felix Singer <felixsinger(a)posteo.net> --- M flashchips.c M flashchips.h 2 files changed, 37 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/14/63914/1 diff --git a/flashchips.c b/flashchips.c index 2cbe413..0afa61a 100644 --- a/flashchips.c +++ b/flashchips.c @@ -19876,6 +19876,40 @@ .voltage = {2700, 3600}, }, + { + .vendor = "SFM", /*http://www.zhktl.com/uploadfile/download/20146131545443060.pdf*/ + .name = "FM25F04", + .bustype = BUS_SPI, + .manufacture_id = SFM_ID, + .model_id = SFM_FM25F04, + .total_size = 512, + .page_size = 256, + .feature_bits = FEATURE_WRSR_WREN, + .tested = TEST_UNTESTED, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .block_erasers = + { + { + .eraseblocks = { {4 * 1024, 128} }, + .block_erase = spi_block_erase_20, + }, { + .eraseblocks = { {64 * 1024, 8} }, + .block_erase = spi_block_erase_d8, + }, { + .eraseblocks = { {512 * 1024, 1} }, + .block_erase = spi_block_erase_60, + }, { + .eraseblocks = { {512 * 1024, 1} }, + .block_erase = spi_block_erase_c7, + } + }, + .printlock = spi_prettyprint_status_register_bp2_srwd, + .unlock = spi_disable_blockprotect_bp2_srwd, + .write = spi_chip_write_256, + .read = spi_chip_read, /* Fast read (0x0B) and multi I/O supported */ + .voltage = {2700, 3600}, + }, { diff --git a/flashchips.h b/flashchips.h index 6abfd1f..5115add 100644 --- a/flashchips.h +++ b/flashchips.h @@ -1033,4 +1033,7 @@ #define FMD_ID 0x0E /* Fremont Micro Devices */ #define FMD_FT25H04 0x4013 +#define SFM_ID 0xA1 /*Shanghai Fudan Microelectronics Group Company Limited*/ +#define SFM_FM25F04 0x3113 + #endif /* !FLASHCHIPS_H */ -- To view, visit https://review.coreboot.org/c/flashrom/+/63914 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I83575ea3a3a8136cf127557add4a08ec17b69445 Gerrit-Change-Number: 63914 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: flashchips: Add Fremont Micro Devices FT25H04
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Felix Singer has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/63913 ) Change subject: flashchips: Add Fremont Micro Devices FT25H04 ...................................................................... flashchips: Add Fremont Micro Devices FT25H04 Change-Id: Ic27bd2598bf6adb0d83d8005214bd04f03a06c2d Signed-off-by: Tomas kovacik <nail(a)nodomain.sk> Signed-off-by: Felix Singer <felixsinger(a)posteo.net> --- M flashchips.c M flashchips.h 2 files changed, 40 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/13/63913/1 diff --git a/flashchips.c b/flashchips.c index 0a1d21d..2cbe413 100644 --- a/flashchips.c +++ b/flashchips.c @@ -19842,6 +19842,43 @@ }, { + .vendor = "FMD", /*http://famousconnections.eu/wp-content/uploads/2017/10/FT25H04.pdf*/ + .name = "FT25H04", + .bustype = BUS_SPI, + .manufacture_id = FMD_ID, + .model_id = FMD_FT25H04, + .total_size = 512, + .page_size = 256, + .feature_bits = FEATURE_WRSR_WREN, + .tested = TEST_UNTESTED, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .block_erasers = + { + { + .eraseblocks = { {4 * 1024, 128} }, + .block_erase = spi_block_erase_20, + }, { + .eraseblocks = { {64 * 1024, 8} }, + .block_erase = spi_block_erase_d8, + }, { + .eraseblocks = { {512 * 1024, 1} }, + .block_erase = spi_block_erase_60, + }, { + .eraseblocks = { {512 * 1024, 1} }, + .block_erase = spi_block_erase_c7, + } + }, + .printlock = spi_prettyprint_status_register_bp2_srwd, + .unlock = spi_disable_blockprotect_bp2_srwd, + .write = spi_chip_write_256, + .read = spi_chip_read, /* Fast read (0x0B) and multi I/O supported */ + .voltage = {2700, 3600}, + }, + + + + { .vendor = "Generic", .name = "Variable Size SPI chip", .bustype = BUS_SPI, diff --git a/flashchips.h b/flashchips.h index ba97719..6abfd1f 100644 --- a/flashchips.h +++ b/flashchips.h @@ -1030,4 +1030,7 @@ #define ZETTADEVICE_ZD25D20 0x2012 #define ZETTADEVICE_ZD25D40 0x2013 +#define FMD_ID 0x0E /* Fremont Micro Devices */ +#define FMD_FT25H04 0x4013 + #endif /* !FLASHCHIPS_H */ -- To view, visit https://review.coreboot.org/c/flashrom/+/63913 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ic27bd2598bf6adb0d83d8005214bd04f03a06c2d Gerrit-Change-Number: 63913 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: flashchips: Add f25l16pa
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63912 ) Change subject: flashchips: Add f25l16pa ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: From https://github.com/flashrom/flashrom/pull/64 -- To view, visit https://review.coreboot.org/c/flashrom/+/63912 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: If80e2bdacc14e94cd70bb0f9dff5f38c9af62ef3 Gerrit-Change-Number: 63912 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-Comment-Date: Thu, 28 Apr 2022 06:07:37 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: flashchips: Add f25l16pa
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Felix Singer has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/63912 ) Change subject: flashchips: Add f25l16pa ...................................................................... flashchips: Add f25l16pa Change-Id: If80e2bdacc14e94cd70bb0f9dff5f38c9af62ef3 Signed-off-by: Felix Singer <felixsinger(a)posteo.net> --- M flashchips.c M flashchips.h 2 files changed, 35 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/12/63912/1 diff --git a/flashchips.c b/flashchips.c index 0a1d21d..5f9ef4e 100644 --- a/flashchips.c +++ b/flashchips.c @@ -4433,6 +4433,40 @@ .read = spi_chip_read, .voltage = {2700, 3600}, }, + { + .vendor = "ESMT", + .name = "F25L16PA", + .bustype = BUS_SPI, + .manufacture_id = ESMT_ID, + .model_id = ESMT_F25L16PA, // 0x14 + .total_size = 2048, + .page_size = 256, + .feature_bits = FEATURE_WRSR_EITHER, + .tested = TEST_UNTESTED, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .block_erasers = + { + { + .eraseblocks = { {4 * 1024, 512} }, + .block_erase = spi_block_erase_20, + }, { + .eraseblocks = { {64 * 1024, 32} }, + .block_erase = spi_block_erase_d8, + }, { + .eraseblocks = { {2 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_60, + }, { + .eraseblocks = { {2 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_c7, + } + }, + .printlock = spi_prettyprint_status_register_bp2_bpl, + .unlock = spi_disable_blockprotect, + .write = spi_chip_write_256, + .read = spi_chip_read, + .voltage = {2700, 3600}, + }, { .vendor = "Eon", diff --git a/flashchips.h b/flashchips.h index ba97719..14ff1b8 100644 --- a/flashchips.h +++ b/flashchips.h @@ -232,6 +232,7 @@ #define ESMT_F25L64QA 0x4117 #define ESMT_F25L128QA 0x4118 #define ESMT_F49B002UA 0x00 +#define ESMT_F25L16PA 0x2015 /* * EN25 chips are SPI, first byte of device ID is memory type, -- To view, visit https://review.coreboot.org/c/flashrom/+/63912 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: If80e2bdacc14e94cd70bb0f9dff5f38c9af62ef3 Gerrit-Change-Number: 63912 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: flashchips: Add IS25WQ040
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Attention is currently required from: roman.stingler(a)gmail.com. Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63910 ) Change subject: flashchips: Add IS25WQ040 ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: From https://github.com/flashrom/flashrom/pull/204 -- To view, visit https://review.coreboot.org/c/flashrom/+/63910 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I6f188e006bda415221c571c6aa9cfece649e572f Gerrit-Change-Number: 63910 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: roman.stingler(a)gmail.com Gerrit-Attention: roman.stingler(a)gmail.com Gerrit-Comment-Date: Thu, 28 Apr 2022 05:35:38 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: flashchips: Add IS25WQ040
by Felix Singer (Code Review) 28 Apr '22

28 Apr '22

Attention is currently required from: roman.stingler(a)gmail.com. Hello roman.stingler(a)gmail.com, I'd like you to do a code review. Please visit https://review.coreboot.org/c/flashrom/+/63910 to review the following change. Change subject: flashchips: Add IS25WQ040 ...................................................................... flashchips: Add IS25WQ040 Change-Id: I6f188e006bda415221c571c6aa9cfece649e572f Signed-off-by: Roman Stingler <roman.stingler(a)gmail.com> Signed-off-by: Felix Singer <felixsinger(a)posteo.net> --- M flashchips.c M flashchips.h 2 files changed, 43 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/10/63910/1 diff --git a/flashchips.c b/flashchips.c index 0a1d21d..ac1e986 100644 --- a/flashchips.c +++ b/flashchips.c @@ -7687,6 +7687,48 @@ { .vendor = "ISSI", + .name = "IS25WQ040", + .bustype = BUS_SPI, + .manufacture_id = ISSI_ID_SPI, + .model_id = ISSI_IS25WQ040, + .total_size = 4096, + .page_size = 256, + /* OTP: 1024B total; read 0x48; write 0x42 */ + /* QPI enable 0x35, disable 0xF5 */ + .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, 1024} }, + .block_erase = spi_block_erase_20, + }, { + .eraseblocks = { {4 * 1024, 1024} }, + .block_erase = spi_block_erase_d7, + }, { + .eraseblocks = { {32 * 1024, 128} }, + .block_erase = spi_block_erase_52, + }, { + .eraseblocks = { {64 * 1024, 64} }, + .block_erase = spi_block_erase_d8, + }, { + .eraseblocks = { {4 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_60, + }, { + .eraseblocks = { {4 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_c7, + } + }, + .unlock = spi_disable_blockprotect, + .write = spi_chip_write_256, + .read = spi_chip_read, + .voltage = {1650, 1950}, + }, + + { + .vendor = "ISSI", .name = "IS29GL064B", .bustype = BUS_PARALLEL, .manufacture_id = ISSI_ID, diff --git a/flashchips.h b/flashchips.h index ba97719..cd1ae43 100644 --- a/flashchips.h +++ b/flashchips.h @@ -479,6 +479,7 @@ #define ISSI_IS25WP064 0x7017 #define ISSI_IS25WP128 0x7018 #define ISSI_IS25WP256 0x7019 +#define ISSI_IS25WQ040 0x1253 #define ISSI_PMC_IS29GL032B 0xF9 #define ISSI_PMC_IS29GL032T 0xF6 #define ISSI_PMC_IS29GL064B 0x7E1000 -- To view, visit https://review.coreboot.org/c/flashrom/+/63910 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I6f188e006bda415221c571c6aa9cfece649e572f Gerrit-Change-Number: 63910 Gerrit-PatchSet: 1 Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: roman.stingler(a)gmail.com Gerrit-Attention: roman.stingler(a)gmail.com Gerrit-MessageType: newchange

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit April 2022