December 2017 - flashrom-gerrit

Change in flashrom[stable]: Remove djgpp-dos target
by Nico Huber (Code Review) 09 Dec '17

09 Dec '17

Nico Huber has abandoned this change. ( https://review.coreboot.org/21777 ) Change subject: Remove djgpp-dos target ...................................................................... Abandoned Abandoning stable branch. -- To view, visit https://review.coreboot.org/21777 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: stable Gerrit-MessageType: abandon Gerrit-Change-Id: I81598358f85443d1d53c9fdf5c5ae705ef360c0d Gerrit-Change-Number: 21777 Gerrit-PatchSet: 1 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: flashchips: Mark W25Q128.W as tested
by build bot (Jenkins) (Code Review) 08 Dec '17

08 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22790 ) Change subject: flashchips: Mark W25Q128.W as tested ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/916/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/802/ : SUCCESS -- To view, visit https://review.coreboot.org/22790 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0ef331ad3b3a8ab05a9472f3053f76c0789b1f9 Gerrit-Change-Number: 22790 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 08 Dec 2017 22:45:52 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[master]: flashchips: Mark W25Q128.W as tested
by David Hendricks (Code Review) 08 Dec '17

08 Dec '17

David Hendricks has uploaded this change for review. ( https://review.coreboot.org/22790 Change subject: flashchips: Mark W25Q128.W as tested ...................................................................... flashchips: Mark W25Q128.W as tested Tested read, erase, and write using W25Q128FWSIG and Dediprog SF600. Change-Id: Id0ef331ad3b3a8ab05a9472f3053f76c0789b1f9 Signed-off-by: David Hendricks <dhendricks(a)fb.com> --- M flashchips.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/90/22790/1 diff --git a/flashchips.c b/flashchips.c index a1ad445..56b978f 100644 --- a/flashchips.c +++ b/flashchips.c @@ -15054,7 +15054,7 @@ /* supports SFDP */ /* OTP: 1024B total, 256B reserved; read 0x48; write 0x42, erase 0x44, read ID 0x4B */ .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_QPI, - .tested = TEST_UNTESTED, + .tested = TEST_OK_PREW, .probe = probe_spi_rdid, .probe_timing = TIMING_ZERO, .block_erasers = -- To view, visit https://review.coreboot.org/22790 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Id0ef331ad3b3a8ab05a9472f3053f76c0789b1f9 Gerrit-Change-Number: 22790 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com>

1 0

Change in flashrom[master]: flashchips: Add ISSI IS25LP128 and IS25WP128
by build bot (Jenkins) (Code Review) 08 Dec '17

08 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22784 ) Change subject: flashchips: Add ISSI IS25LP128 and IS25WP128 ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/915/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/801/ : SUCCESS -- To view, visit https://review.coreboot.org/22784 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia1c7a9a950043c30b7525196e03ee394689e89a5 Gerrit-Change-Number: 22784 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 08 Dec 2017 07:08:15 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[master]: flashchips: Add ISSI IS25LP128 and IS25WP128
by David Hendricks (Code Review) 08 Dec '17

08 Dec '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/22784 to look at the new patch set (#2). Change subject: flashchips: Add ISSI IS25LP128 and IS25WP128 ...................................................................... flashchips: Add ISSI IS25LP128 and IS25WP128 IS25LP128 is the 3.3V variant, IS25WP128 is the 1.8V variant. Tested read, erase, and write using Dediprog SF600 on each. Change-Id: Ia1c7a9a950043c30b7525196e03ee394689e89a5 Signed-off-by: David Hendricks <dhendricks(a)fb.com> --- M flashchips.c M flashchips.h 2 files changed, 85 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/84/22784/2 -- To view, visit https://review.coreboot.org/22784 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia1c7a9a950043c30b7525196e03ee394689e89a5 Gerrit-Change-Number: 22784 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: flashchips: Add ISSI IS25LP128 and IS25WP128
by David Hendricks (Code Review) 08 Dec '17

08 Dec '17

