Sergey Alirzaev has uploaded this change for review.

View Change

ft2232_spi: add an ability to use GPIO for chip selection

Change-Id: I6db05619e0d69ad18549c8556ef69225337b1532
Signed-off-by: Sergey Alirzaev <zl29ah@gmail.com>
---
M flashrom.8.tmpl
M ft2232_spi.c
2 files changed, 27 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/11/28911/1
diff --git a/flashrom.8.tmpl b/flashrom.8.tmpl
index 70af395..cbb3378 100644
--- a/flashrom.8.tmpl
+++ b/flashrom.8.tmpl
@@ -722,9 +722,9 @@
Multi-Protocol Adapter (TUMPA), TUMPA Lite, GOEPEL PicoTAP and Google Servo v1/v2.
.sp
An optional parameter specifies the controller
-type and channel/interface/port it should support. For that you have to use the
+type and channel/interface/port/GPIO-based chip select it should support. For that you have to use the
.sp
-.B " flashrom \-p ft2232_spi:type=model,port=interface"
+.B " flashrom \-p ft2232_spi:type=model,port=interface,csgpiol=gpio"
.sp
syntax where
.B model
@@ -733,14 +733,17 @@
arm-usb-tiny ", " arm-usb-tiny-h ", " arm-usb-ocd ", " arm-usb-ocd-h \
", " tumpa ", " tumpalite ", " picotap ", " google-servo ", " google-servo-v2 \
" or " google-servo-v2-legacy
-and
.B interface
can be
-.BR A ", " B ", " C ", or " D .
+.BR A ", " B ", " C ", or " D
+and
+.B csgpiol
+can be a number between 0 and 3, denoting GPIOL0-GPIOL3 correspondingly.
The default model is
.B 4232H
-and the default interface is
-.BR A .
+the default interface is
+.BR A
+and GPIO is not used by default.
.sp
If there is more than one ft2232_spi-compatible device connected, you can select which one should be used by
specifying its serial number with the
diff --git a/ft2232_spi.c b/ft2232_spi.c
index 819744c..79d9d82 100644
--- a/ft2232_spi.c
+++ b/ft2232_spi.c
@@ -327,6 +327,24 @@
}
free(arg);

+ arg = extract_programmer_param("csgpiol");
+ if (arg && strlen(arg)) {
+ unsigned int temp = 0;
+ char *endptr;
+ temp = strtoul(arg, &endptr, 10);
+ if (*endptr || temp > 3) {
+ msg_perr("Error: Invalid GPIOL specified: \"%s\".\n"
+ "Valid values are between 0 and 3.\n", arg);
+ free(arg);
+ return -2;
+ } else {
+ unsigned pin = temp + 4;
+ cs_bits |= 1 << pin;
+ pindir |= 1 << pin;
+ }
+ }
+ free(arg);
+
msg_pdbg("Using device type %s %s ",
get_ft2232_vendorname(ft2232_vid, ft2232_type),
get_ft2232_devicename(ft2232_vid, ft2232_type));

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6db05619e0d69ad18549c8556ef69225337b1532
Gerrit-Change-Number: 28911
Gerrit-PatchSet: 1
Gerrit-Owner: Sergey Alirzaev <zl29ah@gmail.com>