Patrick Rudolph would like Patrick Rudolph to review this change.

View Change

dediprog: Fix SF600 4BA mode

Flash chips greater than 16MiB require 4BA mode support.
Current master doesn't support such mode as the SF600 uses

Fix protocol version check to support 4BA on PROTOCOL_V3, too.

Tested on SF600 V:7.2.45
Able to read and write "IS25WP256" (32768 kB, SPI).

Change-Id: I2d3089693fbd2f8f7717a8f71be242be131ce707
Signed-off-by: Patrick Rudolph <>
M dediprog.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh:// refs/changes/29/34629/1
diff --git a/dediprog.c b/dediprog.c
index 3566109..add478d 100644
--- a/dediprog.c
+++ b/dediprog.c
@@ -1160,7 +1160,7 @@
if (dediprog_devicetype == DEV_SF100 && protocol() == PROTOCOL_V1)
spi_master_dediprog.features &= ~SPI_MASTER_NO_4BA_MODES;

- if (protocol() == PROTOCOL_V2)
+ if (protocol() >= PROTOCOL_V2)
spi_master_dediprog.features |= SPI_MASTER_4BA;

if (register_spi_master(&spi_master_dediprog) || dediprog_set_leds(LED_NONE))

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I2d3089693fbd2f8f7717a8f71be242be131ce707
Gerrit-Change-Number: 34629
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <>
Gerrit-Reviewer: Patrick Rudolph <>
Gerrit-MessageType: newchange