David Hendricks has abandoned this change. ( https://review.coreboot.org/22783 ) Change subject: flashchips: Add ISSI IS25LP128 and IS25WP128 ...................................................................... Abandoned -- To view, visit https://review.coreboot.org/22783 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: abandon Gerrit-Change-Id: Ia31764f0bf3ba1ed609f83b95e21343d7c23a508 Gerrit-Change-Number: 22783 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: flashchips: Add ISSI IS25LP128 and IS25WP128
by build bot (Jenkins) (Code Review) 08 Dec '17

08 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22784 ) Change subject: flashchips: Add ISSI IS25LP128 and IS25WP128 ...................................................................... Patch Set 1: Verified-1 Build Failed https://qa.coreboot.org/job/flashrom_gerrit/800/ : FAILURE https://qa.coreboot.org/job/flashrom-customrules/914/ : SUCCESS -- To view, visit https://review.coreboot.org/22784 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia1c7a9a950043c30b7525196e03ee394689e89a5 Gerrit-Change-Number: 22784 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 08 Dec 2017 07:06:24 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[master]: flashchips: Add ISSI IS25LP128 and IS25WP128
by David Hendricks (Code Review) 08 Dec '17

08 Dec '17

David Hendricks has uploaded this change for review. ( https://review.coreboot.org/22784 Change subject: flashchips: Add ISSI IS25LP128 and IS25WP128 ...................................................................... flashchips: Add ISSI IS25LP128 and IS25WP128 IS25LP128 is the 3.3V variant, IS25WP128 is the 1.8V variant. Tested read, erase, and write using Dediprog SF600 on each. Change-Id: Ia1c7a9a950043c30b7525196e03ee394689e89a5 Signed-off-by: David Hendricks <dhendricks(a)fb.com> --- M flashchips.c M flashchips.h 2 files changed, 43 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/84/22784/1 diff --git a/flashchips.c b/flashchips.c index a1ad445..3d60a68 100644 --- a/flashchips.c +++ b/flashchips.c @@ -7082,6 +7082,47 @@ { .vendor = "ISSI", + .name = "IS25WP128", + .bustype = BUS_SPI, + .manufacture_id = ISSI_ID_SPI, + .model_id = ISSI_IS25WP128, + .total_size = 16384, + .page_size = 256, + /* OTP: 1024B total; read 0x48; write 0x42 */ + .feature_bits = FEATURE_WRSR_WREN | FEATURE_UNBOUND_READ | FEATURE_OTP, + .tested = TEST_OK_PREWU, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .block_erasers = + { + { + .eraseblocks = { {4 * 1024, 4096} }, + .block_erase = spi_block_erase_20, + }, { + .eraseblocks = { {4 * 1024, 4096} }, + .block_erase = spi_block_erase_d7, + }, { + .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, + } + }, + .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 62eb9a5..de9f3e7 100644 --- a/flashchips.h +++ b/flashchips.h @@ -452,6 +452,8 @@ #define SHARP_LH28F008SC 0xA6 /* Sharp chip, Intel Vendor ID */ #define ISSI_ID 0xD5 /* ISSI Integrated Silicon Solutions, see also PMC. */ +#define ISSI_ID_SPI 0x9D /* ISSI ID used for SPI flash, see also PMC_ID_NOPREFIX */ +#define ISSI_IS25WP128 0x7018 #define ISSI_PMC_IS29GL032B 0xF9 #define ISSI_PMC_IS29GL032T 0xF6 #define ISSI_PMC_IS29GL064B 0x7E1000 -- To view, visit https://review.coreboot.org/22784 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ia1c7a9a950043c30b7525196e03ee394689e89a5 Gerrit-Change-Number: 22784 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com>

1 0

Change in flashrom[master]: flashchips: Add ISSI IS25LP128 and IS25WP128
by build bot (Jenkins) (Code Review) 08 Dec '17

08 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22783 ) Change subject: flashchips: Add ISSI IS25LP128 and IS25WP128 ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/913/ : SUCCESS -- To view, visit https://review.coreboot.org/22783 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia31764f0bf3ba1ed609f83b95e21343d7c23a508 Gerrit-Change-Number: 22783 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 08 Dec 2017 07:05:03 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[master]: flashchips: Add ISSI IS25LP128 and IS25WP128
by David Hendricks (Code Review) 08 Dec '17

08 Dec '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/22783 to look at the new patch set (#2). Change subject: flashchips: Add ISSI IS25LP128 and IS25WP128 ...................................................................... flashchips: Add ISSI IS25LP128 and IS25WP128 IS25LP128 is the 3.3V variant, IS25WP128 is the 1.8V variant. Tested read, erase, and write using Dediprog SF600 on each. Change-Id: Ia31764f0bf3ba1ed609f83b95e21343d7c23a508 Signed-off-by: David Hendricks <dhendricks(a)fb.com> --- M flashchips.c M flashchips.h 2 files changed, 43 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/83/22783/2 -- To view, visit https://review.coreboot.org/22783 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia31764f0bf3ba1ed609f83b95e21343d7c23a508 Gerrit-Change-Number: 22783 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit December 2017