Change in flashrom[master]: Revert "ft2232_spi.c: Rename cs_bits to pinlvl"

[Nico Huber (Code Review)]

Nico Huber has submitted this change. ( https://review.coreboot.org/c/flashrom/+/55691 )

Change subject: Revert "ft2232_spi.c: Rename cs_bits to pinlvl"
......................................................................

Revert "ft2232_spi.c: Rename cs_bits to pinlvl"

This reverts commit 180079632b15758ca0bbbe5782611bd4573e11d8.

The renaming only seemed to match the code because of earlier introduced
regressions (see following reverts). For proper support of the `gpiol`
feature, we'd likely need both a `cs_bits` and a `pinlvl` variable.

Change-Id: Ifa5b2259ccf49ddf729d01176bacd94a95c39925
Signed-off-by: Nico Huber 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/55691
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
Reviewed-by: Edward O'Callaghan 
---
M ft2232_spi.c
1 file changed, 14 insertions(+), 14 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved
  Edward O'Callaghan: Looks good to me, approved

diff --git a/ft2232_spi.c b/ft2232_spi.c
index 97794b5..3757a24 100644
--- a/ft2232_spi.c
+++ b/ft2232_spi.c
@@ -86,7 +86,7 @@
 #define BITMODE_BITBANG_NORMAL	1
 #define BITMODE_BITBANG_SPI	2

-/* The variables pinlvl and pindir store the values for the "set data bits low byte" MPSSE command that
+/* The variables cs_bits and pindir store the values for the "set data bits low byte" MPSSE command that
  * sets the initial state and the direction of the I/O pins. The pin offsets are as follows:
  * TCK/SK is bit 0.
  * TDI/DO is bit 1.
@@ -105,7 +105,7 @@
  *    dir: 0x0b  CS=output, DI=input, DO=output, SK=output
  */
 struct ft2232_data {
-	uint8_t pinlvl;
+	uint8_t cs_bits;
    uint8_t pindir;
    struct ftdi_context ftdic_context;
 };
@@ -226,7 +226,7 @@

msg_pspew("Assert CS#\n");
    	buf[i++] = SET_BITS_LOW;
-		buf[i++] = ~ 0x08 & spi_data->pinlvl; /* assert CS (3rd) bit only */
+		buf[i++] = ~ 0x08 & spi_data->cs_bits; /* assert CS (3rd) bit only */
    	buf[i++] = spi_data->pindir;

/* WREN, OP(PROGRAM, ERASE), ADDR, DATA */
@@ -248,7 +248,7 @@
    	/* Add final de-assert CS# */
    	msg_pspew("De-assert CS#\n");
    	buf[i++] = SET_BITS_LOW;
-		buf[i++] = spi_data->pinlvl;
+		buf[i++] = spi_data->cs_bits;
    	buf[i++] = spi_data->pindir;

/* continue if there is no read-cmd and further cmds exist */
@@ -316,7 +316,7 @@
    char *arg;
    double mpsse_clk;

-	uint8_t pinlvl = 0x08;
+	uint8_t cs_bits = 0x08;
    uint8_t pindir = 0x0b;
    struct ftdi_context ftdic;
    struct ft2232_data *spi_data;
@@ -338,7 +338,7 @@
    		/* JTAGkey(2) needs to enable its output via Bit4 / GPIOL0
    		*  value: 0x18  OE=high, CS=high, DI=low, DO=low, SK=low
    		*    dir: 0x1b  OE=output, CS=output, DI=input, DO=output, SK=output */
-			pinlvl = 0x18;
+			cs_bits = 0x18;
    		pindir = 0x1b;
    	} else if (!strcasecmp(arg, "picotap")) {
    		ft2232_vid = GOEPEL_VID;
@@ -356,7 +356,7 @@
    		/* In its default configuration it is a jtagkey clone */
    		ft2232_type = FTDI_FT2232H_PID;
    		channel_count = 2;
-			pinlvl = 0x18;
+			cs_bits = 0x18;
    		pindir = 0x1b;
    	} else if (!strcasecmp(arg, "openmoko")) {
    		ft2232_vid = FIC_VID;
@@ -369,7 +369,7 @@
    		/* arm-usb-ocd(-h) has an output buffer that needs to be enabled by pulling ADBUS4 low.
    		*  value: 0x08  #OE=low, CS=high, DI=low, DO=low, SK=low
    		*    dir: 0x1b  #OE=output, CS=output, DI=input, DO=output, SK=output */
-			pinlvl = 0x08;
+			cs_bits = 0x08;
    		pindir = 0x1b;
    	} else if (!strcasecmp(arg, "arm-usb-tiny")) {
    		ft2232_vid = OLIMEX_VID;
@@ -380,7 +380,7 @@
    		ft2232_type = OLIMEX_ARM_OCD_H_PID;
    		channel_count = 2;
    		/* See arm-usb-ocd */
-			pinlvl = 0x08;
+			cs_bits = 0x08;
    		pindir = 0x1b;
    	} else if (!strcasecmp(arg, "arm-usb-tiny-h")) {
    		ft2232_vid = OLIMEX_VID;
@@ -473,7 +473,7 @@
    			return -2;
    		} else {
    			unsigned int pin = temp + 4;
-				pinlvl |= 1 << pin;
+				cs_bits |= 1 << pin;
    			pindir |= 1 << pin;
    		}
    	}
@@ -490,11 +490,11 @@
    			unsigned int pin = i + 4;
    			switch (toupper(arg[i])) {
    				case 'H':
-						pinlvl |= 1 << pin;
+						cs_bits |= 1 << pin;
    					pindir |= 1 << pin;
    					break;
    				case 'L':
-						pinlvl &= ~(1 << pin);
+						cs_bits &= ~(1 << pin);
    					pindir |= 1 << pin;
    					break;
    				case 'Z':
@@ -600,7 +600,7 @@

msg_pdbg("Set data bits\n");
    buf[0] = SET_BITS_LOW;
-	buf[1] = pinlvl;
+	buf[1] = cs_bits;
    buf[2] = pindir;
    if (send_buf(&ftdic, buf, 3)) {
    	ret = -8;
@@ -612,7 +612,7 @@
    	msg_perr("Unable to allocate space for SPI master data\n");
    	return SPI_GENERIC_ERROR;
    }
-	spi_data->pinlvl = pinlvl;
+	spi_data->cs_bits = cs_bits;
    spi_data->pindir = pindir;
    spi_data->ftdic_context = ftdic;