Sergey Alirzaev has uploaded this change for review. ( https://review.coreboot.org/28911
Change subject: ft2232_spi: add an ability to use GPIO for chip selection ......................................................................
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));