[flashrom-gerrit] Change in flashrom[master]: dediprog: Fix SF600 4BA mode

[Patrick Rudolph (Code Review)]

Hello Patrick Rudolph,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/flashrom/+/34629

to review the following change.

Change subject: dediprog: Fix SF600 4BA mode
......................................................................

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
PROTOCOL_V3.

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://review.coreboot.org:29418/flashrom 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))